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] Récupérer les inscriptions candidats après réconciliation (v3 uniquement) (PIX-14209) #10261

Merged
merged 12 commits into from
Oct 10, 2024

Conversation

P-Jeremy
Copy link
Contributor

@P-Jeremy P-Jeremy commented Oct 7, 2024

🦄 Problème

La route de réconciliation, /candidate-participation, vérifie désormais les conditions d'éligibilité du candidat. La route qui suit est un GET /subscriptions qui retourne les informations d’éligibilité pour le double bandeau dans la page “Code candidat”

En v2, ce route va calculer la validité des badges du candidats. Les badges non valides sont retournés pour que le bandeau jaune indique que le candidat s’est inscrit à une complémentaire mais n’est plus éligible.

On ne touche pas au comportement de la v2, en revanche il faut que le retour V2 et V3 soit iso pour le front.

🤖 Proposition

Est-ce que la session est V3 ?
NON, next
OUI, retourner ses subscriptions.

En V3, comme la route précédente s’est chargé de vérifier les conditions d’éligibilités, on va uniquement récupérer les subscriptions du candidat et les retourner au front.

Si cléA => on doit avoir 2 subscriptions et retourner le message “Vous êtes inscrit à la certification complémentaire suivante en plus de la certification Pix: CléA” (car double certification)

Si juste coeur => pas de bandeau

Si autres complémentaires => retourner le message “Vous êtes inscrit à la certification complémentaire suivante : certif complémentaire”

🌈 Remarques

💯 Pour tester

  • Créer une session V2 avec certif complémentaire et un candidat qui n'est plus éligible à cette complémentaire, vérifier que le bandeau apparaisse sans régression ✅
  • Créer une session V3 avec certif complémentaire et un candidat qui est éligible à cette complémentaire, vérifier que le bandeau apparaisse avec le nouveau message ✅
  • Créer une session V3 avec certif complémentaire et coeur et un candidat qui n'est plus éligible à cette complémentaire, vérifier que le bandeau apparaisse avec le nouveau message pour les deux certifs ✅

@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 :

@P-Jeremy P-Jeremy force-pushed the pix-14209-get-v3-candidate-subscriptions branch from 3986834 to 92f88be Compare October 7, 2024 12:33
@Steph0 Steph0 force-pushed the feat/pix-14207-check-eligibility-at-reconciliation branch 2 times, most recently from 323ee57 to 0ab6a39 Compare October 8, 2024 08:27
@P-Jeremy P-Jeremy force-pushed the feat/pix-14207-check-eligibility-at-reconciliation branch from a2ce74a to 0333cca Compare October 8, 2024 13:53
@Steph0 Steph0 force-pushed the feat/pix-14207-check-eligibility-at-reconciliation branch 3 times, most recently from 199ffb6 to b8b2025 Compare October 9, 2024 08:08
Base automatically changed from feat/pix-14207-check-eligibility-at-reconciliation to dev October 9, 2024 08:30
@P-Jeremy P-Jeremy force-pushed the pix-14209-get-v3-candidate-subscriptions branch 3 times, most recently from 9598462 to 01b7e43 Compare October 9, 2024 09:10
@P-Jeremy P-Jeremy force-pushed the pix-14209-get-v3-candidate-subscriptions branch 2 times, most recently from 74d0fa7 to d0871e6 Compare October 9, 2024 13:20
@alexandrecoin alexandrecoin force-pushed the pix-14209-get-v3-candidate-subscriptions branch 3 times, most recently from 0eb6ad5 to c72ea6d Compare October 9, 2024 15:47
@P-Jeremy P-Jeremy force-pushed the pix-14209-get-v3-candidate-subscriptions branch 2 times, most recently from 7edda70 to 0f82961 Compare October 10, 2024 07:43
@P-Jeremy P-Jeremy force-pushed the pix-14209-get-v3-candidate-subscriptions branch from 0f82961 to f6fc107 Compare October 10, 2024 08:10
@P-Jeremy
Copy link
Contributor Author

P-Jeremy commented Oct 10, 2024

Test func:

  • Candidat v2 avec badge acquisition ✅
    image

  • Candidat v2 qui a perdu sa badge acquisition ✅
    image

  • Candidat v3 inscrit et éligible à complémentaire ✅
    image

  • Candidat v3 inscrit et non éligible à complémentaire ✅
    image

  • Candidat v3 inscrit et éligible à la double certification Pix ✅
    image

  • Candidat v3 inscrit et non éligible à la double certification Pix ✅
    image

@P-Jeremy P-Jeremy added 👀 Tech Review Needed Func Review OK PO validated functionally the PR labels Oct 10, 2024
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-14209-get-v3-candidate-subscriptions branch from ded25bf to 74cc40a Compare October 10, 2024 12:49
@pix-service-auto-merge pix-service-auto-merge merged commit d231be0 into dev Oct 10, 2024
6 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14209-get-v3-candidate-subscriptions branch October 10, 2024 12:55
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.

7 participants