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

ETQ candidat, j'ai un retour plus clair en cas d'erreur quand je soumet une nouvelle convention valide et je peux retenter la soumission de la convention #2833

Closed
bbohec opened this issue Jan 17, 2025 · 1 comment · Fixed by #2862
Assignees

Comments

@bbohec
Copy link
Contributor

bbohec commented Jan 17, 2025

🐛 Le problème

Les erreurs des partenaires non prévues fait une erreur interne serveur et donc code 503.
Or le schéma de l'API route POST /demandes-immersion n'autorise pas ce type de code retour.

De plus, il n'est pas possible de retenter de soumettre la convention suite à un problème temporaire alors que la convention semble valide.

👣 Étapes pour reproduire

Soumettre une convention et avoir une indispo de l'API Sirene en même temps.

Quand on retente de soumettre la convention, on a un loader infini et l'appel réseau qui ne sort pas, on ne peut que faire un reload car on a les query params de convention brouillon dans l'url.

🎯 Comportement attendu

On ne devrait pas avoir ce message d'erreur. On devrait plutôt avoir le message technique fourni par le back ici:
"Le service Sirene API n'est pas disponible".

D'ailleurs profiter de ce soucis pour modifier ce message par :
"Le service Sirène API est temporairement indisponible. Veuillez réessayer ultérieurement.".

Ca arrive que l'API Sirene ait une indispo, il faudra d'ailleurs faire le point avec eux niveau SLAs..... Cette erreur d'indispo n'arrive un peu trop souvent par rapport aux usage qu'on en fait.

Pour le retry, on devrait pour retenter à l'infini d'envoyer une conv à partir du moment où le form l'a validé.

Screenshots

Image

Image

@bbohec bbohec moved this from Vrac to Prêt à développer in Immersion Facilitée Jan 17, 2025
@bbohec bbohec changed the title ETQ candidat, j'ai un retour plus clair en cas d'erreur quand je soumet une nouvelle convention ETQ candidat, j'ai un retour plus clair en cas d'erreur quand je soumet une nouvelle convention valide et je peux retenter la soumission de la convention Jan 17, 2025
@bbohec
Copy link
Contributor Author

bbohec commented Jan 17, 2025

@gargagif gargagif moved this from Prêt à développer to À faire dans l'itération in Immersion Facilitée Jan 21, 2025
@clement-duport clement-duport moved this from À faire dans l'itération to En développement in Immersion Facilitée Jan 22, 2025
@clement-duport clement-duport self-assigned this Jan 22, 2025
@clement-duport clement-duport moved this from En développement to Ready for review in Immersion Facilitée Jan 22, 2025
@github-project-automation github-project-automation bot moved this from Ready for review to En staging in Immersion Facilitée Jan 23, 2025
@JeromeBu JeromeBu moved this from En staging to En production in Immersion Facilitée Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: En production
2 participants