matrix-org / matrix-org/matrix-python-sdk

Move features of MatrixHttpApi related to application-services into subclass

Abierto
#204 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Api layer architecture breaking enhancement
Lenguaje dominante
Python
Estrellas
266
Forks
118
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

See discussion on #143.

In general, I want to move the sdk more towards a set of composable classes with a clear api for extending them. Something where parameters specific to application-service usage live either in an additional kwarg `extras` or in slurped kwargs `**extras` seems like the right direction for this.

Issues to be resolved still:

- [ ] What if somebody wants to combine functionality available on two different subclasses of `MatrixHttpApi` (e.g. application-service support and async as in #168)? We should support composing those together somehow.

Maybe in addition to swapping out `_send` we should have a list of decorators that get applied to `_send`?

cc @Cadair

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza leyendo MatrixHttpApi y la discusión en #143; después revisa la propuesta relacionada sobre async en #168. Se considera terminado cuando la funcionalidad de application-service se haya trasladado a una subclase o extensión componible y la combinación no resuelta del issue entre application-service y el comportamiento async tenga una solución definida.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
python
Área
api, backend-api-design
Tipo de issue
Refactorización
Dificultad
5/5
Tiempo estimado
Más de una semana
Estado de actividad
Estancado
Claridad
Necesita aclaración
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.