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] Corriger le bug d'affichage de la page de résultat (PIX-16318). #11297

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

matthiasferraina
Copy link
Contributor

@matthiasferraina matthiasferraina commented Jan 31, 2025

🥞 Problème

Actuellement, il existe deux moyens pour envoyer ses résultats en fin de campagne. Un dans le bloc d'afficahge des résultats, l'autre dans le bloc d'affichage des contenus formatif. Celle dans le bloc d'affichage des contenus formatif cause un bug d'affichage du pourcentage de réussite après partage des résultats. Elle n'utilise pas non plus la même logique que le bouton situé dans le bloc d'affichage des résultats, ce qui pourrait causer des écarts de comportement.

🥓 Proposition

Dédupliquer le code utilisé pour partager les réusltats.

🧃 Remarques

Il pourrait être judicieux de déplacer côté api la logique présente dans le service créé. Il serait même nécessaire de le faire dans le cas où il y aurait besoin d'une logique transactionnelle entre le partage des résultats d'une participation de campagne, et le partage d'une attestation.

😋 Pour tester

  • se connecter en RA avec le mail : [email protected]
  • remettre à 0 et tout retenter cette campagne
  • partager le résultat depuis l'onglet des contenus formatifs
  • constater le bon affichage du pourcentage de réussite
  • remettre à 0 la campagne
  • repasser la campagne
  • la partager depuis le bloc d'affichage des résultats
  • constater le bon affichage du pourcentage de réussite

@matthiasferraina matthiasferraina added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally labels Jan 31, 2025
@matthiasferraina matthiasferraina self-assigned this Jan 31, 2025
@matthiasferraina matthiasferraina changed the title [BUGFIX] Corriger le bug d'affichage de la page de résultat. [BUGFIX] Corriger le bug d'affichage de la page de résultat (PIX-16318). Jan 31, 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 :

@matthiasferraina matthiasferraina force-pushed the pix-16318-fix-mastery-rate-display branch from ea6e02b to 765aa17 Compare January 31, 2025 14:29
@matthiasferraina matthiasferraina added the team-evaluation PR relatives à l'expérience d'évaluation label Jan 31, 2025
@matthiasferraina matthiasferraina force-pushed the pix-16318-fix-mastery-rate-display branch from 765aa17 to 659304f Compare January 31, 2025 15:30
@matthiasferraina matthiasferraina force-pushed the pix-16318-fix-mastery-rate-display branch from 8ee4a5f to 6c4848f Compare January 31, 2025 16:25
@pix-service-auto-merge pix-service-auto-merge merged commit 03f7db0 into dev Jan 31, 2025
6 of 10 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the pix-16318-fix-mastery-rate-display branch January 31, 2025 16:32
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 🚀 Ready to Merge team-evaluation PR relatives à l'expérience d'évaluation Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants