-
Notifications
You must be signed in to change notification settings - Fork 55
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] Expliciter la définiton des quêtes (PIX-16445) #11347
base: dev
Are you sure you want to change the base?
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 : |
5919150
to
7d45a16
Compare
850de15
to
d4b5566
Compare
Bloqué car on attend de passer la feature pour éditer les quêtes côté pixadmin et synchroniser l'ensemble |
d4b5566
to
1588ce6
Compare
261bfb3
to
bb247c6
Compare
3fe5efd
to
3286665
Compare
Une fois les applications déployées, elles seront accessibles via les liens suivants :
Les variables d'environnement seront accessibles via les liens suivants : |
Bloqué car on attend la prochaine MEP 🎉 et on va livrer sa branche enfant dans la meme MEP |
a55c802
to
9c22f42
Compare
12e1c71
to
9ef0271
Compare
a97f35d
to
b5a8c55
Compare
85e9717
to
c804aa5
Compare
c62ca8f
to
7af2451
Compare
api/src/quest/domain/models/Quest.js
Outdated
|
||
const eligibilityRequirements = otherRequirements; | ||
if (campaignParticipationRequirements.length > 0) { | ||
eligibilityRequirements.push({ |
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.
thought: Si dans le même COMPOSE on a différents type -> CA BUG (cherchez un peu la)
…inations Co-authored-by: Yvonnick Frin <[email protected]> Co-authored-by: Xavier Carron <[email protected]>
…le truc) Co-authored-by: Yvonnick Frin <[email protected]> Co-authored-by: Xavier Carron <[email protected]>
Co-authored-by: Laura BERGOENS <[email protected]>
Co-authored-by: Yvonnick Frin <[email protected]> Co-authored-by: Xavier Carron <[email protected]>
Co-authored-by: Yvonnick Frin <[email protected]> Co-authored-by: Xavier Carron <[email protected]>
7af2451
to
e40bf13
Compare
🥞 Problème
Rendre plus explicite, dans la défintion de la quête, son fonctionnement.
Pour l’attestation, on s’attend à ce qu’un prescrit ait une participation pour chaque profil cible listé dans la collection fournie dans le requirement. Mais ce n’est pas clair dans la configuration de la quête, et ça a impliqué du code supplémentaire et difficile à lire pour décortiquer cette contrainte.
🥓 Proposition
Si on souhaite qu’un prescrit ait une campagne participation pour plusieurs profil cibles différents, en fait ça doit impliquer 1 requirement pour chaque participation. Il faut rendre ceci explicite dans la définition de la quête, et de fait ça exprimera le parcours pédagogique souhaité plus clairement dans la quête (et au passage ça simplifie un peu le code)
🧃 Remarques
Trucs à faire avant / pendant / après :
😋 Pour tester
Non régression fonctionnelle sur l'obtention d'attestation