-
Notifications
You must be signed in to change notification settings - Fork 53
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
[FEATURE] Ajouter la bannière de la nouvelle page de fin de parcours (PIX-12987). #10230
base: dev
Are you sure you want to change the base?
Conversation
7580547
to
e800a05
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 : |
3a633ba
to
79503fc
Compare
594ae6f
to
89a1ff0
Compare
aeb890c
to
a6e1969
Compare
mon-pix/app/components/campaigns/assessment/skill-review/evaluation-results-hero/index.gjs
Outdated
Show resolved
Hide resolved
...omponents/campaigns/assessment/skill-review/evaluation-results-hero/retry-or-reset-block.gjs
Outdated
Show resolved
Hide resolved
b1630c6
to
29d583d
Compare
...ntegration/components/campaigns/assessment/skill-review/hero/evaluation-results-hero-test.js
Outdated
Show resolved
Hide resolved
acquired: this.args.campaignParticipationResult.reachedStage.reachedStage - 1, | ||
total: this.args.campaignParticipationResult.reachedStage.totalStage - 1, |
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.
refacto à faire dans un autre ticket pour faire :
- le calcul côté back
- renommer reachedStage.reachedStage en quelque chose de plus cohérent
mon-pix/app/components/campaigns/assessment/skill-review/evaluation-results-hero/index.gjs
Show resolved
Hide resolved
@@ -22,14 +22,11 @@ module('Integration | Components | Campaigns | Assessment | Skill Review | Evalu | |||
|
|||
this.owner.register('service:currentUser', currentUserService); | |||
|
|||
this.set('campaignParticipationResult', { masteryRate: 0.755 }); | |||
this.set('campaignParticipationResult', { masteryRate: 0.75 }); |
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.
A faire dans un autre ticket. Arrondir le pourcentage côté api et non côté front.
...ntegration/components/campaigns/assessment/skill-review/hero/evaluation-results-hero-test.js
Show resolved
Hide resolved
...ntegration/components/campaigns/assessment/skill-review/hero/evaluation-results-hero-test.js
Show resolved
Hide resolved
...ntegration/components/campaigns/assessment/skill-review/hero/evaluation-results-hero-test.js
Outdated
Show resolved
Hide resolved
mon-pix/tests/integration/components/routes/campaigns/assessment/evaluation-results-test.js
Outdated
Show resolved
Hide resolved
...ntegration/components/campaigns/assessment/skill-review/hero/evaluation-results-hero-test.js
Show resolved
Hide resolved
2e1f441
to
3a8ca6d
Compare
3a8ca6d
to
155791b
Compare
155791b
to
4067f14
Compare
|
Questions pour le design :
|
|
j'ai ajouté d'autres commentaire de review directement sur le fichier ici : https://www.figma.com/design/tlukMJuPbw6yec01fWDCuV/App---Rdy-Dev?node-id=11321-1342&t=sSMbYrkwKFViMhQV-1 |
🦄 Problème
Dans le nouvel écran de fin de parcours, il manque la bannière.
🤖 Proposition
Voir tous les cas d'affichage dans le ticket (trop de cas à lister ici).
💯 Pour tester
[email protected]
[email protected]
[email protected]