Project

General

Profile

Development #61602

Ajout de message d'aide à la configuration d'un service CAS

Added by Benjamin Renard 12 months ago. Updated 13 days ago.

Status:
En cours
Priority:
Normal
Category:
-
Target version:
-
Start date:
09 February 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

D'expérience, lorsqu'il s'agit d'apprendre à quelqu'un à déclarer un nouveau service CAS, les champs "URLs" et "Nom de l'attribut" (mal-nommé au passage) doivent être explicités. Je vous propose dans le patch joint de modifier le libellé "Nom de l'attribut" et d'ajouter un court message d'aide à la saisie pour ces deux champs.


Files

History

#1

Updated by Thomas Noël 12 months ago

  • Status changed from Nouveau to En cours
  • Assignee set to Benjamin Renard

Pour faire bien, il faut aussi modifier les migrations, ici comme rien ne concerne une modif SQL on peut "bêtement" patcher authentic2_idp_cas/migrations/0001_initial.py pour y ajouter les help_text, modifier verbose_name, etc

Ensuite, nous demanderons juste l'ajout d'un "License: MIT" dans le message de commit.

Avec ces modifs, je verrais à faire l'intégration.

#2

Updated by Benjamin Renard 12 months ago

Thomas Noël a écrit :

Pour faire bien, il faut aussi modifier les migrations, ici comme rien ne concerne une modif SQL on peut "bêtement" patcher authentic2_idp_cas/migrations/0001_initial.py pour y ajouter les help_text, modifier verbose_name, etc

Ensuite, nous demanderons juste l'ajout d'un "License: MIT" dans le message de commit.

Avec ces modifs, je verrais à faire l'intégration.

Voilà. J'ai généré le fichier de la migration via un authentic2-ctl makemigrations, j'espère que cela t'ira.

#3

Updated by Frédéric Péters 12 months ago

Ça va péter sur black, tu pourrais appliquer black ? (on a une config pre-commit pour gérer ça automatiquement, il suffit d'avoir le paquet pre-commit installé).

Aussi, tu pourras mettre en commentaire de ce ticket la traduction que tu souhaites pour ce texte ?

#4

Updated by Benjamin Renard 13 days ago

Frédéric Péters a écrit :

Ça va péter sur black, tu pourrais appliquer black ? (on a une config pre-commit pour gérer ça automatiquement, il suffit d'avoir le paquet pre-commit installé).

Aussi, tu pourras mettre en commentaire de ce ticket la traduction que tu souhaites pour ce texte ?

Voilà une nouvelle version du patch avec black appliqué.

Also available in: Atom PDF