RFC: Is organization/project approval still relevant?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 20/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- elixir
- Área
- authorization, backend
Línea de trabajo
Empieza por las acciones de index/show de organizaciones y proyectos y por el sistema de invitaciones descritos en el issue. Compara las opciones de aprobación keep, drop y combinada con las columnas approved existentes; el trabajo estará terminado cuando se haya decidido una política de aprobación y se haya acordado el alcance de la implementación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
We added an approved column to projects and organizations almost a year ago. I'm not sure if we had the organization invite system in mind when adding it.
The way I see it, the columns could still be relevant.
Keep the columns
- once a user creates an organization and a project, the ember app tells them the two "still need to be approved by an administrator"
- the ember app indicates/shows reduced UI for unapproved projects/organizations
There is then the question of how to most effectively filter out unapproved records
Option A
- the index actions for organizations and projects should only list approved items, if no filters are applied
- the show action and the index actions when filtered to specific ids, should still list all records
- that means unapproved records are unlisted, but can still be reached by direct url, where they are shown in a reduced state
Option B
- index and show should be scoped to current user
- owner can see all approved and their own unapproved records
- regular user can only see approved records
- this means unapproved records are not reachable by regular users
Drop the columns
- Organizations can only be created by admins or via invites - no changes other than normal milestone changes.
- Project creation creates a problem of organization owner being able to create multiple projects once they create the organization. We could fix by
- adding a project limit of 1 to each organization
- we can always manually increase the limit for select organizations, if we want to
A potential problem with dropping the approved columns will be that the user could always do something bad, even a user we trust enough to send an invite to.
Once (hopefully) CodeCorps gets big enough, we will probably have direct org/project creation, not via invites. We'll need an approval system at that point anyway.
We could also do both
We could keep the columns in, but auto-approve an organization created via invite and also maybe auto-approve the first project created by such organization.
- Lenguaje dominante
- Elixir
- Estrellas
- 234
- Forks
- 82
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de code-corps/code-corps-api
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
code-corps/code-corps-api#1385 ·
-
Cannot Create new GitHub App Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
code-corps/code-corps-api#1384 ·
-
Track anon Users that register Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
code-corps/code-corps-api#1383 · 2 comentarios ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
code-corps/code-corps-api#1382 · 2 comentarios ·
-
Difficulty: Medium Skill: Elixir Time: 4 hours
code-corps/code-corps-api#1372 · 3 comentarios · 1 asignado ·
Todos los issues de code-corps/code-corps-api
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
carverauto/serviceradar#4542 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
phoenixframework/phoenix#6847 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
beyond-all-reason/teiserver#1584 ·