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

[TECH] Affichage d'une erreur spécifique à l'enregistrement d'un candidat lorsque la session est finalisée (PIX-13862) #10248

Conversation

mcampourcy
Copy link
Contributor

@mcampourcy mcampourcy commented Oct 4, 2024

🦄 Problème

Sur Pix Certif, lors de l’inscription d’un candidat à une session de certification qui est déjà finalisée, l’API nous renvoie un message d’erreur spécifique (statut: 403, detail: "Cette session a déjà été finalisée, l'ajout de candidat n'est pas autorisé")

À la place d’afficher le message d’erreur spécifique, nous affichons dans le toaster un message beaucoup plus générique

Capture d’écran 2024-10-04 à 11 51 09

🤖 Proposition

Afficher dans le toaster d’erreur le message spécifique renvoyée par l’API dans le cas d’une inscription d’un candidat à une session de certification déjà finalisée.

🌈 Remarques

J'en ai profité pour fixer un bug sur l'adresse. En cas d'erreur sur l'adresse mail à l'ajout d'un candidat, le toaster d'erreur n'apparaissait pas.

💯 Pour tester

  • se connecter avec [email protected]
  • aller sur la liste des sessions
  • finaliser une session
  • vérifier que le message est le suivant :
Capture d’écran 2024-10-04 à 11 32 54

@mcampourcy mcampourcy self-assigned this Oct 4, 2024
@mcampourcy mcampourcy marked this pull request as ready for review October 4, 2024 09:52
@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 :

@@ -248,6 +250,15 @@ export default class EnrolledCandidates extends Component {
}
}

_handleApiError(errorResponse) {
Copy link
Member

Choose a reason for hiding this comment

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

un ptit test ? :)

Copy link
Member

@AndreiaPena AndreiaPena left a comment

Choose a reason for hiding this comment

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

Je me dit qu'on devrait masquer le bouton d'inscription candidat si la session est finalisée 🤔

Mais test func ✅
également le truc des email ✅ 🥳

@mcampourcy
Copy link
Contributor Author

mcampourcy commented Oct 7, 2024

Je me dit qu'on devrait masquer le bouton d'inscription candidat si la session est finalisée 🤔

Mais test func ✅ également le truc des email ✅ 🥳

Ah pas faux, ça serait logique ! Je vais voir avec Anne-So

[EDIT] Vu avec Anne-So, elle prend note, elle prendra le temps de revoir le workflow pour réfléchir au ménage à faire sur les actions !

@mcampourcy mcampourcy force-pushed the pix-13862-add-specific-error-message-on-403-add-candidate-error branch 2 times, most recently from b989836 to 3bff2b9 Compare October 7, 2024 12:42
@alexandrecoin alexandrecoin force-pushed the pix-13862-add-specific-error-message-on-403-add-candidate-error branch from 3bff2b9 to 99d9fde Compare October 8, 2024 08:57
@alexandrecoin alexandrecoin merged commit b97aa51 into dev Oct 8, 2024
6 of 8 checks passed
@alexandrecoin alexandrecoin deleted the pix-13862-add-specific-error-message-on-403-add-candidate-error branch October 8, 2024 09:03
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 team-certif Tech Review OK
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants