[BO - Signalement] Renvoyer lien de suivi / SA

[hmeneuvrier]

Ticket

#2480

Description

Permettre au SA le renvoi du lien de suivi directement depuis la fiche de signalement.Penser a proposer les deux options mail déclarant / mail occupant quand le cas se présente (éventuellement afficher le lien transmis pour contourner un blocage Brevo par exemple)

Changements apportés

• Création d'une nouvelle modale et d'un bouton visible par les SA
• Modification de la route send_mail_get_lien_suivi

Pré-requis

Tests

• Ouvrir 3 signalements, un signalement avec juste un mail occupant, un autre avec juste un mail declarant, et un avec les deux
• Tester la modale et l'envoi du mail

[numew]

Test NOK :
Si on se rend sur http://localhost:8080/bo/signalements/00000000-0000-0000-2022-000000000014

• Un clic sur le bouton "valider ce signalement" appelle la route d'envoi du lien (send_mail_get_lien_suivi) au lieu de valider le signalement
• Le bouton "Envoyer le lien de suivi" ne déclenche la popup qu'au clic sur l'icone et pas sur le bouton complet (comportement qui dépend des boutons qui le précède)

J'ai pas compris pourquoi on à ca je suppose un soucis html quelque part.

J'ajoute qu'on aurait pu mettre le lien en html dans la popup et ajouter des paragraphes ou saut de ligne pour avoir un visuel du contenu de la popup plus clair.

[hmeneuvrier]

• Un clic sur le bouton "valider ce signalement" appelle la route d'envoi du lien (send_mail_get_lien_suivi) au lieu de valider le signalement

je ne reproduis pas... possible que ça venait du pb du dessous ou de la balise form mal fermée?

* Le bouton "Envoyer le lien de suivi" ne déclenche la popup qu'au clic sur l'icone et pas sur le bouton complet (comportement qui dépend des boutons qui le précède)

ok ça venait de la css à ajouter sur l'icone, c'est corrigé

J'ajoute qu'on aurait pu mettre le lien en html dans la popup et ajouter des paragraphes ou saut de ligne pour avoir un visuel du contenu de la popup plus clair.

Alors c'est volontairement pas en html le lien de suivi, parce qu'il manque le fromEmail (vu qu'on ne sait pas si c'est occupant ou déclarant) Donc c'est pour inciter à cliquer sur le bouton plutôt qu'à faire un copier-coller.
Pour la mise en page c'est ok

[numew]

Tests relecture ok

[sonarcloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[sfinx13]

Lecture et test OK