matrix-org / matrix-org/matrix-python-sdk
Move features of MatrixHttpApi related to application-services into subclass
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Python
- Star
- 266
- Fork
- 118
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
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
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu bằng việc đọc MatrixHttpApi và cuộc thảo luận trong #143, sau đó xem xét đề xuất async liên quan trong #168. Công việc được xem là hoàn tất khi chức năng application-service được chuyển vào một lớp con hoặc phần mở rộng có thể kết hợp, đồng thời sự kết hợp chưa được giải quyết trong issue giữa application-service và hành vi async có một giải pháp được xác định.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- python
- Lĩnh vực
- api, backend-api-design
- Loại issue
- Tái cấu trúc
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Cần làm rõ
- Mức phù hợp với người mới
- 25/100