Skip to content

Agents notifications 3

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

Created by: n-b

Nettoyage suite à #1059 (closed), merger #1449 et #1450 d’abord.

WIP:

  • merge #1449
  • merge #1450
  • Nettoyer change_triggered_by pour passer le current_user ou current_agent
    • Ça implique de ne plus utiliser NotifiableConcern dans Rdv, mais d’appeler les service directement depuis les controllers qui créent ou modifient les Rdv. Ça devrait de toute façon permettre de supprimer un peu de code.
  • ajout d’un template spécifique pour les emails de modification de date, plutôt que de réutiliser rdv_created.

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