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] Ajoute une page de loader avant les résultats de fin de parcours (PIX-14944). #10380

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

frinyvonnick
Copy link
Member

@frinyvonnick frinyvonnick commented Oct 22, 2024

🦄 Problème

Dans le cadre de l'Epix sur les attestations, on souhaite ajouter un loader avant l'écran de résultats de fin de parcours pour répondre à deux problématiques :

  • Laisser le temps au système de déterminer si l'attestation a été obtenue ou non
  • Contextualiser l'écran suivant

🤖 Proposition

On ajoute un loader qui affiche ce qui va être potentiellement présenter à l'utilisateur sur l'écran suivant. Cet écran est prévu pour être affiché 3.2 secondes.

🌈 Remarques

On a eu l’erreur “Assessment is already rate” mais on a pas réussi à reproduire. C'est une 412, il faudra être vigilant (cela ressemblait à de la race condition). -> Déjà existant en prod

On en profite pour retirer quelques futurs dépréciations pour ember data 6.x qui s'affichaient dans la console sur les pages concernées pour tester cette PR

💯 Pour tester

  • Passer une campagne d'évaluation
  • Constater que cette nouvelle page est là.

@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 force-pushed the pix-14944-add-loader-before-campaign-results branch from 321036c to 8bd5d26 Compare October 22, 2024 13:51
@frinyvonnick frinyvonnick force-pushed the pix-14944-add-loader-before-campaign-results branch from 84af908 to 7c350ed Compare October 22, 2024 15:35
@Alexandre-Monney Alexandre-Monney force-pushed the pix-14944-add-loader-before-campaign-results branch from 9d8d7d3 to 7d38f1f Compare October 23, 2024 12:37
@Libouk Libouk force-pushed the pix-14944-add-loader-before-campaign-results branch 2 times, most recently from 111c3de to ae7c5ef Compare October 23, 2024 13:32
@frinyvonnick frinyvonnick force-pushed the pix-14944-add-loader-before-campaign-results branch 2 times, most recently from 6fd84e5 to fe88856 Compare October 23, 2024 14:31
@frinyvonnick frinyvonnick marked this pull request as ready for review October 23, 2024 14:32
@Alexandre-Monney Alexandre-Monney force-pushed the pix-14944-add-loader-before-campaign-results branch from fe88856 to 76a1910 Compare October 23, 2024 14:56
@pierrepougetpix
Copy link

On peut enlever les "…" a la fin des phrase (ça fait bizarre quand tout est au vert)

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.

5 participants