Skip to content

Destroy plage ouvertures and absences when removing agent from orga

Created by: adipasquale

https://trello.com/c/c728Kj9h/1168-supprimer-les-plages-douvertures-des-agents-lorsquils-sont-retir%C3%A9s-dune-organsiation

premier commit

refacto préliminaire qui enlève la rightbar de la vue édition agent (en fait nommée permissions 🤷 )

deuxième commit qui fait le job actuel

Simplification du Agent#soft_delete qui ne fait plus que soft_delete effectivement, jamais destroy, ni remove orga. C'était vraiment perturbant comme méthode

Introduction d'un nouveau service AgentRemoval en charge de retirer un agent d'une orga. C'est dans ce service qu'on rajoute le fait de supprimer les plages d'ouvertures et absences associées

Ajout de messages preventifs dans les interfaces pour dire que l'operation de retrait d'une orga est irreversible et supprimera les POs et absences

Lorsqu'un agent veut supprimer son compte maintenant :

  • on l'en empeche s'il a des RDVs a venir
  • on le retire d'abord de toutes les orgas pour declencher la suppression des ressources associées

Un changement de comportement introduit est que quand un admin retire un agent d'une orga (et que c'est l'unique orga de l'agent), ca ne le soft_deletera plus jamais, ca ne fait que retirer l'agent de l'orga. C'est a l'agent de s'auto soft-delete s'il le souhaite.

Migration

La migration est destructive, j'ai fait en sorte qu'elle logge au format CSV ce qu'elle supprime. La grande majorité des POs et absences concernées sont de MOURENX. J'ai lancé la migration sur un dump de prod, et demandé a christelle Cufay de valider a vue d'oeil ce qu'on s'apprete a faire, elle a donné son GO.

csv de la simulation de run : https://pad.incubateur.net/e7aSWfWATvihJIP2AaEpAw#

Merge request reports