entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

workflows: include digest for created forms displayed in history (#74213) #93

Merged
fpeters merged 1 commits from wip/74213-create-formdata-display-digest into main 2023-02-10 09:15:15 +01:00
Conversation 0 Commits 1 Files Changed 4 +20 -7
fpeters commented 2023-02-07 10:31:17 +01:00
Owner
Copy Link

Dans les tests, petit ajout à test_create_formdata_show_link_in_history, qui fait correspondre davantage les assert au commentaire "show link to created formdata" qui précède. (dans une préversion du patch le changement était plus important)

C'est très assez simple, je voulais réutiliser la méthode get_display_label() mais ça faisait passer le texte de "Nouvelle demande «xxx» créée : 12-345" avec juste 12-345 comme lien en "Nouvelle demande créée : xxx 12-235 (digest)", avec "xxx 12-235 (digest)" comme lien.

C'était un peu une marche arrière par rapport à #57714 qui était passé de "Créé une nouvelle demande <a>Inscription aux activités 1-2871</a>" à "Nouvelle demande «Inscription aux activités» : <a>1-2871</a>" donc voilà un nouveau paramètre à get_display_label(), pour permettre de ne pas inclure le nom de la démarche.

Dans les tests, petit ajout à test_create_formdata_show_link_in_history, qui fait correspondre davantage les assert au commentaire "show link to created formdata" qui précède. (dans une préversion du patch le changement était plus important) C'est très assez simple, je voulais réutiliser la méthode get_display_label() mais ça faisait passer le texte de "Nouvelle demande «xxx» créée : 12-345" avec juste 12-345 comme lien en "Nouvelle demande créée : xxx 12-235 (digest)", avec "xxx 12-235 (digest)" comme lien. C'était un peu une marche arrière par rapport à #57714 qui était passé de `"Créé une nouvelle demande <a>Inscription aux activités 1-2871</a>"` à `"Nouvelle demande «Inscription aux activités» : <a>1-2871</a>"` donc voilà un nouveau paramètre à get_display_label(), pour permettre de ne pas inclure le nom de la démarche.
fpeters added 1 commit 2023-02-07 10:31:18 +01:00
lguerin approved these changes 2023-02-07 14:48:41 +01:00
fpeters merged commit f4a000a22f into main 2023-02-10 09:15:15 +01:00
fpeters deleted branch wip/74213-create-formdata-display-digest 2023-02-10 09:15:15 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
lguerin
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/wcs#93
No description provided.
Delete Branch "wip/74213-create-formdata-display-digest"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?