Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto Assign role admin #62

Merged
merged 4 commits into from
Aug 16, 2021
Merged

Auto Assign role admin #62

merged 4 commits into from
Aug 16, 2021

Conversation

Juerodriguez
Copy link
Contributor

@Juerodriguez Juerodriguez commented Aug 4, 2021

Resumen

close Juerodriguez#25
follow #11

In spanish better:

Pensando en la manera de simplificar la asignacion del role admin para el usuario que no tenga muchos conocimientos en informatica como la modificacion manual de la base de datos, se decidio este cambio para asignar automaticamente el role de administrador al primer usuario que descarga la aplicacion.

Checklist

  • Variables, funciones y comentarios en Español
  • Corrector ortográfico ok
  • Cdigo Python cumple PEP8
  • Cobertura de tests con Playwright
  • Revisores asignados (pares & al menos 1 mentor)

Screenshots

(vista rápida, preferir png, gif animado y/o video de Playwright)

@eamanu
Copy link
Collaborator

eamanu commented Aug 14, 2021

Pensando en la manera de simplificar la asignacion del role admin para el usuario que no tenga muchos conocimientos en informatica como la modificacion manual de la base de datos, se decidio este cambio para asignar automaticamente el role de administrador al primer usuario que descarga la aplicacion.

Comentario sin necesidad de ser aplicado: Se podría pensar (en un futuro) en que:

  1. hacer que durante la instalación haya un paso de setear la contraseña del admin
  2. que existe un usuario admin con una password conocida, y que el sistema detecte que esa password no cambió y le pida que la cambie.

(este podría ser una mejora para futuro)

@eamanu eamanu merged commit 051647f into PyAr:master Aug 16, 2021
@eamanu
Copy link
Collaborator

eamanu commented Aug 16, 2021

Great Job @Juerodriguez. Thanks for you contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Define the admin role for the first registered user
2 participants