RFC: Merge project `approved` and `approval_requested` flags into a single field
还没有人认领这个 Issue。
评估
- 难度
- 5/5
- 预计耗时
- 一周以上
- 新手友好度
- 25/100
- Issue 类型
- 重构
- 描述清晰度
- 基本清楚
- 活跃度
- 停滞
- 技术栈
- elixir
调研方向
首先检查当前的 approved 和 approval_requested 字段，并追踪它们的生命周期在 Elixir/Phoenix API 中的体现位置。将提议的状态转换与现有项目模型和 API 行为进行比较；当就替代字段的设计及其迁移和兼容性影响达成一致后，即视为完成。
由索引模型根据 Issue 内容生成。
描述
Problem
Looking at these two fields, it really seems they serve a singular purpose.
# A project is created
`approved: false, approval_requested: false`
# User submits project for approval
`approved: false, approval_requested: true`
# Admin approves project
`approved: true, approval_requested :does_not_matter`
A single field, for example,
# 'created' -> 'pending_approval' -> 'approved'
field :onboarding_status, :string, null: false, default: 'created'
Might serve our purposes much better.
We could add an additional status of rejected later, with an additional field of rejected_reason or something.
- 主要语言
- Elixir
- 星标
- 234
- 派生
- 82
- PR 合并指标
- 30 天内没有已合并 PR
贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
code-corps/code-corps-api 的其他 Issue
-
难度 5/5 一周以上 新手友好度 15/100
code-corps/code-corps-api#1385 ·
-
难度 3/5 1-2 天 新手友好度 25/100
code-corps/code-corps-api#1384 ·
-
难度 4/5 3-5 天 新手友好度 35/100
code-corps/code-corps-api#1383 · 2 条评论 ·
-
难度 3/5 1-2 天 新手友好度 35/100
code-corps/code-corps-api#1382 · 2 条评论 ·
-
Difficulty: Medium Skill: Elixir Time: 4 hours
code-corps/code-corps-api#1372 · 3 条评论 · 已指派 1 人 ·
查看 code-corps/code-corps-api 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 75/100
carverauto/serviceradar#4596 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
-
bug
难度 2/5 1-3 小时 新手友好度 65/100
-
难度 2/5 1-3 小时 新手友好度 75/100
agentjido/jido_harness#80 ·
-
难度 2/5 1-3 小时 新手友好度 70/100
sevenseacat/cinder#235 ·