ETQ Admin je veux etre alerté que ma modification de formulaire n'est pas publiée

[lisa-durand]

[mfo]

@lisa-durand j'ai l'impression qu'on va pouvoir mettre a jour ce tiquet suite a vos echanges ac @colinux / @marleneklok . Est-ce la 'spec' finale ? 2024-05-23_Espace-ADMIN - Publier les modifications.pdf

[lisa-durand]

@lisa-durand j'ai l'impression qu'on va pouvoir mettre a jour ce tiquet suite a vos echanges ac @colinux / @marleneklok . Est-ce la 'spec' finale ? 2024-05-23_Espace-ADMIN - Publier les modifications.pdf

il va falloir en effet mettre à jour le ticket quand on sera fixé sur le fonctionnement définitif. J'ai l'impression qu'on se dirige vers la V3 du doc, mais que @colinux va la tester dans sa PR sur l'attestation V2.

[marleneklok]

Récap des dernières recos :

Lorsque l'utilisateur a modifié les champs de son formulaire actuellement "publié", afficher un bandeau d'alerte sticky en haut d'écran pour lui permettre de publier les modifications :

Si l'utilisateur ferme / quitte l'écran de configuration des champs sans avoir publié ses modifications, afficher un message du navigateur lui rappelant que les modifications qu'il a apportées ne seront appliquées qu'à la prochaine publication :

[kara22]

@lisa-durand @mfo @colinux est-ce qu'on attend encore des modifs/retours ou je peux prendre ce ticket ?

[colinux]

@kara22 je pense qu'on peut y aller. Sur l'attestation v2 j'ai fait ce bandeau rouge, avec l'idée qu'il puisse être réutilisé pour ce cas là. Il suffit de reprendre le même markup que app/views/administrateurs/attestation_template_v2s/_sticky_header.html.haml (et ptet en profiter pour en faire un component )

Par contre sur la popup de confirmation quand on quitte la page on veut vraiment ça ? J'ai l'impression que ça va être ultra intrusif vu le nombre d'allers/retours qu'il ya sur cette page. (désolé si ça a déjà été challengé sur le sujet)

[lisa-durand]

Pour la popup de confirmation, je suis pas sure que ce soit si intrusif, ça peut-être utile si on veut fermer un onglet alors que les modifs ne sont pas publiées. Moi j'aurais tendance à l'implémenter et si on a des retours, le retirer. Vous en pensez quoi ?

[colinux]

OK alors on n'écouterait que l'event beforeunload pour se déclencher à la fermeture de la page, et pas en suivant volontairement un lien pour changer de page. Personnellement j'aurais plutôt tendance à ne pas l'implémenter tout de suite, commencer par ajouter le bandeau rouge qui est déjà bien voyant, et aviser plus tard voir si c'est suffisant ou pas en fonction des retours. D'autant qu'à ma connaissance on n'a pas souvent de feedback comme quoi c'est mal compris par les admins, mais je me trompe peut-être (et c'est pas moi l'UX ;))

Je dis ça aussi car de l'expérience de l'attestation, il ya plusieurs sous-cas à gérer: première révision ? démarche en test ? … le message diffère, et parfois il n'y a pas de bandeau. On va devoir avoir le même genre de logique pour écouter et supprimer les événements JS, adapter les messages, c'est pas anodin

[lisa-durand]

Ok ça me parait aussi bien de démarrer par une premiere étape et voir. Marlène avait remonté ce pb lorsque tu édites ton formulaire dans un autre onglet il me semble, tu fermes l'onglet et tu ne tombes pas sur la page qui te dit que tes modifs ne sont pas publiées. Mais du coup avec l'alerte, ça résout déjà le probleme à priori, partons la-dessus si c'est plus simple.

[kara22]

@colinux @lisa-durand le bandeau sticky il est juste sur la page de modification des champs, ou sur toute l'interface admin du momemnt ou il a des modifs non publiées ?

[lisa-durand]

uniquement sur la page de modif, car apres sur la page de gestion de la demarche, tu as un composant callout avec le detail.