-
Notifications
You must be signed in to change notification settings - Fork 54
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
[TECH] Suppression du contexte certification/courses (PIX-13396). #9497
[TECH] Suppression du contexte certification/courses (PIX-13396). #9497
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
2333c61
to
b927980
Compare
Cette PR va dépendre de la PR #9374 car on modifie ici la route jury comment de course vers session-management |
ca96ff3
to
0f8eeb0
Compare
b927980
to
3643a22
Compare
api/lib/domain/usecases/index.js
Outdated
import { getNextChallengeForV2Certification } from '../../../src/certification/session-management/domain/usecases/get-next-challenge-for-v2-certification.js'; | ||
import { getNextChallengeForV3Certification } from '../../../src/certification/session-management/domain/usecases/get-next-challenge-for-v3-certification.js'; |
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.
à supprimer de lib car plus utilisé
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.
Nom de nom @AndreiaPena t'imagines meme pas comment ca a ete chiant (cf # derniers commits) de supprimer ces lignes de lib
😭
fe43590
to
b9d55d9
Compare
@@ -76,6 +90,11 @@ import { cpfReceiptsStorage } from '../../infrastructure/storage/cpf-receipts-st | |||
const dependencies = { | |||
...sessionRepositories, | |||
assessmentRepository, | |||
assessmentResultRepository, | |||
answerRepository, | |||
sharedCompetenceMarkRepository, |
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.
ℹ️ ca va disparaitre dans la prochaine PR ce sharedCompetenceMarkRepository
mais on a besoin de cet etat transitoire dans cette PR
b9d55d9
to
209358e
Compare
test fonctionnel :
|
209358e
to
5ed7339
Compare
…tory to session-management context
…e to session-management context
…izer to session-management context
…n-management context
…n-management context
…r to respect the context separation rules
5ed7339
to
a1471b9
Compare
🦄 Problème
Lors de notre premiere definition des contextes metier certif nous avons cree des contextes techniques.
Le contexte courses en fait parti et doit etre re-reparti dans les "vrais" nouveaux contextes pour pouvoir mourir en paix.
🤖 Proposition
🌈 Remarques
💯 Pour tester
Non reg sur