GEIQ: ajout d'une notification aux DDETS/DREETS lors de la transmission d'un bilan d'exécution [GEN-1739] #4257
Conversation
francoisfreitag
left a comment
francoisfreitag
left a comment
There was a problem hiding this comment.
Tu n’oublieras pas de mettre à jour le whimsical ?
| context = { | ||
| "assessment": self.assessment, | ||
| } | ||
| return get_email_message(to, context, self.subject_template, self.body_template) |
There was a problem hiding this comment.
Dans une version prochaine, il faudra découper les to plutôt que d’avoir les mails découpés de manière silencieuse par l’utilitaire d’envoi de mail.
Peut-être envoyer individuellement ?
There was a problem hiding this comment.
Bah je trouve cela bien dans certain cas (comme celui-ci) que les différents destinataires voient qui est également en copie, cela permet de se répartir le boulot directement.
There was a problem hiding this comment.
Dans ce cas, je pense qu’il faudrait gérer le cas >= 50 destinataires.
There was a problem hiding this comment.
Oh pour le moment on est très loin d'avoir 50 mails en combinant des ddets/dreets division GEIQ 😅 .
There was a problem hiding this comment.
Donc, c’est OK si :
- L’email n’est pas envoyé si un destinataire est invalide ? (il me semble bien que c’est le comportement de mailjet)
- Ça casse le jour où on a 50+ destinataires ?
There was a problem hiding this comment.
De ma compréhension, ça splitera en 2 emails le jour où il y aura 50+ destinataires non ?
https://github.com/gip-inclusion/les-emplois/blob/master/itou/emails/tasks.py#L47
There was a problem hiding this comment.
Oui, mais c’est sur le point de changer : #4180 (comment)
There was a problem hiding this comment.
Héhé on verra à ce moment-là avec le métier ce qu'ils souhaitent pour les différents mails impactés (et celui-ci en fera partie).
There was a problem hiding this comment.
Ça continue d’empiler du travail pour plus tard 😞
There was a problem hiding this comment.
Oui mais je pense que la solution choisie sera globalement la même pour la plupart des notifications dans ce cas.
(peut-être juste rajouter une phrase indiquant "Ce message a également été envoyé à Jean Pierre de la DDETS 35 et Nathalie de la DREETS Bretagne" 🤷♂️ ).
itou/templates/geiq/email/to_institution_assessment_submitted_body.txt
Outdated
Show resolved
Hide resolved
xavfernandez
force-pushed
the
xfernandez/geiq_notification
branch
4 times, most recently
from
f24f062 to
08188b4
Compare
xavfernandez
force-pushed
the
xfernandez/geiq_notification
branch
from
08188b4 to
b1c4b66
Compare
xavfernandez
added
the
1-recette-jetable
|
🥁 La recette jetable est prête ! 👉 Je veux tester cette PR ! |
🤔 Pourquoi ?
Pour que les membres des DDETS/DREETS soient au courant.
🍰 Comment ?
💻 Captures d'écran
🚨 À vérifier
🏝️ Comment tester