Development #7324
agent/authentic2: construire le nom du service en préfixant avec le type de service
0%
Fichiers
Révisions associées
environment: fix typo in Wcs.Meta.verbose_name_plural (#7324)
agent/authentic2: build service provider name by concatenating service-label and title (#7324)
misc: force service label to be a string (re #7324)
Revert "agent/authentic2: build service provider name by concatenating service-label and title (#7324)"
This reverts commit dec4784c4f370ff3cc37ad14b793c7bd3876a115.
Historique
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
Mis à jour par Frédéric Péters il y a presque 9 ans
Unack.
Du côté de la page d'accueil d'Hobo on essayait de fournir un nom intelligible plutôt que le nom de l'application (le verbose_name des modèles de hobo/environment/models.py), ex: "Fournisseur d'identités Authentic", "Téléformulaires w.c.s", "Porte-document Fargo", etc. (et limite j'en éliminerais même le nom de l'application, qui ne parle qu'à nous)
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
On pourrait passer ce verbose name dans un "service-id-label" ou un truc du genre que je puisse m'en servir coté authentic ?
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Fichier 0001-environment-add-an-Extra.service_label-field-7324.patch 0001-environment-add-an-Extra.service_label-field-7324.patch ajouté
- Fichier 0002-environment-fix-typo-in-Wcs.Meta.verbose_name_plural.patch 0002-environment-fix-typo-in-Wcs.Meta.verbose_name_plural.patch ajouté
- Fichier 0003-agent-authentic2-build-service-provider-name-by-conc.patch ajouté
Nouveau patch:
1. ajoute un champ service_label dans ServiceBase.Extra
2. en passant corrige une typo sur Wcs.Meta.verbose_name_plural
3. modifie authentic2/hobo_deploy.py pour construire le nom du SP en concaténant service-label et title
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Fichier
0003-agent-authentic2-build-service-provider-name-by-conc.patchsupprimé
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
- Fichier 0003-agent-authentic2-build-service-provider-name-by-conc.patch 0003-agent-authentic2-build-service-provider-name-by-conc.patch ajouté
Il manquait un espace dans la chaîne de formatage.
Mis à jour par Frédéric Péters il y a presque 9 ans
Très certainement ne pas appeler .title(); mais surtout parce que ça fait cinq jours et que (désolé) j'ai déjà oublié le contexte, il pourrait être mis dans la description du ticket ? (parce qu'en fait le nom du provider, il apparait à l'utilisateur, par exemple lors du logout, du coup c'est sans doute pas terrible d'y mettre le nom de l'application, mais c'est peut-être pour un affichage ailleurs).
Mis à jour par Benjamin Dauvergne il y a presque 9 ans
C'est suite à discussion avec Pierre, qui voulait appeler appeler les objets services d'authentic des instances, ce que je n'aimais pas. J'ai proposé application mais il a demandé à ce qu'on voit bien de quel type d'instance on cause dans l'admin. N'ayant pas de champ type sur mes services je me suis dit que j'allais ajouter ça au titre.
Concernant ce qui s'affiche sur la page de logout, je suis d'accord que "w.c.s. - Clapiers" c'est pas terrible mais surtout à cause du "w.c.s.", est-ce qu'on donnerait des labels plus parlant ? Ou alors on les rendrait adaptable par exemple "wcs_service_label"="Téléservices" dans les variables ? Ou alors rien de tout ça et on institue une règle comme quoi que dans le titre il faut mettre quelque-chose de plus intelligible que "Clapiers". À discuter.
Mis à jour par Frédéric Péters il y a presque 9 ans
commit ce1abcbc5af8200ca062ec86e8a73dda00f013e3 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Jun 11 15:40:54 2015 +0200 misc: force service label to be a string (re #7324)
et.
commit d5eadee13e39322e02edda069e6c2d7abfc464f9 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Thu Jun 11 15:42:03 2015 +0200 Revert "agent/authentic2: build service provider name by concatenating service-label and title (#7324)" This reverts commit dec4784c4f370ff3cc37ad14b793c7bd3876a115.
le temps de discuter.
Mis à jour par Benjamin Dauvergne il y a plus de 8 ans
- Statut changé de Nouveau à Rejeté
Je crois que ce n'est plus nécessaire, je rejette.
environment: add an Extra.service_label field (#7324)
This new field convey the service "label" for building title in other
applications. It's exported in the service-label key of the service
dictionnary.