Refactoring: améliore la gestion des erreurs des followups #4652
Conversation
| } else { | ||
| return createSimulationRecapUrl(req, res) | ||
| if (phone) await sendSimulationResultsSms(followup) | ||
| if (email) await sendSimulationResultsEmail(followup) |
There was a problem hiding this comment.
J'aurais inversé, d'abord l'e-mail et ensuite le téléphone car plus de chance que le premier fonctionne.
On pourra aussi adapter le message en mode erreur avec le numéro de téléphone, l'email a bien été envoyer ressayer plus tard ou un truc du genre.
There was a problem hiding this comment.
Peut-être dans un second temps, faire les deux envois séparément avec des messages dédiés à chaque service (ex: "l'email a bien été envoyé", puis "Le sms a bien été envoyé") ?
| @@ -228,6 +223,6 @@ export async function logSurveyLinkClick(req: Request, res: Response) { | |||
| } catch (error) { | |||
| Sentry.captureException(error) | |||
There was a problem hiding this comment.
À postériori, je me demande si le captureException et le console error est pertinent, sachant qu'on ne retrouve plus le survey.
There was a problem hiding this comment.
Le console.error c'est sûr qu'il n'est pas utile. Le captureException, comme tu avais pris l'initiative de l'ajouter, je te laisse décider si tu souhaites le retirer. Je l'aurais aussi enlevé de mon côté
PR intermédiaire à #4648 / #4643