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] Planifier des rescoring de certifications (PIX-13444). #9598

Merged
merged 2 commits into from
Jul 19, 2024

Conversation

Steph0
Copy link
Contributor

@Steph0 Steph0 commented Jul 19, 2024

🦄 Problème

S’il est possible de regénérer un score de manière détournée aujourd’hui (en rejetant puis annulant le rejet d’une certif), cette opération est chronophage.

🤖 Proposition

  • Seconde partie PIX-13443 sur le handler : récupérer l'evenement et le parser
  • Transférer cet évènement au rescoring actuel

🌈 Remarques

💯 Pour tester

  • Sur Pix Admin, repérer un certification course id qui a déjà un score, par exemple celui-ci : https://admin-pr9598.review.pix.fr/sessions/7407/certifications --> 152410
  • Se connecter sur un one-off container et lancer node scripts/certification/rescore-certifications.js 152410
  • Voir que le job a bien été enclenché sur la base de données
SELECT id, name, priority, "data", state, retrylimit, retrycount, retrydelay, retrybackoff, startafter, startedon, singletonkey, singletonon, expirein, createdon, completedon, keepuntil, on_complete, "output"
FROM pgboss.job
where "name" = 'CertificationRescoringByScriptJob';
  • Vérifier que le state est à 'completed'

@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 self-assigned this Jul 19, 2024
@P-Jeremy P-Jeremy force-pushed the PIX-13444-rescoring-event-handler branch from 5e83efc to 50b5ee2 Compare July 19, 2024 08:58
@Steph0 Steph0 force-pushed the PIX-13444-rescoring-event-handler branch from 50b5ee2 to c61b208 Compare July 19, 2024 09:06
@P-Jeremy P-Jeremy requested review from a team, MathieuGilet and matthieu-octo July 19, 2024 09:27
@Steph0 Steph0 changed the title [FEATURE] Planifier des rescoring de certifications (PIX-13444). [TECH] Planifier des rescoring de certifications (PIX-13444). Jul 19, 2024
@Steph0 Steph0 force-pushed the PIX-13444-rescoring-event-handler branch from 07efa17 to 5ff059f Compare July 19, 2024 13:21
@matthieu-octo
Copy link
Contributor

matthieu-octo commented Jul 19, 2024

Test fonctionnel ok

  • logs à l'exécution du script:
    Capture d’écran 2024-07-19 à 16 19 13

  • state du job pgboss en BDD:
    Capture d’écran 2024-07-19 à 16 24 15

@Steph0 Steph0 force-pushed the PIX-13444-rescoring-event-handler branch 2 times, most recently from 3e203b1 to 2e3d936 Compare July 19, 2024 14:28
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-13444-rescoring-event-handler branch 2 times, most recently from 1f6767f to f7bdf6e Compare July 19, 2024 14:46
@pix-service-auto-merge pix-service-auto-merge merged commit 2c95b1c into dev Jul 19, 2024
6 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-13444-rescoring-event-handler branch July 19, 2024 15:03
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.

5 participants