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

Handle response errors #10

Open
cbdm opened this issue Aug 2, 2023 · 0 comments
Open

Handle response errors #10

cbdm opened this issue Aug 2, 2023 · 0 comments
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers

Comments

@cbdm
Copy link
Owner

cbdm commented Aug 2, 2023

Currently there are no checks to validate if the response we got from the API is valid, so if the server is down or we have exceeded the daily quota, the behavior is undefined (e.g., we might silently fail, return a messed up calendar, show the user an exception). We should probably handle this in a more elegant manner, e.g., alert whether we don't have quota / the server is down and they should retry in X hours.

@cbdm cbdm added bug Something isn't working enhancement New feature or request good first issue Good for newcomers labels Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant