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] Améliorer les déclenchements d'action de l'écran d'épreuve (PIX-16784). #11565

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

Conversation

Jeyffrey
Copy link
Contributor

@Jeyffrey Jeyffrey commented Mar 4, 2025

🥞 Problème

À l'heure actuelle, rien n'empêche de déclencher plusieurs actions simultanément sur l'écran d'épreuve.

🥓 Proposition

  1. Si on clique sur "Valider" :

    • on passe le bouton en loading
    • on disable les boutons "Passer" et "Signaler un problème avec la question"
    • Si les sous-actions de signalement étaient visibles, alors elles disparaîssent
  2. Si on clique sur "Passer" :

    • on passe le bouton en loading
    • on disable les boutons "Valider" et "Signaler un problème avec la question"
    • Si les sous-actions de signalement étaient visibles, alors elles disparaîssent

🧃 Remarques

On en profite aussi pour migrer le bouton "Signaler un problème avec la question" sur le composant PixButton.

Aussi : côté PixButton, il faut améliorer l'a11y lors de l'état loading.

😋 Pour tester

  • Aller sur une épreuve en certif
  • Sur votre navigateur, passer le réseau en mode 3G pour ralentir la connexion
  • Cliquer sur "Je passe" ou "Je valide"
  • Constater que les actions de signalements sont plus accessibles

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

@Jeyffrey Jeyffrey force-pushed the pix-16784-improve-challenge-actions-triggering branch 3 times, most recently from 340ad0d to e67a928 Compare March 5, 2025 14:53
@Jeyffrey Jeyffrey marked this pull request as ready for review March 5, 2025 14:55
@Jeyffrey Jeyffrey force-pushed the pix-16784-improve-challenge-actions-triggering branch 2 times, most recently from 29a1a65 to 9c0f52f Compare March 5, 2025 16:00
@Jeyffrey Jeyffrey added 👀 Tech Review Needed 👀 Func Review Needed Need PO validation for this functionally and removed Development in progress labels Mar 5, 2025
@Jeyffrey Jeyffrey force-pushed the pix-16784-improve-challenge-actions-triggering branch from 9c0f52f to e61db02 Compare March 5, 2025 16:47
@Steph0
Copy link
Contributor

Steph0 commented Mar 6, 2025

Test func

Je passe puis tenter un signalement ✅
Je valide puis tenter un signalement ✅
Je valide, Ja7rrive a faire je passe avant d'avoir la question suivante ce qui trigger un PATCH sur la answers, ℹ️ ca n'a pas l'air de poser soucis si le candidat fait cela
Changement du feedback button ✅

Signalement > oui > je passe = HTTP 403 ⚠️
image

Copy link
Contributor

@Steph0 Steph0 left a comment

Choose a reason for hiding this comment

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

Vu ensemble a voir le "oui je suis sur" (a part cela c'est bon pour moi)

@Jeyffrey Jeyffrey force-pushed the pix-16784-improve-challenge-actions-triggering branch from e61db02 to 8205442 Compare March 6, 2025 10:22
@Jeyffrey Jeyffrey added the team-contenu PR relatives à l'équipe Contenu label Mar 6, 2025
Copy link
Contributor

@fael-b fael-b left a comment

Choose a reason for hiding this comment

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

Je teste ça fonctionnellement demain matin, mais ça à l'air bien. Merci ☺️

Copy link
Contributor

@fael-b fael-b left a comment

Choose a reason for hiding this comment

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

Func OK ✅

@Jeyffrey Jeyffrey force-pushed the pix-16784-improve-challenge-actions-triggering branch from 8205442 to b02bba6 Compare March 7, 2025 10:50
@Jeyffrey Jeyffrey force-pushed the pix-16784-improve-challenge-actions-triggering branch from b02bba6 to 6de09f4 Compare March 7, 2025 10:51
…dy answered

refactor(mon-pix): use PixButton for the challenge live-alert trigger
@Jeyffrey Jeyffrey force-pushed the pix-16784-improve-challenge-actions-triggering branch from 6de09f4 to f4b3b67 Compare March 7, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
👀 Func Review Needed Need PO validation for this functionally team-certif team-contenu PR relatives à l'équipe Contenu Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants