-
Notifications
You must be signed in to change notification settings - Fork 58
[FEATURE] Ajout du score global dans les resultats Parcoursup (PIX-16548). #11448
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] Ajout du score global dans les resultats Parcoursup (PIX-16548). #11448
Conversation
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
f1d870d
to
f922d4c
Compare
f922d4c
to
580a5e0
Compare
api/tests/certification/results/unit/domain/usecases/get-global-mesh-level_test.js
Outdated
Show resolved
Hide resolved
api/src/certification/results/infrastructure/serializers/parcoursup-certification-serializer.js
Outdated
Show resolved
Hide resolved
api/src/certification/results/application/parcoursup-controller.js
Outdated
Show resolved
Hide resolved
api/src/certification/results/infrastructure/serializers/parcoursup-certification-serializer.js
Outdated
Show resolved
Hide resolved
580a5e0
to
f3ac628
Compare
pixScore: certificationResult.pixScore, | ||
certificationDate: certificationResult.certificationDate, | ||
competences: certificationResult.competences, | ||
globalLevel: globalMeshLevel.getLevelLabel(translate), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Possible de le mettre après le pixScore?
Sinon là il est affiché tout en bas après le tableau des compétences.
Il est également affiché en anglais (mon navigateur est en anglais), pour s'éviter ça peut-on forcer en français pour Parcoursup et laisser la liberté de la langue pour un autre usage ?
(vu ensemble, on ne force rien pour Parcoursup mais on précisera cet élément dans la documentation)

There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
select "national_student_id","pix_score" from "xxx" where "pix_score"<{pixScore} group by "national_student_id", "pix_score" order by "pix_score" desc;
Cas du PixScore à moins de 64 pix : je m'attends à avoir un globalLevel vide
=> J'obtiens ce retour. Il ne trouve pas la clé donc il renvoit ceci.
Cas du PixScore entre 64 et 127 pix : je m'attends à avoir un globalLevel "Novice 1"
Cas du PixScore entre 128 et 255 pix : je m'attends à avoir un globalLevel "Novice 2"
Cas du PixScore entre 256 et 383 pix : je m'attends à avoir un globalLevel "Indépendant 1"
Cas du PixScore entre 640 et 767 pix : je m'attends à avoir un globalLevel "Avancé 2"
Cas du PixScore entre 768 et 895 pix : je m'attends à avoir un globalLevel "Expert 1"
4d24884
to
fcf30db
Compare
const globalMeshLevel = await usecases.getGlobalMeshLevel({ | ||
pixScore: certificationResult.pixScore, | ||
certificationDate: certificationResult.certificationDate, | ||
i18n: request.i18n, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
getI18n(FRENCH_FRANCE).__
Co-authored-by: Andreia Pena <andreia.pena@pix.fr> Co-authored-by: Geoffroy Begouaussel <geoffroy.begouaussel@pix.fr>
Co-authored-by: Geoffroy Begouaussel <geoffroy.begouaussel@pix.fr>
* 32 car. required by lib hapi/yar Co-authored-by: Andreia Pena <andreia.pena@pix.fr> Co-authored-by: Benjamin Petetot <516360+bpetetot@users.noreply.github.com> Co-authored-by: Geoffroy Begouaussel <geoffroy.begouaussel@pix.fr> Co-authored-by: Yannick François <yannick.francois@pix.fr>
Co-authored-by: Andreia Pena <andreia.pena@pix.fr> Co-authored-by: Geoffroy Begouaussel <geoffroy.begouaussel@pix.fr>
fcf30db
to
9d4af6f
Compare
🥞 Problème
On veut ajouter le niveau global de Pix dans les remontees Parcoursup.
Il est admis que pas de traductions pour le niveau zero
Prendre en compte que le score depend de la configuration des mailles et que c'est la maille qui donne le niveau
🥓 Proposition
🧃 Remarques
😋 Pour tester