Development #65478
écran configuration méthode "mot de passe", afficher le paramétrage
0%
Description
Sur un écran de configuration d'un service oidc on affiche les différents paramètres, sur l'écran de la méthode "mot de passe" on affiche juste un message "Click "Edit" to change configuration."; ça serait bien de plutôt afficher la configuration.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 2 ans
On affiche ce message seulement si la configuration est vide, tu préférerais un message plus explicite qui dise que la configuration actuelle est celle par défaut ?
Alternativement afficher le paramétrage même vide ?
Condition d'affichage : Aucun Durée de la session : Aucun Inclure le choix de la collectivité dans le formulaire de connexion : Non
Mis à jour par Frédéric Péters il y a presque 2 ans
Alternativement afficher le paramétrage même vide ?
Plutôt, oui. (mais des lignes d'exemple données il est clair que ça va demander différents ajustements pour ne pas avoir simplement None affiché comme "Aucun").
Mis à jour par Valentin Deniaud il y a presque 2 ans
Et tu es OK avec la manière dont ça fonctionne actuellement pour OIDC ? Certains champs (estimés importants) sont repris dans la description, d'autres ne sont visibles qu'en allant éditer.
Si on suit ce principe ici ça voudrait dire ne pas parler de durée de session ni de choix de la collectivité, du moins quand ces champs sont vides, parce qu'ils le seront sur 99% des instances. De là ce ticket deviendrait juste « lorsqu'il n'y a pas de condition d'affichage configurée, le dire explicitement dans la description ».
Mis à jour par Frédéric Péters il y a presque 2 ans
Certains champs (estimés importants) sont repris dans la description, d'autres ne sont visibles qu'en allant éditer.
Je n'ai pas de problème avec ça.
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-authenticators-view-show-condition-even-if-empty-654.patch 0001-authenticators-view-show-condition-even-if-empty-654.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a presque 2 ans
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-authenticators-view-show-condition-even-if-empty-654.patch 0001-authenticators-view-show-condition-even-if-empty-654.patch ajouté
Rebasé
Mis à jour par A. Berriot il y a plus d'un an
Une typo sur show condtion
je crois, dans le pgettext_lazy, sinon ça me parait bon :)
Mis à jour par Valentin Deniaud il y a plus d'un an
- Fichier 0001-authenticators-view-show-condition-even-if-empty-654.patch 0001-authenticators-view-show-condition-even-if-empty-654.patch ajouté
Yep, bien vu !
Mis à jour par Serghei Mihai il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Go dès que jenkins est vert.
Mis à jour par Valentin Deniaud il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit e2e2a42d297ef932ef14e09c9e7aba9e398d4368 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Tue Jun 7 14:30:42 2022 +0200 authenticators: view show condition even if empty (#65478)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
authenticators: view show condition even if empty (#65478)