Configurer l'affichage des messages de retours #58

Suite au changement de politique des messages de retour (#44) les gens n'ont pas les mêmes attentes suivant les sites et même suivant les formulaires. Il faudrait donc pouvoir choisir :

si on veut tous les messages de retour
ou seulement celui personnalisé

Il faut :

choisir lequel des deux est le comportement par défaut
que l'autre se fasse si une case cochée, sous la rédaction du message perso

Je n'ai pas d'avis arrêté sur lequel devrait être le par défaut :

d'après les retours, il me semble que les gens s'attendent plutôt à ce s'ils personnalisent le message, il n'y ait que ça, et si on veut tout, on coche une case
mais dans le même temps, si des traitements ajoutent des infos importantes et/ou du JS, ça veut dire que concrètement ça va pas marcher si on n'a pas pensé à cocher la case en plus, et ça les gens peuvent pas du tout le savoir sauf par essai/erreur

Donc ça pourrait être tout par défaut, et la case pour dire "seulement le message personnalisé".

Je plussoie dans ce sens.

Donc ça pourrait être tout par défaut, et la case pour dire "seulement le message personnalisé".

Oui, en assortissant d'une mise en garde tout de même : « en cochant cette case je comprends que ça peut tout casser et je m'engage à ne pas demander de support dans ce cas. »

Bon mais sans rire, une courte mise en garde pour prévenir des dommages collatéraux possibles.

@tcharlss la future version de formidable permettra de mettre les scripts à part, pour pas qu'ils passent par propre.

Je m'occupe par ailleurs de ce ticket.

Release prévue demain.

Et ce sera beaucoup mieux et moins "bidouille" :)

Du coup, le comportement par défaut reviendrait à ce qu'il était l'origine, à savoir : si on personnalise le message de retour, il n'y a que ça d'affiché ?

Je ne suis pas convaincu, car je pense que ce qu'on avait été a mon avis un mauvais comportement. Tu peux avoir des messages de retours lié à des traitements qui sont nécessaires pour expliciter ce qui s'est passé.

Nous avons recu votre réponse
Elle a été envoyé au service xxx
Vous avez été inscrit à la tache YYY

etc

