RFC: Merge project `approved` and `approval_requested` flags into a single field
まだ誰も着手していません。
評価
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 初心者へのやさしさ
- 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 にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
code-corps/code-corps-api のほかの issue
-
難易度 5/5 1週間以上 初心者へのやさしさ 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
-
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 ·
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
phoenixframework/phoenix#6847 ·