[TECH] Expliciter la définiton des quêtes (PIX-16445) #11347

laura-bergoens · 2025-02-05T15:59:14Z

🥞 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)

// avant
{
  type: campaignParticipations,
  data: {
     targetProfileIds: [1, 2],
  }
  comparison: all,
};

// après
[
  {
     type: campaignParticipations,
     data: {
      targetProfileId: 1,
     },
    comparison: all,
  }, 
  {
     type: campaignParticipations,
     data: {
      targetProfileId: 2,
     },
    comparison: all
  }, 
]

🧃 Remarques

Trucs à faire avant / pendant / après :

Attendre que les quêtes soient éditées dans le nouveau format
lancer le script de rattrapage d'attestation sur la durée entre le moment où les quêtes sont modifiées VS le déploiement de la MEP qui porte cette PR

😋 Pour tester

Non régression fonctionnelle sur l'obtention d'attestation

pix-bot-github · 2025-02-05T16:00:02Z

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 · 2025-02-06T13:31:52Z

Bloqué car on attend de passer la feature pour éditer les quêtes côté pixadmin et synchroniser l'ensemble

api/src/quest/domain/models/Eligibility.js

api/src/quest/domain/models/Quest.js

pix-bot-github · 2025-02-10T08:46:13Z

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 · 2025-02-10T09:32:04Z

Bloqué car on attend la prochaine MEP 🎉 et on va livrer sa branche enfant dans la meme MEP

laura-bergoens · 2025-02-12T10:20:51Z

api/tests/quest/unit/domain/models/Criterion_test.js

+    });
+
+    describe('when criterion attribute is an Array', function () {
+      describe('when item attribute is flat', function () {


is not an array

laura-bergoens · 2025-02-12T10:24:19Z

api/tests/quest/unit/domain/models/EligibilityRequirement_test.js

+      });
+
+      describe('when comparison is ONE_OF', function () {
+        it('should return true if one of the requirements is met', function () {


ce n'est pas un ou exclusif !
SI les deux reuqirements sont met, alors true aussi

l'ajouter dans ma doc !

it('should return true if all requi are met")

…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]>

… definition

Co-authored-by: Yvonnick Frin <[email protected]> Co-authored-by: Xavier Carron <[email protected]>

Alexandre-Monney · 2025-02-13T15:59:43Z

api/src/quest/domain/models/Quest.js

+
+    const eligibilityRequirements = otherRequirements;
+    if (campaignParticipationRequirements.length > 0) {
+      eligibilityRequirements.push({


thought: Si dans le même COMPOSE on a différents type -> CA BUG (cherchez un peu la)

laura-bergoens added Development in progress team-noa(h) noa - noah 🎾 labels Feb 5, 2025

laura-bergoens assigned frinyvonnick, xav-car and laura-bergoens Feb 5, 2025

xav-car force-pushed the pix-16445-refacto-quests branch 2 times, most recently from 5919150 to 7d45a16 Compare February 5, 2025 17:12

laura-bergoens force-pushed the pix-16445-refacto-quests branch 2 times, most recently from 850de15 to d4b5566 Compare February 6, 2025 13:31

laura-bergoens added ⚠️ Blocked 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Feb 6, 2025

laura-bergoens force-pushed the pix-16445-refacto-quests branch from d4b5566 to 1588ce6 Compare February 6, 2025 13:45