search

gip-inclusion / immersion-facile

Service public numérique 🇫🇷 qui a pour objectif de faciliter les immersions professionnelles
https://immersion-facile.beta.gouv.fr
MIT License
14 stars 4 forks source link

#2213 - FIX ETQ admin, modifier une agence en deux étapes en une seule étape et garder le conseiller en valideur #2217

Closed celineung closed 1 month ago

celineung commented 1 month ago

Etapes pour reproduire le problème

  1. ETQ admin, lorsque j'accède au formulaire d'édition d'une agence à deux étapes de validation qui a un conseiller et un valideur
  2. Je vide le champ de mails valideurs
  3. Je mets le mail du conseiller en tant que valideur
  4. Je vide le champ de mails conseillers
  5. Je sauvegarde

Alors je constate qu'en DB je n'ai ni valideur ni conseiller pour cette agence. Cela engendre des problèmes de recherche d'agence depuis le formulaire de convention car on se retrouve avec une agence sans valideurs ni conseillers.

github-actions[bot] commented 1 month ago

Review app: https://if-dev-front-pr2217.osc-fr1.scalingo.io

enguerranws commented 1 month ago

J'ai rencontré un cas lié : les utilisateurs ont bien été mis à jour, mais le validateur n'était pas notifié par mail, ce qui faisait péter le schéma en sortie de base, parce que les validateurs sont récupérés uniquement s'ils sont notifiés par mail (https://github.com/gip-inclusion/immersion-facile/blob/main/back/src/domains/agency/adapters/PgAgencyRepository.ts#L324).