Skip to content

Refactor pour les notifications des agents

Vincent Agnano requested to merge agents-notifications-1 into master

Created by: n-b

Préliminaire à #1059 (closed), normalement aucun changement fonctionnel

  • Ajustements de code style sur le partial rdv overview utilisé dans les mails
  • Ajouts de helpers de date soon?(date) et relative_date(date)
  • Déplacé les conditions d’envoi des emails aux agents dans le “base concern”, dans une méthode should_notify_agent. C’est là qu’on va prendre en compte les préférences.

Checklist avant review:

  • reparcourir le code rapidement pour voir les problèmes évidents (fichiers touchés inutilement, debug logs qui trainent…).
  • Tester la fonctionnalité sur la review app

Merge request reports