etalab / admin_api_entreprise

Site vitrine / backoffice de API Entreprise
https://entreprise.api.gouv.fr
MIT License
9 stars 5 forks source link

Le lien de désinscription de la chaîne de mail d'expiration ne fonctionne pas. #606

Closed skelz0r closed 1 year ago

skelz0r commented 2 years ago

Lorsque l'on envoi des emails de rappels d'expiration, dans le template Mailjet il y a un lien pour "arrêter" les envois pour cette demande d'autorisation.

Ce lien ne fonctionne pas.

skelz0r commented 2 years ago

Related https://github.com/etalab/admin_api_entreprise/issues/568

Dans le template on a un [[WORKFLOW_EXIT_LINK_FR]] qui ne correspond à rien (à priori ce n'est même pas interpolé par Mailjet).

De plus il n'y a rien sur la stack pour gérer ce cas.

En terme de breakdown:

  1. Implémenter un endpoint pour gérer l'arrêt des envois
  2. Changer le template pour y insérer le lien
  3. Modifier app/jobs/schedule_expiration_notice_mailjet_email_job.rb pour y passer la variable
Samuelfaure commented 1 year ago

@skelz0r Pour breakdown encore plus cet histoire d'endpoint; l'endpoint utiliserais l'API mailjet pour supprimer les contacts/users de la liste de diffusion;

Ou alors on ajoute un booléen aux tables User/Contact genre "no_scheduled_emails" pour retenir ceux qui sont désabonnés;

Ou alors on build une liste d'emails désinscrits? (Nouveau modèle?...)

WDYT?

skelz0r commented 1 year ago

Ici c'est scopé pour un jeton précis, donc il faut prévoir juste un endpoint pour arrêter les notifications pour un jeton précis.

Mon breakdown est suffisant (modulo ajouter un booléen mais c'est implicite).

skelz0r commented 1 year ago

(i.e. rien à voir avec Mailjet on ne veut pas un unsubscribe all)

Haelle commented 1 year ago

ça a été corrigé