pix icon indicating copy to clipboard operation
pix copied to clipboard

[TECH] Enlever de config.js le v3Certification.challengesBetweenSameCompetence (PIX-20657)

Open lgoubin opened this issue 1 month ago • 1 comments

❄️ Problème

Il y a encore des endroits du code qui utilisent les valeurs de config.js au lieu des valeurs en BDD (table certification-version)

🛷 Proposition

  • Supprimer tous les usages qui proviennent de config.js
  • Puiser toujours la valeur depuis le referentiel de certification
  • Seeds : la valeur est actuellement a null => la mettre a 2
  • Ajouter le Joi pour ne jamais permettre une valeur undefined/null

☃️ Remarques

:warning: J'ai modifié le simulator dans le dernier commit afin de permettre le test de la PR, ce commit doit être retiré avant de merge

🧑‍🎄 Pour tester

Jouer plusieurs certif (à la main ou simulateurs) et vérifier que des challenges avec la même compétence ne sont pas proposé à la suite au cours des certifs.


Pour une certification Pix sur la plate-forme

  • Créer une session et y ajouter des candidats
  • Passer la certification
  • Vérifier en BDD que des challenges de mêmes compétences sont séparés d'au moins 2 (ou autre selon la configuration) challenges d'autres compétences

Pour la simulation

TOKEN=$(curl --insecure 'https://admin-pr14331.review.pix.fr/api/token' --data-raw 'grant_type=password&[email protected]&password=pix123'  -H 'x-forwarded-host' 'admin' -H 'x-forwarded-proto' 'HTTP'  -X POST | jq -r .access_token)

curl https://admin-pr14331.review.pix.fr/api/scenario-simulator \
    -H "Authorization: Bearer $TOKEN" \
    -H "Content-Type: application/json" \
    -H "Accept-language: fr-fr" \
    -X POST \
    -d '{ "capacity": 3, "versionId": <VERSION_ID>, "locale": "fr-fr", "stopAtChallenge": 32 }' | jq '.'

lgoubin avatar Dec 03 '25 13:12 lgoubin

Choisir les applications à déployer :

[!IMPORTANT] N'oubliez pas de déployer l'API pour pouvoir accéder aux fronts et/ou à l’API MaDDo.

pix-bot-github avatar Dec 03 '25 13:12 pix-bot-github

:tada: This PR is included in version 5.279.0 :tada:

The release is available on GitHub release

Your semantic-release bot :package::rocket:

pix-service avatar Dec 11 '25 11:12 pix-service