-
Notifications
You must be signed in to change notification settings - Fork 31
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
Meilleure gestion d'erreur dans les numéros de téléphone #4643
base: main
Are you sure you want to change the base?
Meilleure gestion d'erreur dans les numéros de téléphone #4643
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
J'ai l'impression qu'il y a une PR de refacto intermédiaire à réaliser sur le catch des erreurs et l'envoi à sentry lié à createAxiosInstance de sms-service.ts
Est-ce que tu peux détailler la procédure de test pour cette correction s'il te plaît ? |
Ok. Ça ne règle pas le problème de la #4648 pour la partie front avec un chargement infini (d'où la réalisation de cette dernière) : Avec le numéro de téléphone indiqué dans la description de la #4648, je n'arrive jamais dans les conditions que tu as modifiées. |
Peut-être qu'on devrait finaliser les autres tickets (2) et voir si cela suffit à résoudre le problème ? |
Une première itération pour réduire le volume d'erreur dans Sentry.
D'autres itérations avec d'autres PR sont à prévoir, notamment envoyer potentiellement l'e-mail même si le numéro de téléphone n'est pas bon.