Development #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.
Fichiers
Révisions associées
authenticators: forbid enabling if configuration is not complete (#65842)
Historique
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0002-authenticators-forbid-enabling-if-configuration-is-n.patch 0002-authenticators-forbid-enabling-if-configuration-is-n.patch ajouté
- Fichier 0001-authenticators-redirect-to-edit-view-on-creation-654.patch 0001-authenticators-redirect-to-edit-view-on-creation-654.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
0001 petite amélioration au passage, 0002 le patch.
Mis à jour par Thomas Noël il y a presque 2 ans
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 ?
Mis à jour par Valentin Deniaud il y a presque 2 ans
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.
Mis à jour par Thomas Noël il y a presque 2 ans
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"
Mis à jour par Valentin Deniaud il y a presque 2 ans
Mis à jour par Thomas Noël il y a presque 2 ans
- Statut changé de Solution proposée à 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)
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution validée à 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)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
authenticators: redirect to edit view on creation (#65842)