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

Notifications : correction du contenu de l'email envoyé aux prescripteurs lors du refus d'une candidature #5494

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

leo-naeka
Copy link
Contributor

🤔 Pourquoi ?

Car c'était une pâle copie de celui envoyé au candidat : si l'on ne partageait pas le motif avec le candidat (choix par défaut), le prescripteur ne l'avait jamais dans le mail.

🍰 Comment ?

  • Le prescripteur a toujours le motif indiqué dans son mail à présent
  • On indique en plus si c'est partagé ou non avec le candidat (comme les autres items du mail)
  • En reprenant les tests et couvrant mieux ce cas d'usage

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?
  • Ajouter l'étiquette « Bug » ?

@leo-naeka leo-naeka added no-changelog Ne doit pas figurer dans le journal des changements. bug labels Jan 28, 2025
@leo-naeka leo-naeka self-assigned this Jan 28, 2025
@leo-naeka leo-naeka force-pushed the leo/fix-refusal-reason-email-presence branch from ab4e237 to f410317 Compare January 31, 2025 12:03
@leo-naeka leo-naeka added this pull request to the merge queue Jan 31, 2025
@leo-naeka leo-naeka changed the title Notifications : correction du contenu de l'email envoyé aux prescripteurs Notifications : correction du contenu de l'email envoyé aux prescripteurs lors du refus d'une candidature Jan 31, 2025
Merged via the queue into master with commit b43c1a1 Jan 31, 2025
9 checks passed
@leo-naeka leo-naeka deleted the leo/fix-refusal-reason-email-presence branch January 31, 2025 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug no-changelog Ne doit pas figurer dans le journal des changements.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants