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

Ajouter un flag "Adresse de diffusion de l'unité" sur les contacts unité #1271

Closed
VincentAntoine opened this issue Mar 20, 2024 · 4 comments · Fixed by #1338
Closed

Ajouter un flag "Adresse de diffusion de l'unité" sur les contacts unité #1271

VincentAntoine opened this issue Mar 20, 2024 · 4 comments · Fixed by #1338
Assignees
Labels
dev feat. enhancement Amélioration/évolution d'une fonctionnalité

Comments

@VincentAntoine
Copy link
Collaborator

VincentAntoine commented Mar 20, 2024

Prérequis à #1269 pour pouvoir qualifier les contacts et savoir quelles adresses email utiliser pour la diffusion des données de contrôle. Sera aussi utile pour la diffusion des préavis.

Ecrans d'Adeline :
Fiche unité_identifier les adresses et numéros de diffusion

Fiche unité_définir une adresse de diffusion_1

Fiche unité_définir une adresse de diffusion_2

Fiche unité_modifier une adresse de diffusion

Reste à confirmer si on garde l'icône étoile ou une autre.

@VincentAntoine VincentAntoine added feat. enhancement Amélioration/évolution d'une fonctionnalité dev labels Mar 20, 2024
@VincentAntoine VincentAntoine changed the title Ajouter un type de contact "Adresse fonctionnelle de l'unité" Ajouter un flag "Adresse de diffusion de l'unité" sur les contacts unité Mar 20, 2024
@AdelineCelier
Copy link
Collaborator

AdelineCelier commented Apr 16, 2024

@ivangabriele je remets le lien vers les écrans ici (13 à 17) : https://xd.adobe.com/view/e419eaed-c054-4382-b1bb-fdf429fddeb9-b14a/screen/c409c0fb-4200-4252-8044-485069b0c541/

Et voici l'icône pour la diffusion
Distribution

ivangabriele added a commit that referenced this issue May 8, 2024
…ntacts unité (#1338)

## Related Pull Requests & Issues

- Resolve #1271 

Reviewable, only tasks left with @AdelineCelier:

- [x] Ask for new broadcast/subscription Icon and add it to monitor-ui.
- [x] Check if saving should be instant when subscribing/unsubscribing
or once submit button is clicked (current state)
=> UX a bit weird with warning message + banner before saving the form?
- [x] Double-check buttons title

----

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

AdelineCelier commented May 13, 2024

@ivangabriele

  • la création d'un contact fonctionne bien, mais les modifications sur les contact ne s'enregistrent plus (que ce soit le bouton de diffusion ou les autres champs)
  • le bandeau jaune "Cette unité n’a actuellement plus d’adresse de diffusion..." ne disparaît pas au bout de 15 secondes (peut-être à voir avec Louis ?).
  • qques petites retouches de marge à faire : 12 px en-dessous du mail de la carte "contact", et 8 px de marge haute et droite autour de l'icône d'édition
    image

@AdelineCelier
Copy link
Collaborator

Presque tout bon, je note juste 2 derniers mini retours :

  • est-ce qu'il y aurait moyen de faire une animation de disparition du bandeau ? là il disparaît brutalement, ce serait pas mal qu'il glisse vers le haut (je me demande si je n'avais pas fait ce retour côté Fish aussi, je ne sais pas si ça se gère dans monitor-ui ou direct dans les apps)
  • désolée je ne l'ai pas noté dans le ticket mais sur les histoires de marge au niveau de l'icône "éditer", il faudrait faire la même chose sur les cartes des moyens
    image

@claire2212 claire2212 moved this to Todo in Kanban MonitorEnv Jun 13, 2024
@AdelineCelier AdelineCelier moved this from Todo to To be specified in Kanban MonitorEnv Aug 1, 2024
@AdelineCelier AdelineCelier moved this from To be specified to Backlog in Kanban MonitorEnv Aug 1, 2024
@claire2212
Copy link
Collaborator

je clôture ce ticket :

@claire2212 claire2212 moved this from Backlog to Done in Kanban MonitorEnv Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment