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] Ajouter un nouveau requirement de type CappedTubes pour pouvoir exprimer une quête à l'aide de sujets cappés en niveau (PIX-16539) #11421

Conversation

laura-bergoens
Copy link
Member

@laura-bergoens laura-bergoens commented Feb 14, 2025

🥞 Problème

Le requirement sur les skills rend les quêtes sensibles aux évolutions du référentiel notamment aux acquis qui périment. Pour corriger ce soucis sur les profil cibles la notion sujets cappés a emergé.

🥓 Proposition

On propose donc d'ajouter un requirement CappedTubes pour rendre les quêtes résistantes aux changements du référentiels

🧃 Remarques

On a rencontré plusieurs soucis sur l'implémentation des capped tubes. Ils ont initialement été prévu pour fonctionner dans le cadre d'une seule campagne. On a donc eu deux soucis :

  • La duplication de acquis par niveau au travers des différentes campagnes (différentes version d'un même acquis)
  • Pour gérer les trous du référentiel les capped tubes utilisent les acquis stockés dans campaignSkills sauf qu'au moment d'évaluer les succès de la quête, l'utilisateur n'a potentiellement pas encore passé l'ensemble des campagnes. On calculait donc un pourcentage de succès non pertinent.

Pour solutionner le premier problème nous avons décidé de nous baser sur les KEs comme source de vérité. Pour un tubeId et un level donné on utilise donc le KE le plus récent. Peu importe qu'il soit valide ou non. L'idée est d'avoir un résultat le plus cohérent possible avec les actions de l'utilisateur.

Pour le deuxième problème, on a décidé de faire une photo à l'instant T des profiles cibles pour lesquels l'utilisateur n'a pas encore passé de campagnes. C'est ce qui se rapproche le plus de ce sur quoi l'utilisateur est sensé être évalué. Il est possible qu'au final les campagnes qu'il passe porte sur des acquis plus anciens. Cependant la différence de pourcentage sur le résultat devrait être négligeable.

😋 Pour tester

  • Avec un utilisateur vierge
  • Passer le niveau 1 sur la comptéence 1.1
  • Faire la campagne HDDWHC481 . qui couvre la compétence 2.1

⚠️ il doit répondre à sa dernière question sans le bouton magique sinon nous ne passerons pas dans les requirement reward-user ⚠️

l'attestation sera disponible sur l'orga PRO Classic

@laura-bergoens laura-bergoens added the team-noa(h) noa - noah 🎾 label Feb 14, 2025
@laura-bergoens laura-bergoens self-assigned this Feb 14, 2025
@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 :

@laura-bergoens laura-bergoens force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch from 091e833 to 5445cd1 Compare February 14, 2025 13:54
@laura-bergoens laura-bergoens force-pushed the pix-16375-add-eligibility-requirement-that-checks-if-participation-has-ended branch from b067a11 to 7719619 Compare February 17, 2025 09:41
@laura-bergoens laura-bergoens force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch from 5445cd1 to de0770e Compare February 17, 2025 09:48
@xav-car xav-car force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch from de0770e to c945fac Compare February 18, 2025 15:04
@xav-car xav-car changed the base branch from pix-16375-add-eligibility-requirement-that-checks-if-participation-has-ended to pix-16445-refacto-quests February 18, 2025 15:04
@xav-car xav-car changed the title [TECH] Les prérequis d'éligibilité et de succès sont de même type et interchangeables + essai d'ajouter cappedTubes requirement (PIX-???) [TECH] Ajouter cappedTubes requirement (PIX-???) Feb 18, 2025
@laura-bergoens laura-bergoens force-pushed the pix-16445-refacto-quests branch from 33312a3 to 6cadbff Compare February 19, 2025 09:13
@xav-car xav-car changed the title [TECH] Ajouter cappedTubes requirement (PIX-???) [TECH] Ajouter cappedTubes requirement (PIX-16539) Feb 20, 2025
@xav-car xav-car force-pushed the pix-16445-refacto-quests branch 2 times, most recently from ea33528 to e9c6604 Compare February 24, 2025 10:02
Base automatically changed from pix-16445-refacto-quests to dev February 24, 2025 10:29
@laura-bergoens laura-bergoens force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch 4 times, most recently from e5f98a0 to 07fe149 Compare March 17, 2025 10:31
@frinyvonnick frinyvonnick force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch 3 times, most recently from 7da4c0d to 54016c5 Compare March 17, 2025 15:43
@laura-bergoens laura-bergoens force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch from 54016c5 to d2f6741 Compare March 17, 2025 15:49
@xav-car xav-car force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch 2 times, most recently from b6beda3 to a353f29 Compare March 18, 2025 16:05
@frinyvonnick frinyvonnick force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch 2 times, most recently from 248e6fe to bd47f0c Compare March 19, 2025 13:27
@xav-car xav-car removed the request for review from a team March 21, 2025 15:44
@xav-car
Copy link
Contributor

xav-car commented Mar 21, 2025

Valider en RA par @AlexNaudyy

@xav-car xav-car added Tech Review OK 🚀 Ready to Merge Func Review OK PO validated functionally the PR and removed 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Mar 21, 2025
laura-bergoens and others added 20 commits March 21, 2025 15:45
Co-authored-by: Yvonnick Frin <yvonnick.frin@pix.fr>
Co-authored-by: Xavier Carron <xavier.carron@pix.fr>
Co-authored-by: Yvonnick Frin <yvonnick.frin@pix.fr>
Co-authored-by: Xavier Carron <xavier.carron@pix.fr>
Co-authored-by: Yvonnick Frin <yvonnick.frin@pix.fr>
Co-authored-by: Xavier Carron <xavier.carron@pix.fr>
 Co-authored-by: Yvonnick Frin <yvonnick.frin@pix.fr>
@pix-service-auto-merge pix-service-auto-merge force-pushed the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch from dd441ba to ffe6f45 Compare March 21, 2025 15:45
@pix-service-auto-merge pix-service-auto-merge merged commit 57ec20a into dev Mar 21, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the tech-everything-is-a-requirement-and-adding-capped-tube-requirement branch March 21, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Func Review OK PO validated functionally the PR 🚀 Ready to Merge team-noa(h) noa - noah 🎾 Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants