coderedcorp / coderedcorp/coderedcms
Use correct site when loading settings
- Ngôn ngữ chính
- Python
- Star
- 765
- Fork
- 154
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
When loading wagtail settings, such as `GoogleApiSettings` or `MailchimpApiSettings` we have a tendency to do these in a place where no `Site` is available and explicitly load the default site. This is a bad practice as it totally breaks the concept of multi-site.
This practice needs to be refactored to get the site from the page or request, and if those are not available, use dependency injection techniques.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Hướng nghiên cứu
Bắt đầu bằng cách xác định các cách sử dụng GoogleApiSettings và MailchimpApiSettings có tải rõ ràng Site mặc định, sau đó truy vết xem mỗi caller có ngữ cảnh trang hoặc request hay không. Xác định các caller cần dependency injection; hoàn thành có nghĩa là settings phân giải Site tương ứng trong các ngữ cảnh multisite mà không fallback về Site mặc định khi có ngữ 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ệ
- django, python
- Lĩnh vực
- backend
- Loại issue
- Tái cấu trúc
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- 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
- 35/100