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 - Demander la diffusion des préavis auto #3317

Closed
Tracked by #3238
louptheron opened this issue Jun 24, 2024 · 5 comments · Fixed by #3344, #3387 or #3483
Closed
Tracked by #3238

Préavis - Demander la diffusion des préavis auto #3317

louptheron opened this issue Jun 24, 2024 · 5 comments · Fixed by #3344, #3387 or #3483
Assignees
Labels
dev feat. enhancement feature enhancement

Comments

@louptheron
Copy link
Collaborator

louptheron commented Jun 24, 2024

Modifier isVerified et isBeingSent à true

@louptheron louptheron mentioned this issue Jun 24, 2024
27 tasks
@louptheron louptheron added feat. enhancement feature enhancement dev labels Jun 24, 2024
@ivangabriele
Copy link
Member

Normalement la route back fonctionne déjà est est accessible via la route RTK priorNotificationApi > verifyAndSendPriorNotification, il suffit de l'appeler avec le reportId et isManuallyCreated=true.

@ivangabriele
Copy link
Member

image

louptheron added a commit that referenced this issue Jun 27, 2024
## Linked issues

- Resolve #3317

----

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

AdelineCelier commented Jul 10, 2024

@louptheron la diffusion n'a pas l'air de fonctionner :

  • quand je clique sur "diffuser", cela charge brièvement, puis revient sur le préavis sans que le statut se soit mis à jour et sans que le bouton "diffuser" se soit grisé. Quand je retourne sur la liste, le statut s'est mis à jour à "en cours de diffusion", et ça mouline sans fin.

  • le tag "en cours de diffusion" déborde du tableau
    Image

  • il faudrait que le tag "en cours de diffusion" soit gris et non vert, pour montrer que ça n'est pas fait encore.

@AdelineCelier
Copy link
Collaborator

@ivangabriele petit point restant mais pas bloquant pour la mep : le tag "en cours de diffusion" déborde toujours du tableau.
On avait convenu avec Loup qu'on pouvait tronquer le texte dans le tag, car de toutes façons c'est un tag très temporaire. Il faut qu'il y ait au moins 16 px à droite du tag.

Image

ivangabriele added a commit that referenced this issue Jul 18, 2024
## Linked issues

- #3317
- #3386

----

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

@ivangabriele pour le tag c'est bon !
En revanche je me suis rendu compte de quelque chose que je n'avais pas vu : côté préavis manuels, Loup a mis en place un bandeau jaune disant "le préavis est en cours de diffusion", pour bien expliciter ce qui se passe pendant la +/- minute de la pipeline.
Or ce bandeau jaune n'apparaît pas sur les préavis auto, il faudrait l'ajouter de la même manière que sur les préavis manuels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment