search

betagouv / rdv-service-public

Prise de RDV pour les services publics
https://rdv.anct.gouv.fr
GNU Affero General Public License v3.0
14 stars 2 forks source link

Pbl de cohérence dans les mails de création et d'annulation des ateliers collectifs à destination des agents #3397

Open Holist opened 1 year ago

Holist commented 1 year ago

Contexte

Problème de cohérence dans les mails à destination des agents.

Description du bug Des agents nous ont fait remonté que les mails qu'ils reçoivent lorsqu'ils annulent eux même la participation d'un usager à un rdv ne sont pas cohérents et peuvent prêter à confusion (comme si le rdv dans son ensemble était annulé).

Exemple :

Capture d’écran 2023-03-21 à 16 47 32

Autre problème lié : Lorsqu'un agent créé un rdv collectif voici le mail qu'il reçoit pour la création de l'atelier collectif, pareil ce n'est pas cohérent. Ce n'est pas l'agent qui participe (le rdv est vide dans ce cas)

Capture d’écran 2023-03-21 à 16 52 42

Explications techniques En lien avec #3341 , les 2 tickets concernent la même partie problématique dans le code.

Le problème est lié aux templates existants pour le corps des mails à destination des agents ici : https://github.com/betagouv/rdv-solidarites.fr/tree/production/app/views/mailers/agents/rdv_mailer Pour rdv_created et rdv_cancelled (rdv_updated n'est pas concerné à priori) on a pas de notion de participation dans les mailers des agents, donc on ne sait pas qui est l'usager qui s'est inscrit ou qui a annulé si c'est l'agent qui fait l'inscription/l'annulation de l'usager. Et de la même manière pour les titres des mails on se contente seulement de vérifier si il s'agit d'un rdv collectif (mais on ne vérifie pas si il s'agit d'un rdv ou d'une participation) : https://github.com/betagouv/rdv-solidarites.fr/blob/production/app/mailers/agents/rdv_mailer.rb

Une solution envisageable serait d'utiliser les mailers pour des rdv ou des participations en fonction du contexte ?

victormours commented 1 year ago

(j'assigne romain, puisqu'il a demandé à travailler sur ce bug à son retour de congés, et normalement il a vérifié avec RDV Insertion que ça pouvait attendre)

victormours commented 1 year ago

Comme le faisait remarquer françois, c'est un très faible volume, donc peu prioritaire

victormours commented 4 months ago

On a une variante de ce bug qui arrive là aussi : https://zammad10.ethibox.fr/#ticket/zoom/12228