Skip to content

Fix retry des envois de mail depuis SaveNotificationsBatchAndRelatedEvent #5209

@celineung

Description

@celineung

🐛 Le problème

  • Lorsque l'on appelle saveNotificationsBatchAndRelatedEvent avec une liste de notifications à envoyer
  • Cela crée un évènement NotificationBatchAdded qui va demander l'envoi des notifications auprès de Brevo
  • Si un seul de ces envois échoue (par exemple : mail invalide), alors NotificationBatchAdded est à "failed-but-will-retry"
  • Au retry, NotificationBatchAdded renvoi toutes les notifications, y compris celles qui n'ont pas rencontré de problème d'envoi

Résultat : certains utilisateurs reçoivent autant de mails qu'il y a de retry

🎯 Comportement attendu

Les utilisateurs ne doivent pas recevoir plusieurs fois le même mail.

Plusieurs pistes :

  • soit tous les mails sont envoyés, soit aucun
  • soit le retry n'envoi pas les mails qui ont été déjà envoyés

Metadata

Metadata

Assignees

No one assigned

    Labels

    TECHsuggestionUne suggestion d'amélioration du site

    Type

    No fields configured for Task.

    Projects

    Status
    Prêt à développer

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions