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

Betere foutafhandeling in api services #118

Open
cedrikv opened this issue Sep 29, 2023 · 0 comments
Open

Betere foutafhandeling in api services #118

cedrikv opened this issue Sep 29, 2023 · 0 comments
Labels

Comments

@cedrikv
Copy link
Contributor

cedrikv commented Sep 29, 2023

uit https://github.com/OnroerendErfgoed/plannen/pull/1067#discussion_r1341055490

Het zou imho ook properder zijn om de httpservice promise een error te laten gooien wanneer de response status niet 2XX is, en zo direct in het catch block van de caller komt, waarin de nodige error handling gebeurt. (De code zou niet tot aan die (reading 'data') mogen geraken)
Er is een optie "validateStatus" om dit tegoei uit te werken: https://axios-http.com/docs/handling_errors.
Nieuw issue eventueel om in de Vue Component lib uit te werken.

@cedrikv cedrikv changed the title Betere foutafandeling in api services Betere foutafhandeling in api services Jan 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant