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

[FEATURE] Relier le composant transitoire à la page de fin de parcours et à la modal du tab "Formation"(PIX-17016) #11827

Merged
merged 8 commits into from
Mar 28, 2025

Conversation

dianeCdrPix
Copy link
Contributor

@dianeCdrPix dianeCdrPix commented Mar 25, 2025

🌸 Problème

La modale de fin de parcours existe mais n'est pas affichée quand on envoie ses résultats.

🌳 Proposition

Afficher la modal de fin de parcours quand on envoie ses résultats. Quand on clique sur "Fermer et voir mes résultats", on retourne à la page avec les résultats et les récompenses.

🐝 Remarques

  • Dans quit-results.gjs, <PixModal> a été extrait du bloc if/else afin d'éviter la destruction de cette modal lorsque on partage les résultats de campagne. Ainsi, la modale ne se détruisant pas, le modifier trap-focus n'appelle pas son destructor dont le rôle est de rajouter un scroll sur la page.
  • A faire : ajouter l'action d'afficher la modale pour le bouton de EvaluationResultsHero -> dans un autre ticket
  • Un featureToggle a été créé pour cette fonctionnalité. Son nom : isModalSentResultEnabled

🤧 Pour tester

  • Aller sur la campagne EDUSIMPLE en se connectant avec le compte [email protected]
  • Aller dans l'onglet Formations
  • Cliquer sur le bouton J'envoie mes résultats
  • Vérifier que la modale s'affiche et qu'elle contient des contenus formatifs 😸

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

@er-lim er-lim force-pushed the PIX-17016-display-sent-result-modal branch 2 times, most recently from ec3aeb5 to 48acf8b Compare March 25, 2025 10:52
@dianeCdrPix dianeCdrPix force-pushed the PIX-17016-display-sent-result-modal branch 2 times, most recently from e36475d to b4024ce Compare March 26, 2025 16:03
@er-lim er-lim force-pushed the PIX-17016-display-sent-result-modal branch from b4024ce to af020cb Compare March 27, 2025 08:20
@dianeCdrPix dianeCdrPix force-pushed the PIX-17016-display-sent-result-modal branch from 4dc000e to 714d36f Compare March 27, 2025 10:30
@dianeCdrPix dianeCdrPix reopened this Mar 27, 2025
@dianeCdrPix dianeCdrPix requested review from clemlatz and er-lim March 27, 2025 10:35
@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 :

@er-lim er-lim reopened this Mar 27, 2025
@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 :

featureToggles = { isModalSentResultEnabled: true };
}
this.owner.register('service:featureToggles', FeatureTogglesStub);
this.model.trainings = [
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: on pourrait factoriser la génération des trainings dans une méthode privée pour alléger la lecture du test car au final tous les détails n'impactent pas la réussite test

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dans un beforeEach, qu'en penses-tu ?

dianeCdrPix and others added 3 commits March 28, 2025 09:19
in quit-results

Co-authored-by: Yann Bertrand <[email protected]>
Co-authored-by: Eric Lim <[email protected]>
in sent results modal

Co-authored-by: Diane Cordier <[email protected]>
Co-authored-by: Eric Lim <[email protected]>
by adding not sending results learners in EDUSIMPLE campaign
@pix-service-auto-merge pix-service-auto-merge force-pushed the PIX-17016-display-sent-result-modal branch from 20c5e58 to c08f960 Compare March 28, 2025 09:19
@pix-service-auto-merge pix-service-auto-merge merged commit 211a01c into dev Mar 28, 2025
9 of 11 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the PIX-17016-display-sent-result-modal branch March 28, 2025 09:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants