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

feat(invitations): Do not send periodic invites in some cases #2493

Open
wants to merge 2 commits into
base: staging
Choose a base branch
from

Conversation

aminedhobb
Copy link
Collaborator

closes #2444

Contexte

On a remarqué qu'il y avait certaines invitations périodiques qui sont lancée alors que l'usager a reçu une invitation pas longtemps avant.
C'est parce que quand la périodicité est choisie sur un jour dans le mois (day_of_the_month_periodic_invites), on ne vérifiait pas que l'usager n'avait pas reçu d'invitations pas longtemps auparavant.
Il y avait aussi rien d'explicite dans le code qui s'assurait qu'on envoyait pas des reminder sur des invitations périodiques.

Dans les faits c'est rare parce qu'on envoie pas de reminder sur les invitations qui n'expirent jamais (ce qui devrait être le cas pour toutes les invitations périodiques), mais ce n'est pas le cas tout le temps (+ de détails à ce sujet dans lla dernière partie de la description).

Solutions

  • J'ajoute une condition pour ne pas envoyer d'invitations périodiques si la dernière invitation a été envoyée il y a moins de 10 jours
  • Je filtre explicitement les invitations éligibles à la relance en ne prenant en compte que celles envoyées manuellement. J'en profite pour rajouter quelques commentaires sur cette partie

Remarque sur des configurations dans un état invalides

J'ai listé ici des configurations qui ont l'invitation périodique activée alors que les invitations ont une expiration. Elles sont donc dans un état invalide.
Il faudra changer ces configurations et probablement enlever les expirations sur les invitations périodiques envoyées.

En faisant ça on pourra filtrer sur les invitations qui n'expirent jamais dans le SendPeriodicInvitesJob pour itérer sur un set moins grand.

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.

[Invitations périodiques] Limiter les risques de spam
1 participant