matrix-org / matrix-org/matrix-python-sdk
Move features of MatrixHttpApi related to application-services into subclass
Nessuno ha ancora preso questa issue.
- Lingua principale
- Python
- Stelle
- 266
- Fork
- 118
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
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
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia leggendo MatrixHttpApi e la discussione in #143, quindi esamina la proposta async correlata in #168. Il lavoro è completato quando la funzionalità di application-service è stata spostata in una sottoclasse o estensione componibile e la combinazione irrisolta nell'issue tra application-service e il comportamento async ha una soluzione definita.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- python
- Ambito
- api, backend-api-design
- Tipo di issue
- Refactoring
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100