Projet

Général

Profil

Development #1588

Possibilité de définir des statuts "invisibles" dans les workflows

Ajouté par Victor Claudet il y a plus de 11 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
21 août 2012
Echéance:
% réalisé:

100%

Temps estimé:
Patch proposed:
Planning:

Description

Au moment de la définition d'un nouveau statut dans un workflow, il faudrait pouvoir préciser si ce statut sera visible ou non par l'usager. L'intérêt est de pouvoir cacher aux usagers les étapes de relances internes lors de demandes "hors délais"


Fichiers

Historique

#1

Mis à jour par Frédéric Péters il y a plus de 11 ans

Fred: "par statut "invisible", en fait, tu voudrais qu'un pan de l'évolution dans le workflow ne soit pas visible à l'utilisateur ?"
Victor: oui, c'est ça.

Ensuite, pour la réalisation, il y a plusieurs options, Victor pensait à une case à cocher "visible à l'usager", attachée au statut.

De mon côté je pencherais plutôt pour marquer les objets de type Evolution, et tant qu'à faire d'y ajouter un champ "roles", où None serait "visible à tout le monde", mais où pourrait également se trouver autre chose (le cas commun étant alors ['receiver']). Ce qui pourrait modifier ça, ce serait une action de workflow ("limiter la visibilité" c'est pas top, faudrait trouver autre chose).

Il y a des avantages et des inconvénients dans les deux approches. Par exemple un avantage de l'approche de Victor c'est que ça peut être modifié après coup, juste au niveau du workflow, alors que dans mon approche ça signifierait devoir modifier tous les formdata, en exécutant un bout de workflow, ce qui est juste pas possible.

#2

Mis à jour par Victor Claudet il y a plus de 11 ans

Comme je le disais à Fred, la mise en place des statuts de workflow est très simple et doit le rester. De mon point de vue l'ajout de la case à cocher est le plus simple et le fait que l'activation ou non soit réversible me parait important. Donc personnellement je penche pour la solution case à cocher.

#3

Mis à jour par Thomas Noël il y a plus de 11 ans

L'approche "case à cocher" au niveau du statut me parait effectivement plus simple à appréhender à l'usage.

Pour la réalisation, j'aime bien des "rôles autorisés" de Fred : on accroche au statut une liste des rôles qui peuvent voir le statut. Par défaut c'est None = tout le monde, sinon ['receiver'] si la case "Seulement le destinataire" est cochée.

A noter que lors d'une relance hors-délais, le demandeur verra un affichage de son historique un peu bizarre :
  1. statut : Votre demande est à l'étude
  2. (statut caché = relance après timeout, qui envoie les mails d'alerte puis revient au statut "à l'étude")
  3. statut : Votre demande est à l'étude

Autrement dit, l'utilisateur verra deux fois le statut "votre demande est à l'étude", ça lui fera penser qu'il y a un bogue, mais bon...

#4

Mis à jour par Benjamin Dauvergne il y a plus de 11 ans

écrivait:

La demande #1588 a été mise à jour par Thomas Noël.

L'approche "case à cocher" au niveau du statut me parait effectivement plus simple à appréhender à l'usage.

Pour la réalisation, j'aime bien des "rôles autorisés" de Fred : on accroche au statut une liste des rôles qui peuvent voir le statut. Par défaut c'est None = tout le monde, sinon ['receiver'] si la case "Seulement le destinataire" est cochée.

A noter que lors d'une relance hors-délais, le demandeur verra un affichage de son historique un peu bizarre :
  1. statut : Votre demande est à l'étude
  2. (statut caché = relance après timeout, qui envoie les mails d'alerte puis revient au statut "à l'étude")
  3. statut : Votre demande est à l'étude

On pourrait nettoyer l'évolution pour l'affichage i.e. supprimer les
statuts redondant une fois les statuts techniques enlevés.

#5

Mis à jour par Victor Claudet il y a plus de 11 ans

une remarque au passage, si le statut est privé, les commentaires éventuellement postés dans ceux ci doivent également l'être.

#6

Mis à jour par Thomas Noël il y a plus de 11 ans

  • Version cible changé de Au-quotidien 2012.2 à Au-quotidien 2012.3
#7

Mis à jour par Frédéric Péters il y a plus de 11 ans

  • Fichier 0001-workflows-possibility-to-hide-some-status-from-the-u.patch ajouté
#8

Mis à jour par Frédéric Péters il y a plus de 11 ans

  • Statut changé de Nouveau à Solution déployée
#9

Mis à jour par Thomas Noël il y a plus de 11 ans

Marche bien... mais il faudrait indiquer quelque part dans l'interface quand un statut n'est pas "visible pour tout le monde".

Du genre

if self.status.visibility:
    '(hidden)'

Mais je ne sais pas où faire cet affichage, à vrai dire...

#10

Mis à jour par Frédéric Péters il y a plus de 11 ans

Nouvelle version qui ajoute une ligne "This status is hidden from the user." au-dessus des actions du workflow.

#11

Mis à jour par Frédéric Péters il y a plus de 11 ans

  • Fichier 0001-workflows-possibility-to-hide-some-status-from-the-u.patch supprimé
#12

Mis à jour par Thomas Noël il y a plus de 11 ans

Ca me va. Adopté !

#13

Mis à jour par Frédéric Péters il y a plus de 11 ans

  • Statut changé de Solution déployée à Résolu (à déployer)

Poussé.

#14

Mis à jour par Thomas Noël il y a plus de 11 ans

... et posé sur auquo-test

#15

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

  • % réalisé changé de 0 à 100
#16

Mis à jour par Frédéric Péters il y a plus de 10 ans

  • Statut changé de Résolu (à déployer) à Fermé
#17

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Version cible Au-quotidien 2012.3 supprimé

Formats disponibles : Atom PDF