matrix-org / matrix-python-sdk

Matrix Client-Server SDK for Python 2 and 3

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Move features of MatrixHttpApi related to application-services into subclass

non-Jedi opened this issue · comments

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