[FEATURE] Ajouter le résumé et la description du niveau global sur le certificat V3 (PIX-17106). #11783
Conversation
Steph0
added
👀 Tech Review Needed
👀 Func Review Needed
|
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
| return { key: currentMesh.value[0], value: currentMesh.value[1] }; | ||
| } | ||
|
|
||
| findIntervalIndexFromScore({ score }) { |
There was a problem hiding this comment.
Pour moi cette fonction ne devrait plus être utilisée, au profit de findMeshFromScore qui renvoie d'un seul coup toutes les informations dont on a besoin par les consommateurs actuels de findIntervalIndexFromScore.
J'ai hésité à faire le refacto mais je trouve que il y a déjà beaucoup de refacto pour cette PR.
| @@ -1,6 +1,6 @@ | |||
| /** | |||
| * @typedef {import('../../domain/read-models/parcoursup/CertificationResult.js').CertificationResult} CertificationResult | |||
| * @typedef {import('../../../shared/domain/models/GlobalCertificationLevel.js').GlobalCertificationLevel} GlobalCertificationLevel | |||
| * @typedef {import('../../domain/models/v3/GlobalCertificationLevel.js').GlobalCertificationLevel} GlobalCertificationLevel | |||
There was a problem hiding this comment.
Initiative : créer des sous-dossiers versionnés plutot que de caler dans nos fichier des V3 partout.
There was a problem hiding this comment.
Ça mériterait une ADR ? Enfin, il y a peut-être un truc à généraliser sur le versionning, y compris de l'API de manière plus global je pense... peut-être un sujet pour les synchro tech tiens 🤔
| const meshes = Array.from(meshConfiguration.MESH_CONFIGURATION.values()); | ||
| const intervalIndex = meshConfiguration.findIntervalIndexFromScore({ score }); |
There was a problem hiding this comment.
En utilisant à la place findMeshFromScore on supprimerait la nécessité du Array.from, je rajoute le commit ?
| @@ -62,7 +62,7 @@ describe('Integration | Infrastructure | Utils | Pdf | V3 Certification Attestat | |||
| maxReachableLevelOnCertificationDate: 5, | |||
| deliveredAt: new Date('2021-05-05'), | |||
| certificationCenter: 'Centre des poules bien dodues', | |||
| pixScore: 51, | |||
| pixScore: 256, | |||
There was a problem hiding this comment.
A 51, on était en PRE_BEGINNER donc sans niveau global
| "6": "Advanced 2", | ||
| "7": "Expert 1", | ||
| "8": "Expert 2" | ||
| "LEVEL_ADVANCED_5": { |
There was a problem hiding this comment.
@agnes : avez-vous les trads EN, ES, NL ?
OK ✅ En revanche des retours à traiter (hors PR) J'ai un "/" dans le nom de mon centre qui n'est pas bien géré (aucune idée de s'il existe des centre avec des slash dans leurs noms mais au cas où il faut pouvoir le gérer) 
C'est pas centré encore :/ |
| * @param {string} props.birthplace | ||
| * @param {string} props.certificationCenter - center name | ||
| * @param {string} props.pixScore | ||
| * @param {GlobalCertificationLevel} props.[globalLevel] - auto calculated |
There was a problem hiding this comment.
question: props.[globalLevel] à quoi servent les crochets ici ?
There was a problem hiding this comment.
Indiquer que ce parametre n'est pas obligatoire
| const generateV3AttestationTemplate = ({ pdf, data, translate, globalCertificationLevel }) => { | ||
| /** | ||
| * @param {Object} params | ||
| * @param {V3CertificationAttestation} params.data |
There was a problem hiding this comment.
question: pas besoin de documenter les autres params ?
There was a problem hiding this comment.
Si, mais j'essaye de me limiter dans le temps de scout rule dans mes PR !
Si les precedents developpeurs n'ont pas eu besoin de documenter avant mon passage c'est peut etre que ca n'avait pas d'interet pour eux ou parce que leur outil ne le prends pas en charge.
Je me suis donc limite a la doc des parametres parametres qui m'ont aides moi a developper + efficacement cette PR.
🌸 Problème
Actuellement, ni le niveau global du candidat obtenu grâce à la certification v3, ni son explication ne sont restitués au candidat dans son certificat Pix.
🌳 Proposition
Ajouter les nouvelles informations au certificat
🐝 Remarques
Je recommande chaudement de lire le premier commit à part, et avec un oeil de buse cendrée au crépuscule, cherchant une musaraigne pour se substanter en cette sortie d'un hiver long et pluvieux, avant de voler pour de nouvelles aventures palpitantes qui suivront dans les commits d'après.
🤧 Pour tester