RFC: Merge project `approved` and `approval_requested` flags into a single field
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
- Tipo di issue
- Refactoring
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- elixir
- Ambito
- backend-api-design
Direzione di ricerca
Inizia esaminando i campi attuali approved e approval_requested e tracciando dove viene rappresentato il loro ciclo di vita nell’API Elixir/Phoenix. Confronta le transizioni di stato proposte con il modello di progetto esistente e con il comportamento dell’API; il lavoro sarà considerato completato quando sarà disponibile un design concordato per il campo sostitutivo e per il relativo impatto sulla migrazione e sulla compatibilità.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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.
- Lingua principale
- Elixir
- Stelle
- 234
- Fork
- 82
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di code-corps/code-corps-api
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 15/100
code-corps/code-corps-api#1385 ·
-
Cannot Create new GitHub App Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 25/100
code-corps/code-corps-api#1384 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
code-corps/code-corps-api#1383 · 2 commenti ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
code-corps/code-corps-api#1382 · 2 commenti ·
-
Difficulty: Medium Skill: Elixir Time: 4 hours
code-corps/code-corps-api#1372 · 3 commenti · 1 assegnatario ·
Tutte le issue di code-corps/code-corps-api
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
phoenixframework/phoenix#6847 ·
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 64/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
moderation
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
beyond-all-reason/teiserver#1584 · 1 commento ·