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

[BUGFIX] Ne pas afficher l'encart attestation pour les participations a des campagnes sans lien avec les quêtes (PIX-15308) #10550

Conversation

Alexandre-Monney
Copy link
Contributor

🍂 Problème

Pendant des tests fonctionnels sur cette PR : #10411
Nous nous sommes rendus compte que l'on affichait l'encart pour n'importe quelle participation a une campagne.
On essayait de réecrire la valeur d'une propriété d'un objet, sauf que cette propriété était un getter. Donc ça ne fonctionnait pas, car à chaque fois le getter renvoyait la donnée d'origine

🌰 Proposition

Ecrire un test mettant en avant le bug que nous avons rencontré, et corriger ce soucis bloquant pour la suite du dev.
On utilise Object.defineProperty pour nous permettre de faire ce qu'on veut. C'est une des solutions qu'on a trouvé pour corriger le bug sans avoir besoin de retoucher a tout le code utilisant l'objet Eligibility

🎃 Remarques

🪵 Pour tester

Participer a une campagne n'ayant aucun lien avec une qûete
Verifier que l'encart d'attestation n'apparait pas a la fin du parcours
Verifier que par contre dans un parcours avec lien sur une quête cet encart apparait bien

@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 :

@Alexandre-Monney Alexandre-Monney changed the base branch from pix-13826-display-attestation-on-assessment-result-page to dev November 15, 2024 10:34
@Alexandre-Monney Alexandre-Monney force-pushed the pix-15308-fix-quest-results-at-end-of-campaign-participations branch from 58ea041 to 47d5fcf Compare November 15, 2024 10:35
@Alexandre-Monney Alexandre-Monney force-pushed the pix-15308-fix-quest-results-at-end-of-campaign-participations branch from 47d5fcf to a5904b8 Compare November 15, 2024 10:37
@Alexandre-Monney Alexandre-Monney enabled auto-merge (squash) November 15, 2024 10:39
@Alexandre-Monney Alexandre-Monney merged commit 059c525 into dev Nov 15, 2024
9 checks passed
@Alexandre-Monney Alexandre-Monney deleted the pix-15308-fix-quest-results-at-end-of-campaign-participations branch November 15, 2024 10:46
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 team-evaluation PR relatives à l'expérience d'évaluation team-prescription Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants