Move workshop domain rules out of Admin::WorkshopsController
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
This umbrella issue is decomposed into #2939, #2940, and #2941, so start with the relevant sub-issue rather than changing it directly. Read Admin::WorkshopsController and Workshop, then follow the named methods and destroy path. Done means each independently shippable extraction is implemented and the controller no longer owns the specified domain rules.
由索引模型根据 Issue 内容生成。
描述
Problem
Admin::WorkshopsController is the largest controller in the app (~295 lines) and holds domain logic below the presentation layer:
- Domain validation in the controller (
workshop_type_valid?adds a model error from the controller) - Deletion policy in the controller (4-hour window + no-invitees rule)
- Organiser grant/revoke role diffing in private controller methods
Plan
This decomposes into three independent, individually shippable changes, tracked as sub-issues:
- #2939 — Move the "physical workshop needs a host" validation onto
Workshop - #2940 — Add
Workshop#deletable?and use it inAdmin::WorkshopsController#destroy - #2941 — Extract organiser grant/revoke diffing from
Admin::WorkshopsController
Each can land separately; no ordering required.
- 主要语言
- Ruby
- 星标
- 106
- 派生
- 208
- 平均合并
- 19 小时 51 分钟
- 30 天内合并 PR
- 57
环境准备
- 提供 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
codebar/planner 的其他 Issue
-
Preload workshop_host in DashboardQuery.upcoming_events_for_user (per-workshop N+1 on /dashboard)可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭performance
难度 2/5 1-3 小时 新手友好度 82/100
codebar/planner#3070 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
enhancement
难度 4/5 3-5 天 新手友好度 48/100
维护者通常 1 天内回复
-
难度 3/5 半天 新手友好度 35/100
codebar/planner#3069 · 1 条评论 · 1 个 reaction ·
维护者通常 1 天内回复
-
Dual Invitiation issue可能已有人在做 @mroderick 于 20 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 42/100
维护者通常 1 天内回复
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
swombat/souls-house#795 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 67/100
-
Missing hi translation: components/common/CommunitySolution.tsx:communitySolution.numTimesStarred未关闭
难度 1/5 1 小时以内 新手友好度 63/100
维护者通常 1 天内回复
-
Tracking: SQLite usage and the Windows GVL/SQLite deadlock (lich-5#2254): moonwatch.lic, low risk未关闭
难度 2/5 1-3 小时 新手友好度 76/100
elanthia-online/dr-scripts#7968 · 2 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 67/100
fullsend-ai/agents#1901 ·
维护者通常 1 天内回复