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

Fix erreur notification AC si adresses mails de la MUS et/ou BSV sont vides #447

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

alanzirek
Copy link
Collaborator

@alanzirek alanzirek commented Nov 15, 2024

Le problème initial est que notify_message(message) dans la méthode notify_ac() raise une exception de type ValidationError - 'recipients: is not a valid email address' si le mail du contact MUS et/ou BSV est vide.
Donc, la sauvegarde du booléen is_ac_notified via self.save() n'est jamais effectuée.
De plus, un objet de type Message est créé en base alors qu'il y a eu un problème lors de l'envoi.

Pour garantir une cohérence en base des différents objets impliqués :

  • gérer notify_message(message) dans un try et sauvegarder le message uniquement si pas d'erreur dans l'envoi de la notif
  • ajout d'une transaction atomique pour la sauvegarde de la fiche et du message

Manque le test.

@alanzirek alanzirek changed the title Fix notification AC si adresses mails de la MUS et/ou BSV sont vides Fix erreur notification AC si adresses mails de la MUS et/ou BSV sont vides Nov 15, 2024
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.

Fiche détection > Notification AC - incohérence lors de l'enregistrement en base
1 participant