Translate error message when service connection in loginView fails #662
Conversation
Bulgus
changed the title
| @@ -90,6 +90,15 @@ const LoginView: React.FC<{ | |||
| onLogin(username, password, customFieldsDict); | |||
| }; | |||
|
|
|||
| const translateError = (error: string | null): string | null => { | |||
| if (!error) return null; | |||
| if (error.includes("challenge")) { | |||
There was a problem hiding this comment.
challenge ne fais pas référence à un captcha, mais au processus d'authentification
https://github.com/LiterateInk/Pawnote/blob/c6a09623f095275942941058ffc4ac11557f4c5d/src/api/helpers/login.ts#L211-L221
There was a problem hiding this comment.
Mince, j'ai fais un mélange des deux, le problème est arrivé de mon côté lorsqu'un captcha a été mis en place sur educonnect.
Mais ça peut être d'autres choses que ça effectivement ^^'
Je passe le texte à quelque chose de plus général, invitant l'élève à utiliser la connexion par l'ENT
There was a problem hiding this comment.
J'ai fais au plus simple, suppression de la fonction de traduction, et affichage d'un message d'erreur par défaut invitant à vérifier ses identifiants, ou se connecter via l'ENT.
Je ne vois pas d'autres potentiels messages d'erreurs nécessitants des traductions, mais dis moi si je dois laisser la fonction pour permettre la prise en charge de plusieurs messages d'erreurs différents.
Bulgus
changed the title
Checklist d'avant pull request
Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].
TODO(aka des annotations pour du code manquant) dans vos modificationsChangelogs proposés
loginViewfails : Impossible de se connecter, vérifie tes identifiants ou utilise le portail de ton ENT pour te connecter.The current message is not understandable for most users, and the problem can often be solved by logging on via ENT.
This prevents the user from abandoning the connection without understanding the problem.