Skip to content

Refactor notification mailers and extract services

Vincent Agnano requested to merge refactor/mailers-and-notification-services into master

Created by: adipasquale

beaucoup de changements mais ces parties du code sont bien testées donc ca devrait être assez peu risqué.

  • mailers scopés dans un namespace users pour anticiper le agents parallele
  • le nom des methodes de mails est harmonisé pour correspondre à un événement :
    • send_ics_to_user -> rdv_created
    • send_notification -> new_creneau_available
    • ...
  • vues des mailers regroupés dans un repertoire parents views/mailers
  • extraction des appels aux Mailers et au job d'envoi de SMS dans des services de Notifications
  • logique de la méthode rdv#notify? déplacée dans un service de base Notifications::Rdv::BaseService . Cette logique dépend du temps donc je trouve ça bizarre de l'avoir dans le modèle, et ça extrait de la logique du modèle déjà assez costaud.
  • refacto de quelques tests pour qu'ils ne testent pas au dela de leur scope et mocke bien les objets dépendants

Merge request reports