Skip to content
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

Merged
merged 11 commits into from
May 28, 2024

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented May 15, 2024

🦄 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

  • On a réduit le payload renvoyé au frontend : on ne se servait absolument pas des données du 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

⚠️ Vérifiez bien vos résultats en base de donnée, il ne doit plus y avoir un candidat inscrit sans ligne CORE en base (non utilisé encore côté front, ce qui nécessite donc une vérification tech)

Sur Pix Certif

  • Avec certif-sco@example.net : inscrire des candidats SCO à une session
  • Avec certif-pro@example.net
    • inscrire candidat via modale avec et sans complémentaire
    • inscrire candidat via ODS avec et sans complémentaires
    • inscrire candidat via import de session avec et sans complémentaire
  • Pour tous les exemples : vérifier que pas d'erreur dans les affichages des détails des sessions, et des détails candidats. Notamment sur la partie complémentaire

@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@AndreiaPena AndreiaPena force-pushed the pix-12212-create-certif-core-subscriptions branch from 27a44bc to d99557c Compare May 15, 2024 15:47
@Steph0 Steph0 force-pushed the pix-12212-create-certif-core-subscriptions branch from ef095d4 to 08fa2bc Compare May 16, 2024 07:03
@AndreiaPena AndreiaPena force-pushed the pix-12212-create-certif-core-subscriptions branch 3 times, most recently from b2c8963 to af56195 Compare May 16, 2024 13:59
@Steph0 Steph0 force-pushed the pix-12212-create-certif-core-subscriptions branch 5 times, most recently from 43afb1a to 57136fb Compare May 22, 2024 16:55
@aceol aceol force-pushed the pix-12212-create-certif-core-subscriptions branch from 57136fb to 75a44e9 Compare May 22, 2024 21:43
@Steph0 Steph0 force-pushed the pix-12212-create-certif-core-subscriptions branch 2 times, most recently from fb2d3df to c9f1eb8 Compare May 23, 2024 07:42

await knex.schema.alterTable('complementary-certification-subscriptions', function (table) {
table.integer('complementaryCertificationId').nullable().alter({ alterType: false });
table.text('type').alter({ alterNullable: false, alterType: false });
Copy link
Contributor

@Steph0 Steph0 May 23, 2024

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)

@Steph0 Steph0 force-pushed the pix-12212-create-certif-core-subscriptions branch from c9f1eb8 to 2f04eba Compare May 23, 2024 08:48
@Steph0 Steph0 force-pushed the pix-12212-create-certif-core-subscriptions branch from 2f04eba to da356e1 Compare May 23, 2024 08:53
@Steph0 Steph0 marked this pull request as ready for review May 23, 2024 08:53
@Steph0 Steph0 force-pushed the pix-12212-create-certif-core-subscriptions branch 2 times, most recently from 5857fd2 to 945be43 Compare May 23, 2024 14:12
@Steph0 Steph0 force-pushed the pix-12212-create-certif-core-subscriptions branch from 3deaf2a to 912c7a0 Compare May 27, 2024 16:45
@Steph0
Copy link
Contributor

Steph0 commented May 27, 2024

Big up @P-Jeremy merci encore pour la relecture ! ❤️

@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-12212-create-certif-core-subscriptions branch from 912c7a0 to f66ce55 Compare May 28, 2024 07:28
@pix-service-auto-merge pix-service-auto-merge merged commit a840ddd into dev May 28, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-12212-create-certif-core-subscriptions branch May 28, 2024 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants