Skip to content

NotionException: Use status from http response #88

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

Conversation

fschirinzi
Copy link

In a try-catch it is useful to get the http status code. For example, to check if too many requests were made (Status 429).

johguentner and others added 2 commits August 1, 2022 18:46
@johguentner johguentner self-assigned this Dec 7, 2022
@johguentner johguentner added fix tests-required Tests for this PR are required labels Dec 7, 2022
@johguentner johguentner changed the base branch from main to dev December 14, 2022 15:07
@johguentner johguentner added this to the 🌳 v1.0.0 milestone Jan 26, 2023
@johguentner
Copy link
Member

Thank you for your PR @fschirinzi.
This has already been solved within #95 (including tests), that's why I'm closing this PR.
This fix will be released with Version v1.0.0 in the next few days

@johguentner johguentner closed this Feb 2, 2023
johguentner added a commit that referenced this pull request Feb 2, 2023
NotionException: Use status from http response #88 additions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix tests-required Tests for this PR are required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants