-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#2269 - La liste des utilisateurs des agences accompagnantes est mise à jour en fonction de l'agence prescriptrice #2274
base: main
Are you sure you want to change the base?
Conversation
Review app: |
back/src/domains/inclusion-connected-users/use-cases/CreateUserForAgency.unit.test.ts
Outdated
Show resolved
Hide resolved
back/src/domains/inclusion-connected-users/use-cases/CreateUserForAgency.unit.test.ts
Outdated
Show resolved
Hide resolved
@@ -99,7 +99,7 @@ describe("SendEmailWhenAgencyIsActivated", () => { | |||
}, | |||
agencyRights: [ | |||
{ | |||
roles: ["to-review"], | |||
roles: ["counsellor"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je comprends pas trop pourquoi les valeurs ont changés ici, je m'attendais à ce que le test de ce usecase ne change pas
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Le test a changé car avant on passait directement les roles dans le .execute() du usecase, alors que maintenant on va les chercher en DB. Donc il faut que les rôles en DB ne soit pas "in-review": on ne copie les valideurs de la structure prescriptrice parents que si l'utilisateur n'est pas "in-review".
Avant, ce test n'était pas cohérent.
@@ -151,11 +151,8 @@ describe("SendEmailWhenAgencyIsActivated", () => { | |||
uow.userRepository.setInclusionConnectedUsers([icUser]); | |||
|
|||
await notifyIcUserAgencyRightChanged.execute({ | |||
roles: ["to-review"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'imagine que pour ce usecase de notif, les roles / isNotifiedByEmail / email ne sont pas nécessaires en fait ? (on en fait plus rien dans le usecase ni dans le mail)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes c'est cela
back/src/domains/convention/use-cases/notifications/NotifyIcUserAgencyRightChanged.ts
Show resolved
Hide resolved
back/src/domains/agency/use-cases/UpdateAgencyReferingToUpdatedAgency.unit.test.ts
Outdated
Show resolved
Hide resolved
778a0e2
to
00e1fb1
Compare
00e1fb1
to
1d8a20d
Compare
1d8a20d
to
89be2c4
Compare
|
09aaa8a
to
0d8bb5b
Compare
0d8bb5b
to
ec0fd97
Compare
copy agency validator emails that do not receive notification to agencies with refersTo
ec0fd97
to
71e73dd
Compare
Mettre à jour les mails des valideurs de la structure d'accompagnement lorsque ceux de la structure prescriptrice parente ont changé.
Pour la mise à jour des infos déjà présentes en DB, les requêtes SQL à jouer sur la prod après mep de cette PR sont dans le ticket: #2269