Development #47154
Afficher le caractère final d'un statut et en permettre le changement si possible
0%
Description
Si on a un statut qui n'a qu'un saut automatique, il n'est pas considéré comme final.
Mais si on ajoute une condition sur le saut, le statut peut en réalité être final...
Sans modifier le calcul de la finalité, je propose qu'on affiche à droite, à la place de « Changer le caractère final du statut », selon le calcul :- Statut final [changer] (bouton "changer" présent uniquement si le caractère final a été imposé)
- Statut non final [changer]
Fichiers
Révisions associées
backoffice: use a template to render workflow status page (#47154)
backoffice: display note about endpoint situation of status (#47154)
translations: adjust force terminal status (#47154)
Historique
Mis à jour par Frédéric Péters il y a presque 3 ans
- Fichier 0003-backoffice-display-note-about-endpoint-situation-of-.patch 0003-backoffice-display-note-about-endpoint-situation-of-.patch ajouté
- Fichier 0002-backoffice-use-a-template-to-render-workflow-status-.patch 0002-backoffice-use-a-template-to-render-workflow-status-.patch ajouté
- Fichier 0001-general-save-session-at-the-very-end-47154.patch 0001-general-save-session-at-the-very-end-47154.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
À rebours,
- 0003 fait l'affichage, pas tout à fait comme suggéré dans le ticket, il affiche "This status has been manually set to be considered as terminal." quand c'est forcé, "This status has been automatically evaluated as being terminal." quand c'est calculé automatique. Quand c'est forcé il y a un lien "Unforce Terminal Status" sous le texte, pour ouvrir la popup pour modifier ça, autrement le lien ne bouge pas reste au milieu des autres (mais son libellé devient "Force Terminal Status").
- 0002 passe avant et transforme la vue pour que le rendu soit fait par un gabarit.
- 0001 sonne indépendant, assure l'enregistrement de la session en fin de requête, en effet ça n'avait pas lieu pour les vues "django native", et du coup le jeton du formulaire de la barre latérale n'était pas enregistré et ça empêchait l'ajout d'action.
Il y a une branche -bis qui fait juste 0002 et 0003, mais passe le formulaire "rendu" dans le contexte du gabarit, pour être là suffisamment tôt. Ça fonctionne mais ça me semble juste remettre à plus tard le 0001 (et on pourrait se demander pourquoi faire le rendu à ce moment et changer et revivre le problème).
Mis à jour par Thomas Noël il y a presque 3 ans
- Statut changé de Solution proposée à Solution validée
Ça marche bien. Je valide.
Note que je ne suis pas forcément à l'aise avec le "infowarning" qui donne un peu l'impression d'un problème. Comme toujours en terme d'UI je n'ai rien d'autre à proposer. À la rigueur un div simple (sans classe) ça m'irait presque.
Pour le "Force Terminal Status" je ne le voyais pas apparaître, mais c'est lié à la traduction qui n'est pas géniale actuellement :
#: admin/workflows.py:883 msgid "Force Terminal Status" msgstr "Caractère final du statut"
... il faudra penser à changer cela lors de la prochaine mise à jour du django.po (peut-être le faire ici dans un mini patch qui ne corrige que cette ligne ?)
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
C'est juste "infonotice", pas warning ou error, et je trouve utile d'attirer l'attention (sans doute parce que ticket lié je suis passé à côté trop longtemps).
J'ai tapé un mini-commit d'ajustement sur la traduction derrière; et je vois de toute façon pour la traduction générale avant de tagguer.
commit b1811b58e4dbca87871bd9d1ef55863021f89691 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue May 4 09:08:11 2021 +0200 backoffice: display note about endpoint situation of status (#47154) commit e424eebbba3597f64dbca5b6ad0ef10ed10c0d24 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon May 3 23:28:36 2021 +0200 backoffice: use a template to render workflow status page (#47154) commit d6816b90adceea9312216791a821ea283c63bf73 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue May 4 10:18:36 2021 +0200 general: save session at the very end (#47154) It's necessary for native django responses as rendering will not go through CompatWcsPublisher.
Mis à jour par Frédéric Péters il y a presque 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
general: save session at the very end (#47154)
It's necessary for native django responses as rendering will not go
through CompatWcsPublisher.