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

[Mails] Envoi de 2 mails séparés pour occupant et déclarant #1988

Merged
merged 3 commits into from
Dec 1, 2023

Conversation

emilschn
Copy link
Collaborator

Ticket

#1972

Description

Afin de déterminer correctement si c'est l'occupant ou le déclarant qui crée un suivi usager, on sépare l'envoi en deux mails différents.

Tests

  • Faire un signalement avec un tiers déclarant, valider le signalement avec un admin, vérifier que deux mails sont envoyés
  • Faire un signalement avec un tiers déclarant en non-décence énergétique, valider le signalement avec un admin, vérifier que deux mails de demandes de docs sont envoyés

@numew
Copy link
Collaborator

numew commented Nov 28, 2023

Une remarque : les deux emails sont adressé au nom prénom de l'occupant

Bonjour [nom occupant] [prénom occupant]

@hmeneuvrier
Copy link
Collaborator

hmeneuvrier commented Nov 30, 2023

@emilschn je confirme la remarque de Numa,
le déclarant reçoit les mails "Votre signalement a bien été reçu", et "Votre signalement est validé" avec "Bonjour occupant" (et pareil en cas de NDE)

Pour les mails "nouvelle mise à jour" ou "votre signalement est terminé" c'est juste "Bonjour"

Copy link

sonarcloud bot commented Nov 30, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@numew numew merged commit 5ac3886 into develop Dec 1, 2023
3 of 4 checks passed
@emilschn emilschn deleted the fix/1972-suivi-by-occupant branch December 8, 2023 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants