Use correct site when loading settings
还没有人认领这个 Issue。
评估
调研方向
首先定位那些明确加载默认 Site 的 GoogleApiSettings 和 MailchimpApiSettings 使用处,然后追踪每个调用方是否具有页面或请求上下文。识别需要 dependency injection 的调用方;完成标准是:在多站点上下文中,settings 能解析出相关 Site,并且在上下文可用时不会回退到默认 Site。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Python
- 星标
- 761
- 派生
- 153
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 没有贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
coderedcorp/coderedcms 的其他 Issue
-
Type: Bug
难度 1/5 1 小时以内 新手友好度 78/100
coderedcorp/coderedcms#714 ·
-
Type: Bug
难度 1/5 1 小时以内 新手友好度 65/100
coderedcorp/coderedcms#711 ·
-
Type: Bug
难度 3/5 1-2 天 新手友好度 45/100
coderedcorp/coderedcms#713 · 1 条评论 ·
-
Form submissions return 404 with Wagtail 7.x due to FormMixin isinstance check可能已有人在做 @bradrice 于 85 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 55/100
coderedcorp/coderedcms#710 · 4 条评论 ·
-
难度 2/5 1-3 小时 新手友好度 58/100
coderedcorp/coderedcms#709 ·
查看 coderedcorp/coderedcms 的全部 Issue
相似的 Issue
-
enhancement good first issue
难度 2/5 1-3 小时 新手友好度 78/100
-
python-version
难度 1/5 1 小时以内 新手友好度 88/100
-
bug
难度 2/5 1-3 小时 新手友好度 62/100
维护者通常 1 天内回复
-
bug javascript P2-medium python release:v3.1
难度 2/5 1-3 小时 新手友好度 68/100
adrirubio/claude-deck#546 ·
维护者通常 1 天内回复
-
area: desktop area: website priority: P2 type: feature
难度 2/5 1-3 小时 新手友好度 62/100
appandflow/stim#3411 · 1 条评论 ·
维护者通常 1 天内回复