Développement #65842
authenticators, ne pas permettre d'activer si la configuration n'est pas terminée
0%
Description
Le formulaire d'ajout permet de créer un objet auquel il manque des champs requis, c'est plus sûr de ne pas permettre l'activation tant que la configuration n'est pas complète.
Files
Associated revisions
authenticators: forbid enabling if configuration is not complete (#65842)
History
Updated by Valentin Deniaud over 2 years ago
- File 0002-authenticators-forbid-enabling-if-configuration-is-n.patch 0002-authenticators-forbid-enabling-if-configuration-is-n.patch added
- File 0001-authenticators-redirect-to-edit-view-on-creation-654.patch 0001-authenticators-redirect-to-edit-view-on-creation-654.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
0001 petite amélioration au passage, 0002 le patch.
Updated by Thomas Noël (congés → 5 décembre) over 2 years ago
Valentin Deniaud a écrit :
0001 petite amélioration au passage, 0002 le patch.
Est-ce qu'on peut pas arriver à avoir un authenticators qui fonctionne, activé, et on flingue un bout de sa configuration, et pas on ne peut plus le désactiver ?
Updated by Valentin Deniaud over 2 years ago
Thomas Noël a écrit :
Valentin Deniaud a écrit :
0001 petite amélioration au passage, 0002 le patch.
Est-ce qu'on peut pas arriver à avoir un authenticators qui fonctionne, activé, et on flingue un bout de sa configuration, et pas on ne peut plus le désactiver ?
Je ne vois pas comment, ici tout ce qu'on fait c'est jouer la validation du modèle qui vérifie que les champs avec blank=False sont remplis, et on ne pourra pas les déremplir.
Updated by Thomas Noël (congés → 5 décembre) over 2 years ago
J'ai demandé à mon voisin et j'ai pigé.
Je propose que sur la page de présentation du servie affiche un message « La configuration n'est pas complète, ce système ne peut pas être activé. » lorsque "not object.has_valid_configuration"
Updated by Valentin Deniaud over 2 years ago
Updated by Thomas Noël (congés → 5 décembre) over 2 years ago
- Status changed from Solution proposée to Solution validée
Tout bon.
(Je me dis quand même qu'en toute «cohérence», authenticator.enabled devrait être faux quand has_valid_configuration l'est aussi, mais ça sera pour un prochain jour, et un autre ticket)
Updated by Valentin Deniaud over 2 years ago
- Status changed from Solution validée to Résolu (à déployer)
commit 00e8114452b08b203afb953a412a1be9b6da87c9
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Jun 1 10:39:28 2022 +0200
authenticators: forbid enabling if configuration is not complete (#65842)
commit 2bd4c5e622ff85cd809163a95e59d1fb948576fa
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Jun 1 10:29:18 2022 +0200
authenticators: redirect to edit view on creation (#65842)Updated by Transition automatique over 2 years ago
- Status changed from Résolu (à déployer) to Solution déployée
authenticators: redirect to edit view on creation (#65842)