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] Empêcher la validation d'un live-alert assigné à une épreuve déjà répondue (PIX-16783). #11542

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

Conversation

Jeyffrey
Copy link
Contributor

@Jeyffrey Jeyffrey commented Feb 28, 2025

🥞 Problème

En prod, on a eu le cas de live-alerts acceptées alors qu'en fait l'épreuve avait déjà été répondue.
Ça créait, lors de la finalisation de session, des erreurs 500.

🥓 Proposition

Dans le use-case validate-live-alert, checker si l'épreuve concernée est déjà répondue.
Si oui, la révoquer et déclencher une erreur.

Celle-ci sera visible par le surveillant via une notification lui indiquant que l'épreuve a déjà été répondue.

🧃 Remarques

On veut aussi empêcher la validation + le signalement d'une même épreuve côté front.
Cela sera fait dans une autre PR.

Reste à faire

  • traduction de l'erreur renvoyée

😋 Pour tester

  • Créer une session de certif
  • La commencer avec un utilisateur
  • Répondre à une épreuve
  • Via SQL, ajouter une live-alert pour cette épreuve déjà répondue
  • Côté surveillant, valider cette live-alert
  • ✅ Constater l'affichage de l'erreur et, en BDD, vérifier que cette live-alert est bien dismiss.
  • ✅ Finaliser la session et vérifier que tout se passe bien !

@Jeyffrey Jeyffrey requested a review from a team as a code owner February 28, 2025 15:31
@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-16783-prevent-answered-challenge-to-be-alert-validated branch from bcd31c8 to ec9495a Compare February 28, 2025 16:18
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.

3 participants