Project

General

Profile

Développement #65842

authenticators, ne pas permettre d'activer si la configuration n'est pas terminée

Added by Valentin Deniaud over 2 years ago. Updated over 2 years ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
01 June 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision 2bd4c5e6 (diff)
Added by Valentin Deniaud over 2 years ago

authenticators: redirect to edit view on creation (#65842)

Revision 00e81144 (diff)
Added by Valentin Deniaud over 2 years ago

authenticators: forbid enabling if configuration is not complete (#65842)

History

#2

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 ?

#3

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.

#4

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"

#6

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)

#7

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)
#8

Updated by Transition automatique over 2 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#9

Updated by Transition automatique over 2 years ago

Automatic expiration

Also available in: Atom PDF