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

Move features of MatrixHttpApi related to application-services into subclass

Ouverte
#204 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Api layer architecture breaking enhancement
Langage dominant
Python
Étoiles
266
Forks
118
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

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

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par lire MatrixHttpApi et la discussion dans #143, puis examinez la proposition async associée dans #168. Le travail sera considéré comme terminé lorsque la fonctionnalité de application-service aura été déplacée dans une sous-classe ou une extension composable, et qu’une solution définie aura été apportée à la combinaison non résolue dans l’issue entre application-service et le comportement async.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
api, backend-api-design
Type d'issue
Refactorisation
Difficulté
5/5
Temps estimé
Plus d'une semaine
Activité
À l'abandon
Clarté
À clarifier
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.