RFC: Merge project `approved` and `approval_requested` flags into a single field

Abierto
#1,279 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
5/5
Tiempo estimado
Más de una semana
Aptitud para principiantes
25/100
Tipo de issue
Refactorización
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
elixir
Área
backend-api-design

Línea de trabajo

Comienza revisando los campos actuales approved y approval_requested y rastreando dónde se representa su ciclo de vida en la API de Elixir/Phoenix. Compara las transiciones de estado propuestas con el modelo de proyecto existente y el comportamiento de la API; el trabajo se considerará terminado cuando exista un diseño acordado para el campo de reemplazo y sus implicaciones de migración y compatibilidad.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

needs clarification needs estimate RFC

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.

Lenguaje dominante
Elixir
Estrellas
234
Forks
82
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de code-corps/code-corps-api

Todos los issues de code-corps/code-corps-api

Issues similares

Más issues de Elixir

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.