matrix-org / matrix-org/matrix-python-sdk
Move features of MatrixHttpApi related to application-services into subclass
未关闭
还没有人认领这个 Issue。
Api layer
architecture
breaking
enhancement
- 主要语言
- Python
- 星标
- 266
- 派生
- 118
- PR 合并指标
- 30 天内没有已合并 PR
描述
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
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
调研方向
首先阅读 MatrixHttpApi 和 #143 中的讨论,然后查看 #168 中相关的 async 提案。当 application-service 功能被移入可组合的子类或扩展中,并且 issue 中尚未解决的 application-service 与 async 行为组合问题有了明确的解决方案时,即视为完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- python
- 领域
- api, backend-api-design
- Issue 类型
- 重构
- 难度
- 5/5
- 预计耗时
- 一周以上
- 活跃度
- 停滞
- 描述清晰度
- 需要澄清
- 新手友好度
- 25/100