Formulaire personnalisé de réinitialisation de mot de passe

thubrecht commented

2021-06-10 11:16:18 +02:00

(Migrated from git.eleves.ens.fr)

Closes #19

mpepin commented

2021-10-02 17:43:55 +02:00

(Migrated from git.eleves.ens.fr)

Salut Tom,

Il y a deux truc que je comprends mal :

Dans quel cas est-ce qu'on ne voudrait pas qu'un utilisateur puisse réinitialiser son mot de passe ?
Pourquoi est-ce que ça devrait se passer dans les formulaires ? Si on ne veut pas qu'un utilisateur puisse modifier son mot de passe, pourquoi ne pas simplement lui renvoyer une 403 oui lui afficher un message dans la vue ? Ça me semble plus naturel et ça évite d'ajouter un truc dans authens.

Salut Tom, Il y a deux truc que je comprends mal : - Dans quel cas est-ce qu'on ne voudrait pas qu'un utilisateur puisse réinitialiser son mot de passe ? - Pourquoi est-ce que ça devrait se passer dans les formulaires ? Si on ne veut pas qu'un utilisateur puisse modifier son mot de passe, pourquoi ne pas simplement lui renvoyer une 403 oui lui afficher un message dans la vue ? Ça me semble plus naturel et ça évite d'ajouter un truc dans authens.

thubrecht commented

2021-10-02 17:49:48 +02:00

(Migrated from git.eleves.ens.fr)

C'est pas vraiment pour empêcher certains utilisateurs de réinitialiser, c'est que quand tu rentres ton mail ça contacte tous les utilisateurs qui ont cet email, ce qui veut dire sur kadenios tous les utilisateurs spécifiques à une élection, et la sélection des utilisateurs à contacter est faite dans le formulaire

mpepin commented

2021-10-02 17:59:51 +02:00

(Migrated from git.eleves.ens.fr)

Okay je comprends, mais ça me paraît être un cas d'usage un peu spécifique.
Je suis un peu réticent à l'idée de rajouter du code dans authens pour ça alors qu'on peut faire autrement :

C'est une class-based view donc dans Kadenios tu peux hériter de authens.views.PasswordResetView et surcharger la méthode get_form_class à cet endroit.
Après il faut ajuster les urls du coup mais ça me semble moins ad-hoc et un peu plus « à la Django »

Tu en penses quoi ? Tu penses que ton patch serait utile ailleurs que dans kadenios ?

Okay je comprends, mais ça me paraît être un cas d'usage un peu spécifique. Je suis un peu réticent à l'idée de rajouter du code dans authens pour ça alors qu'on peut faire autrement : C'est une class-based view donc dans Kadenios tu peux hériter de `authens.views.PasswordResetView` et surcharger la méthode `get_form_class` à cet endroit. Après il faut ajuster les urls du coup mais ça me semble moins ad-hoc et un peu plus « à la Django » Tu en penses quoi ? Tu penses que ton patch serait utile ailleurs que dans kadenios ?

thubrecht commented

2021-10-02 18:06:50 +02:00

(Migrated from git.eleves.ens.fr)

Pour surcharger et faire une classe qui hérite je veux bien, mais le souci c'est que la view d'authens sera toujours accessible, et ça me paraît relou, sinon une autre solution c'est de passer des utilisateurs inactifs et bidouiller comme ça

mpepin commented

2021-10-02 19:05:24 +02:00

(Migrated from git.eleves.ens.fr)

Tu peux masquer l'URL en mettant par exemple path("authens/logout", my_custom_view) plus haut dans la liste des urls.
Alors, la vue d'authens sera plus accessible

Tu peux masquer l'URL en mettant par exemple `path("authens/logout", my_custom_view)` plus haut dans la liste des urls. Alors, la vue d'authens sera plus accessible

thubrecht commented

2021-10-04 10:15:50 +02:00

(Migrated from git.eleves.ens.fr)

Passer les users en inactifs a fait le travail en fait

thubrecht commented

2021-10-04 10:15:58 +02:00

(Migrated from git.eleves.ens.fr)

resolved all threads

mpepin commented

2021-10-04 13:00:40 +02:00

(Migrated from git.eleves.ens.fr)

Nice, encore mieux !

Pull request closed

Please reopen this pull request to perform a merge.

Formulaire personnalisé de réinitialisation de mot de passe #55

Pull request closed