Oui mais qui sont des cas exceptionnels, pas la règle générale (d'où les interrogations et "signalements de bugs" exprimés par moins deux utilisateurs).

bon, oki. Perso je m'en fiche. mais je pense que les "signalements de bugs" sont surtout lié au changement brutal :)

bon, vu l'avis initial de Rasta, je part sur ca

Voilà c'est fait. Du coup on a passé aussi le formulaire/editer_formulaire en mode "déclaration par full saisies".

Exemple pour @tcharlss avec formidable_mailsubscriber

Cas 0 : pas de messge personnalisé

Cas 1 (par défaut) : je ne coche pas la case "afficher le message de retours des traitements individuels"

Cas 2 : je coche la case "afficher le message de retours des traitements individuels"

Addendum

Il reste encore un petit bug, mais c'est plus lié à saisies : spip-contrib-extensions/saisies#78. Ca n'a aucune conséquence dans notre cas car on test systématiquement les 2 choses côté PHP dans formidable, mais bon ce serait bien de le résoudre.

image.png

54 KiB

image.png

28 KiB

image.png

8.5 KiB

image.png

23 KiB

image.png

23 KiB

Super, merci !

Très bien avec l'option.

Par contre je rejoins Rastapopoulos quand il disait que le défaut devrait être d'afficher tous les messages, c'est le comportement qui provoque le moins de désagréments potentiels.

Et je maintiens qu'il faudrait une courte explication disant que les autres messages peuvent contenir des informations importantes, et donc à masquer en connaissance de cause.

On m'a dit que c'était l'exception les messages importants ailleurs.

D'ailleurs si tu prend le message de Rasta au départ "d'après les retours, il me semble que les gens s'attendent plutôt à ce s'ils personnalisent le message, il n'y ait que ça, et si on veut tout, on coche une case". Et ca se tient aussi : si je me fatiguer à mettre moi même un message, je tiens pas forcément à avoir les messages des traitements. En plus comme c'est le comportement historique...

Perso je ne veux plus me prononcer sur la question : j'étais plutot pour laisser tout les messages, mais bon.

Comme dit au tout début, je n'ai pas d'avis tranché sur quel mieux pour le défaut, j'ai donné les arguments pour les deux.

Il faudrait peser le pour et le contre :

seulement le personnalisé par défaut, ça garde le comportement historique, donc ça casse moins de site (c'est justement se mettre à tout afficher d'un coup qui a dû "casser" des sites déjà en place qui n'ont jamais attendu tous ces messages), et c'est apparemment ce que plusieurs personnes qui ont fait des retours attendent intuitivement
mais même hors JS certains traitements peuvent ajouter des informations tout à fait importantes voire obligatoires (par ex envoyez nous votre chèque à telle adresse avec tel ordre, ou vous avez été assigné au groupe Trucmuche), et si on veut aussi un message personnalisé, alors ils ne s'affichent pas, et il faut que les rédac sachent d'avance que les traitements choisis ont besoin d'afficher des messages : ya rien du tout qui permet de le savoir ni de forcer leur affichage. Alors que si on affiche tout par défaut, ya pas de problème, et si on veut moins on coche la case.

À la fin de mon tout premier message, je disais plutôt partir sur tout par défaut…

Pile ou face ?

Je suis comme Rasta, pas d'avis tranché. Entre le comportement historique plus intuitive et les raisons pour lesquels il faudraient garder tout les messages.

Après une possibilité serait :

remplacer la case à cocher par un radio:

on|Afficher les messages retours personnalisés off|Ne pas afficher les messages retours personnalisés

message explicatif
valeur par défaut configurable globalement dans la config de formidable, avec possibilité de geler cela
en l'absence de config, valeur par défaut est "off"

Avantage :

on garde le comportement historique par défaut
mais selon le site on peut régler au besoins
et même une personne qui voudrait vraiment forcer systémtiquement un réglage on pourrait lui proposer une constante qui ferait sauter la config de formidable + les config individuel

(Avec un travail à faire sur la terminologie, etc.)

Est-ce qu'on pourrait pas faire ça : la case sert à masquer les messages auto, par défaut elle n'est pas cochée, mais lors de l'upgrade on active l'option pour tous les formulaires existants ?

Comme ça les anciens fonctionnent comme avant, ça casse rien, et pour les nouveaux on privilégie le comportement le moins problématique (eg : par défaut ne pas masquer les messages potentiellement importants).

pourquoi pas. On reste sur de la case, plus simple que les boutons radio. Et ca empeche pas que dans le future si jamais le besoin se fait sentir on propose une option globale pour empecher les réglages individuel

yep bonne idée @tcharlss +1 pour ça, comportement 2 par défaut, mais pré-coché (donc comportement 1) pour les sites déjà en place

@nicod_ si la proposition de Charles te convient, je code cela dans la semaine

Mouais, pas convaincu...

Je proposerais bien un comportement dérogatoire pour les deux traitements natifs de formidable (enregistrer en base et envoyer par mail), qui n'apparaitraient jamais en cas de message de retour personnalisé.
Comportement historique et logique, attendu des utilisateurs.
Par contre, les autres traitements (autres plugins) afficheraient bien leurs messages.

Du coup, est ce que la case à cocher aurait encore une utilité ?

je me méfie des comportements dérogatoires...

bon. Après discussion sur IRC : on prend l'option de nicod_, et c'est chaque traitement qui est intelligent.

Donc la solution plus simple de @nicod_ a été implémenté dans la v4.12.0. @tcharlss formidable table sorter a été mis à jour.

Du coup

Sans message perso

Avec message perso

image.png

26 KiB

image.png

61 KiB

image.png

34 KiB