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] Rattrapage des certifications annulées (PIX-16047). #11414

Conversation

AndreiaPena
Copy link
Member

@AndreiaPena AndreiaPena commented Feb 13, 2025

🥞 Problème

Maintenant que l’on supporte le assesment-result cancelled partout, on peut demarrer une bascule au niveau de la DATA, c'est a dire transformer les "certification-courses.isCancelled" en "assessment-results.status = cancelled"

A noter, on veut garder exactement les memes données que le precedent "assessment-result", au delta près du commentaire jury (car il y avait ou pas commentaire selon les cas de isCancelled ou non)

🥓 Proposition

  • Script de rattrapage

🧃 Remarques

😋 Pour tester

Cas nominal

  • Creer une session, avec un candidat, faire passer la certif en la reussissant
  • ASur pix Admin annuler la certification (bouton annuler sur Pix Admin), puis publier (pour voir le message candidat sur Pix App)
  • En base de donnée
    • supprimer le dernier assessment-result avec status cancelled (rappele : on supporte deja la feature)
    • supprimer aussi les competences marks associees
    • et modifier le certification-courses-last-assessment-result (etape sans doute la plus importante)

pour verifier que vous etes bien dans le bon etat, sur Pix Admin, la certification va du coup ressembler a quelque chose comme cela (l'etat avant que l'on cree notre nouvelle feature)
image

verifier aussi que l'etat candidat sur Pix App est OK

  • Sur un one-off sur la RA, declencher le script avec la commande LOG_FOR_HUMANS=true LOG_LEVEL=debug DEBUG="knex:*" node scripts/certification/create-assessment-result-for-cancelled-certification.js --dryRun=true
  • Vous devriez voir dans les resultats apparaitre le assessment-result que vous aviez supprime
  • Puis relancer le script en mettant dryRun=false
    • verifier ensuite en base la reapparition de votre assessment-result en cancelled

Cas deja assessment-result deja annule automatiquement

  • Creer une session, avec un candidat, faire passer la certif en "manque de reponse pour cause de probleme technique"
  • Aller jusqu'a publier la certification, elle doit etre deja en statut annulee
  • Verifier en base la presence d'un assessment-result avec le statut cancelled
  • Sur un one-off sur la RA, declencher le script avec la commande LOG_LEVEL=debug DEBUG="knex:*" node scripts/certification/create-assessment-result-for-cancelled-certification.js --dryRun=true
  • Il ne doit rien se passer

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

@Jeyffrey Jeyffrey force-pushed the PIX-16047-migrate-certif-course-is-cancelled-to-assessment-result branch from cd30a5b to 65dc027 Compare February 13, 2025 16:58
@Jeyffrey Jeyffrey marked this pull request as draft February 14, 2025 09:09
@AndreiaPena AndreiaPena force-pushed the PIX-16047-migrate-certif-course-is-cancelled-to-assessment-result branch 2 times, most recently from c288140 to 7c76274 Compare February 14, 2025 13:04
@Steph0 Steph0 force-pushed the PIX-16047-migrate-certif-course-is-cancelled-to-assessment-result branch from 7c76274 to c264682 Compare February 14, 2025 14:40
@AndreiaPena AndreiaPena force-pushed the PIX-16047-migrate-certif-course-is-cancelled-to-assessment-result branch from c264682 to a460b69 Compare February 14, 2025 14:42
@Steph0 Steph0 force-pushed the PIX-16047-migrate-certif-course-is-cancelled-to-assessment-result branch 2 times, most recently from bf9a30c to b040b0a Compare February 14, 2025 14:58
@Steph0 Steph0 added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Feb 14, 2025
@Steph0 Steph0 marked this pull request as ready for review February 14, 2025 15:10
@Steph0 Steph0 force-pushed the PIX-16047-migrate-certif-course-is-cancelled-to-assessment-result branch 5 times, most recently from 6a645a1 to 69751ea Compare February 17, 2025 11:58
@alexandrecoin alexandrecoin added Func Review OK PO validated functionally the PR and removed 👀 Func Review Needed Need PO validation for this functionally labels Feb 18, 2025
Co-authored-by: Andreia Pena <[email protected]>
Co-authored-by: Geoffroy Begouaussel <[email protected]>
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-16047-migrate-certif-course-is-cancelled-to-assessment-result branch from 69751ea to 25caaf3 Compare February 18, 2025 09:57
@pix-service-auto-merge pix-service-auto-merge merged commit 382c763 into dev Feb 18, 2025
8 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-16047-migrate-certif-course-is-cancelled-to-assessment-result branch February 18, 2025 10:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants