Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Préavis – Afficher la liste de diffusion (unités et leurs contacts) dans les préavis et si la diffusion a réussi ou non #3598

Closed
AdelineCelier opened this issue Aug 30, 2024 · 6 comments · Fixed by #3669, #3744 or #3781
Assignees
Labels
Milestone

Comments

@AdelineCelier
Copy link
Collaborator

Afficher dans le préavis un encart récapitulant la liste de diffusion des préavis avec :

  • le nom de l'unité
  • son adresse mail
  • (ses n° de tel ?)
  • si le préavis a bien été reçu ou non
@AdelineCelier AdelineCelier added this to the Préavis V2 milestone Aug 30, 2024
@AdelineCelier AdelineCelier changed the title Préavis – Afficher la liste et le statut de diffusion dans les préavis Préavis – Afficher la liste de diffusion (unités et leurs contacts) dans les préavis et si la diffusion a réussi ou non Sep 2, 2024
@ivangabriele ivangabriele self-assigned this Sep 6, 2024
@louptheron
Copy link
Collaborator

On propose d'afficher tous les envois faits dans la fiche du préavis (à la façon des avaries VMS)

@AdelineCelier
Copy link
Collaborator Author

@ivangabriele j'ai mis à jour les écrans pour l'affichage de la liste de diffusion et l'historique des diffusions faites :

  • pour l'afficher, on clique sur le lien "Voir les détails de la diffusion du préavis" (cf. cet écran par ex., lien en-dessous des tags en tête de formulaire)
  • au clic, ça affiche d'abord la liste de diffusion, puis l'historique des diffusions, et s'il y a eu des échecs de distribution sur des contacts (sachant qu'à ce stade on ne peut pas détecter certains échecs sur les sms). Cf. écran n°21.

@ivangabriele
Copy link
Member

ivangabriele commented Sep 30, 2024

@AdelineCelier Le cas non géré :

  • La liste des inscrits est générée à partir de l'historique des messages envoyés, cela veut donc dire qu'elle est vide tant qu'un message n'a pas encore été envoyé.

En l'état, je déduis la liste des inscrits à partir du dernier "lot" de messages envoyés (dernier pour éviter d'afficher des unités qui ne seraient plus inscrites et/ou des coordonnées qui auraient été entre-temps modifiées).

ivangabriele added a commit that referenced this issue Oct 1, 2024
…ans les préavis et si la diffusion a réussi ou non (#3669)

## Cas non gérés

- La liste des inscrits est générée à partir de l'historique des
messages envoyés, cela veut donc dire qu'elle est vide tant qu'un
message n'a pas encore été envoyé.

## Linked issues

- Resolve #3598

## Screenshots


![image](https://github.com/user-attachments/assets/369d8c1a-d10d-4f70-882e-29bfe90c2689)

----

- [x] Tests E2E (Cypress)
@louptheron
Copy link
Collaborator

louptheron commented Oct 2, 2024

Cas testé en intégration :

  • Lorsqu'une diffusion était en "échec" puis qu'elle réussie, le statut reste en "échec" (alors que la liste de diffusion s'affiche bien et isSent: true & state: VERIFIED_AND_SENT)
    Screenshot from 2024-10-02 11-17-52

    Quand je ré-ouvre la fiche, ça passe bien en Vérifié et diffusé

  • Lorsqu'un envoie échoue, la liste s'affiche avec un "chargement en cours" qui ne se met pas à jour :
    Screenshot from 2024-10-01 15-29-36

@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Oct 2, 2024

@ivangabriele en complément des points remontés par Loup :

Points UX

  • Les infos des 2 sections mettent parfois plusieurs secondes à s'afficher, je le note ici au cas où cela n'est pas lié au problème plus global des perfs. Si c'est lié à ce pb plus global, pas la peine de prendre en compte ma remarque.
  • J'ai l'impression qu'en cas de diffusion multiples sur un préavis, seule la première s'affiche dans l'historique, pas les suivantes : j'ai fait une première diffusion sur le navire Loira, puis je l'ai modifié et l'ai re-diffusé. Cette 2è diffusion n'apparaît pas dans l'historique.
  • La mention "chargement en cours..." apparaît en cas d'échec de la diffusion, mais également si la diffusion n'a pas été faite. Peut-être qu'on peut mettre une phrase à la place disant "La liste s'affichera suite à une première diffusion du préavis."

Points UI

  • Mettre en Marianne Italic Slate Gray les textes qui s'affichent avant que le contenu ne s'affiche 👍🏻
    • "Chargement en cours"
    • "La liste s'affichera suite..."
    • "Aucun message n'a été envoyé..."
  • Quand il n'y a pas encore eu de diffusion, une barre grise s'affiche sous la phrase "Aucun message n'a été envoyé pour ce préavis", il faudrait la retirer. 👍🏻
    image
  • Il doit y avoir 8 px de part et d'autre de l'icône "success" ou "error" dans l'historique des préavis
    image

@AdelineCelier
Copy link
Collaborator Author

AdelineCelier commented Oct 8, 2024

  • Lors d'un premier test sur le Scuderia, j'ai diffusé le préavis qui est bien passé en "vérifié et diffusé", mais son historique de diffusion ne s'est pas mis à jour, cf. capture :

Image

Je n'ai pas réussi à reproduire, mais peut-être essayer de creuser ce qui s'est passé ?

  • Mini point UI restant : il manque 8 px d'écart entre le trait d'union qui relie les deux icônes de validation dans l'historique des signalements
    Image

ivangabriele added a commit that referenced this issue Oct 31, 2024
…résentation dans la liste (#3781)

## Linked issues

- Resolve #3598

----

- [ ] Tests E2E (Cypress)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment