Projet

Général

Profil

Development #47154

Afficher le caractère final d'un statut et en permettre le changement si possible

Ajouté par Thomas Noël il y a plus de 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
30 septembre 2020
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision d6816b90 (diff)
Ajouté par Frédéric Péters il y a presque 3 ans

general: save session at the very end (#47154)

It's necessary for native django responses as rendering will not go
through CompatWcsPublisher.

Révision e424eebb (diff)
Ajouté par Frédéric Péters il y a presque 3 ans

backoffice: use a template to render workflow status page (#47154)

Révision b1811b58 (diff)
Ajouté par Frédéric Péters il y a presque 3 ans

backoffice: display note about endpoint situation of status (#47154)

Révision 682ad691 (diff)
Ajouté par Frédéric Péters il y a presque 3 ans

translations: adjust force terminal status (#47154)

Historique

#1

Mis à jour par Frédéric Péters il y a presque 3 ans

  • Assigné à mis à Frédéric Péters
#3

Mis à jour par Frédéric Péters il y a presque 3 ans

À 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).

#4

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 ?)

#5

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.
#6

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

Formats disponibles : Atom PDF