matrix-org / matrix-org/matrix-python-sdk
Move features of MatrixHttpApi related to application-services into subclass
Personne n'a encore pris cette issue.
- 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
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- 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