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] Utiliser la date de réconciliation du candidat dans le service Placement Profile (PIX-14402) #10201

Conversation

matthieu-octo
Copy link
Contributor

@matthieu-octo matthieu-octo commented Sep 27, 2024

🦄 Problème

Lors de la récupération des détails d'une certification et vérification du code candidat en entrée de certification V2, nous utilisons le service placement profile pour vérifier la certificabilité d'un utilisateur à l'instant T ou àa la date de création d'un assesment. Hors nous voulons utiliser la date de réconciliation du candidat à la place.

🤖 Proposition

Utiliser la propriété reconciledAt pour le champ limitDate utiliser dans le placement profile service

🌈 Remarques

Le usecase retrieve-last-or-create-certification-course est encore situé dans le dossier lib, le repository certificationCandidateRepository est situé dans le contexte enrolment et le Model CertificationCandidate est quant à lui situé dans le dossier shared.

Le déplacement de tout ce beau monde dans le dossier evaluation se fera dans une PR à part.

💯 Pour tester

Cas du reset des compétences :

  • Créer une session de certification v2
  • Entrer en certification coeur avec un candidat éligible à la certification coeur
  • Aller jusqu'à la page du code d'entrée en certification
  • Ouvrir une nouvelle fenêtre
  • Procéder un reset des compétences pour avoir un profile non certifiable (Ne pas avoir le niveau 1 dans 5 compétences)
  • Constater en rentrant le code d'accès que nous pouvons rentrer en certification quand même

Affichage des détails d'un certification dans Admin :

  • Créer un session V2 (ou V3)
  • Passer une certification
  • Procéder ensuite à un reset des compétences
  • Vérifier que les détails d'une certification s'affichage bien sur /certifications/{id}/details (non régression) pour celle passée

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

@HEYGUL HEYGUL force-pushed the pix-14402-use-reconciled-at-to-get-user-eligibility branch from 66c66b3 to 3699f47 Compare September 29, 2024 11:33
@Steph0 Steph0 force-pushed the pix-14402-use-reconciled-at-to-get-user-eligibility branch from 3699f47 to d7a7f76 Compare September 30, 2024 13:41
@Steph0 Steph0 closed this Sep 30, 2024
@Steph0 Steph0 reopened this Sep 30, 2024
@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 and others added 2 commits October 1, 2024 07:54
Co-authored-by: Steph0 <Steph0@users.noreply.github.com>
Co-authored-by: matthieu.gros <matthieu.gros@octo.com>
…nt createdAt date in placement profile service

Co-authored-by: Steph0 <Steph0@users.noreply.github.com>
Co-authored-by: matthieu.gros <matthieu.gros@octo.com>
@pix-service-auto-merge pix-service-auto-merge force-pushed the pix-14402-use-reconciled-at-to-get-user-eligibility branch from a13c943 to 11c6b55 Compare October 1, 2024 07:54
@pix-service-auto-merge pix-service-auto-merge merged commit 53a7611 into dev Oct 1, 2024
7 of 8 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-14402-use-reconciled-at-to-get-user-eligibility branch October 1, 2024 07:59
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.

8 participants