-
Notifications
You must be signed in to change notification settings - Fork 57
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
Merged
pix-service-auto-merge
merged 20 commits into
dev
from
tech-everything-is-a-requirement-and-adding-capped-tube-requirement
Mar 21, 2025
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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
commented
Feb 14, 2025
091e833
to
5445cd1
Compare
b067a11
to
7719619
Compare
5445cd1
to
de0770e
Compare
de0770e
to
c945fac
Compare
33312a3
to
6cadbff
Compare
ea33528
to
e9c6604
Compare
e9c6604
to
4d12a0a
Compare
e5f98a0
to
07fe149
Compare
7da4c0d
to
54016c5
Compare
54016c5
to
d2f6741
Compare
b6beda3
to
a353f29
Compare
248e6fe
to
bd47f0c
Compare
Valider en RA par @AlexNaudyy |
Co-authored-by: Yvonnick Frin <yvonnick.frin@pix.fr> Co-authored-by: Xavier Carron <xavier.carron@pix.fr>
…ations in campaign API
Co-authored-by: Yvonnick Frin <yvonnick.frin@pix.fr> Co-authored-by: Xavier Carron <xavier.carron@pix.fr>
… of skill, even if first version validated
… quest in provided data
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>
…nContributingToQuest in Quest model
…cess requirements
…to get target profiles id contributing
dd441ba
to
ffe6f45
Compare
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
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🥞 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 :
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
HDDWHC481
. qui couvre la compétence 2.1l'attestation sera disponible sur l'orga PRO Classic