Syncro CRM : utilisation du capture de message Sentry au lieu de raise

[AntoineGirard]

Contexte

Dans le cadre du job de synchro avec Notion, j’avais raise quand le format de l’URL ne permettait pas de déterminer le territoire ou l’organisation.
En faisant cela, le job s’arrête et n’essaye pas d’aller synchroniser les éléments restants.

Solution

On utilise un capture message Sentry pour avoir l’info sans pour autant arrêter la synchro.

[victormours]

pour un job qui fait plusieurs appels d'api, l'idéal ça serait qu'il lance un batch de job unitaire qui ferait chacun un unique appel d'api. Ça évite ce genre de bug, et ça permet de faire qu'un retry du job ne fait que le minimum d'appel nécessaire.
Mais pour une amélioration de cooldown, c'est très bien comme ça. 👍

[AntoineGirard]

Yes totalement aligné avec toi @victormours
L’idée ici c’était juste de faire une v0 rapide qui répond au besoin :)
On pourra itérer dessus par la suite

[sentry-sentry-incubateur-net]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ Unrecognized account URL: https://rdv.anct.gouv.fr/super_admins/agents/1164 CronJob::SynchronizeCrm View Issue

_{Did you find this useful? React with a 👍 or 👎}