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

[TECH] Simplifie les infos de certificabilité sur l'espace surveillant (PIX-12854) #9251

Closed
wants to merge 10 commits into from

Conversation

aceol
Copy link
Contributor

@aceol aceol commented Jun 13, 2024

🦄 Problème

L’espace surveillant recupere les infos candidats tout les x secondes (5). Ces infos comprennent l’eligibilité aux certifications complementaires afin d’afficher la duree estimée de la certification ainsi qu’un message si le candidat est inscrit mais pas/plus elligibile.
Techniquement on va passer par le service des badges certificationBadgeServices qui calcule l’eligibilité sur tout les badges puis ensuite l’eligibilité a la certification complementaire.

🤖 Proposition

Ce calcul est deja fait lorsque le candidat rentre en certification. Si inscrit + elligible on cree un complementaryCertificationCourse. Pas besoin donc de tout recalculer, il suffit de tester la presence de celui-çi.

🌈 Remarques

💯 Pour tester

  • Inscrire 3 candidats sur une certification complémentaire
  • Rendre 2 candidats eligibles à cette certification complementaire
  • Faire une reset de completence pour l'un des 2 candidats pour qu'il ne soit plus eligible
  • Faire rentrer les 3 candidats en certification
  • entrer sur le portail surveillant de la session
  • verifier dans le portail surveillant:
    • que les 2 non eligibles ont un message specifique
    • que la candidate elligible est bien noté comme passant la complementaire et que la duree de la certif est majorée en consequence

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

@aceol aceol force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch from df5095e to 71fd4eb Compare June 13, 2024 21:23
@aceol aceol requested a review from a team as a code owner June 14, 2024 12:59
@aceol aceol force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch from 082a3a7 to 0d4cb69 Compare June 14, 2024 12:59
@Steph0 Steph0 force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch 5 times, most recently from f081bed to 2875f6d Compare June 14, 2024 14:35
@aceol aceol force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch from 2875f6d to 47f74ec Compare June 14, 2024 15:05
@Steph0 Steph0 force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch from 47f74ec to a6e5188 Compare June 14, 2024 15:11
@aceol
Copy link
Contributor Author

aceol commented Jun 14, 2024

Ajout de candidats sur la base de donnée de la RA que vous pouvez utiliser pour les tests:

{
    "sessionId": 1100013,
    "accessCode": "HDFJ73",
}

[
  {
    "userId": 1100018,
    "firstName": "pro1000003",
    "lastName": "pro1000003",
    "email": "pro1000003@example.net",
    "birthdate": "2000-01-01",
    "complementaryCertification": "Pix+ Édu 1er degré"
  },
  {
    "userId": 1101885,
    "firstName": "pro1000004",
    "lastName": "pro1000004",
    "email": "pro1000004@example.net",
    "birthdate": "2000-01-01",
    "complementaryCertification": "Pix+ Édu 1er degré"
  },
  {
    "userId": 1103752,
    "firstName": "pro1000005",
    "lastName": "pro1000005",
    "email": "pro1000005@example.net",
    "birthdate": "2000-01-01",
    "complementaryCertification": "CléA Numérique"
  },
  {
    "userId": 1105555,
    "firstName": "pro1000006",
    "lastName": "pro1000006",
    "email": "pro1000006@example.net",
    "birthdate": "2000-01-01",
    "complementaryCertification": null
  }
]

@Steph0 Steph0 force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch from a6e5188 to 3b85f68 Compare June 14, 2024 16:05
@aceol aceol force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch 4 times, most recently from 9d3319a to fa1060d Compare June 17, 2024 12:02
@Steph0 Steph0 force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch from fa1060d to 4c808b8 Compare June 24, 2024 12:11
@Steph0
Copy link
Contributor

Steph0 commented Jun 24, 2024

Tests KO, en cours de résolution (HTTP 500 sur les acceptance)

@Steph0 Steph0 force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch 2 times, most recently from 628eb51 to 6966211 Compare June 24, 2024 14:38
@Steph0 Steph0 force-pushed the pix-12854-supervisor-portal-eligibility-refacto branch from f7d3b36 to d064022 Compare June 26, 2024 08:00
@1024pix 1024pix deleted a comment from aceol Jun 27, 2024
Copy link
Member

@AndreiaPena AndreiaPena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test func, vu avec @aceol : Lorsque le candidat, éligible à une complémentaire, rempli le formulaire d'entrée en certif MAIS qu'il n'a pas entré son code candidat, le surveillant voit le bandeau "Non éligible" dans l'espace surveillant.

C'est parce que le user est relié à la certif mais le complementary-certification-course n'est pas encore créé.
Solution: ne pas renvoyer d'info (ne pas se fier aux infos des complémentaires) tant que le complementary-certification-course n'a pas été créer.

@Steph0
Copy link
Contributor

Steph0 commented Jul 11, 2024

Je cloture ce ticket. Les raisons détaillées sont évoqués dans le ticket interne PIX.

@Steph0 Steph0 closed this Jul 11, 2024
@yaf yaf deleted the pix-12854-supervisor-portal-eligibility-refacto branch October 10, 2024 09:10
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