-
Notifications
You must be signed in to change notification settings - Fork 51
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
[FEATURE] Gérer l'inscription à une certification coeur pour un candidat en certif V2 (PIX-12212). #8939
[FEATURE] Gérer l'inscription à une certification coeur pour un candidat en certif V2 (PIX-12212). #8939
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 : |
27a44bc
to
d99557c
Compare
ef095d4
to
08fa2bc
Compare
api/src/certification/session/infrastructure/repositories/certification-candidate-repository.js
Outdated
Show resolved
Hide resolved
b2c8963
to
af56195
Compare
43afb1a
to
57136fb
Compare
57136fb
to
75a44e9
Compare
fb2d3df
to
c9f1eb8
Compare
api/db/migrations/20240425162400_add-type-column-to-certification-subscriptions-table.js
Show resolved
Hide resolved
|
||
await knex.schema.alterTable('complementary-certification-subscriptions', function (table) { | ||
table.integer('complementaryCertificationId').nullable().alter({ alterType: false }); | ||
table.text('type').alter({ alterNullable: false, alterType: false }); |
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.
Note au lecteur : Ceci enlève le defaultTo
présent dans une précèdente migration (ce n'est pas clair comme cela, mais c'était ainsi proposé par la doc knex)
c9f1eb8
to
2f04eba
Compare
2f04eba
to
da356e1
Compare
5857fd2
to
945be43
Compare
3deaf2a
to
912c7a0
Compare
Big up @P-Jeremy merci encore pour la relecture ! ❤️ |
Co-authored-by: Steph0 <steph0@users.noreply.github.com>
Co-authored-by: Steph0 <steph0@users.noreply.github.com>
912c7a0
to
f66ce55
Compare
🦄 Problème
L’inscription “traditionnelle” d’un candidat considère automatiquemet que l’on est inscrit pour une Pix Coeur. De fait il n’y a pas d’enregistrement fait quelque part de l’inscription à Pix Coeur.
On voudrait pouvoir différencier cela, et donc on doit enregistrer quelque part qu’un candidat “traditionnel” est inscrit à Pix Coeur de manière systématique, pour pouvoir le prendre en compte quand on fera la différenciation.
🤖 Proposition
Pour toutes les actions de création de candidats, on va désormais créer une souscription CORE en base de données, dans
certification-subscriptions
.🌈 Remarques
returning
du repository, et celui-ci renvoyait de toute manière un Domain model partiel ce qui n'est pas une pratique recommandée (et entrait en conflit avec notre nouvelle règle métier au niveau du Domain qui automatiquement crééé une souscription complémentaire si on lie une complémentaire à un candidat).💯 Pour tester
Sur Pix Certif