-
Notifications
You must be signed in to change notification settings - Fork 1
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
Children #81
Children #81
Conversation
tests/test_children.py
Outdated
|
||
|
||
def test_create_and_delete_blocks(): | ||
url = "https://www.notion.so/innolitics/Test-Page-46bc615bed5a479e8390d9373279426f" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should use the same notion pages and access token we use for the end to end tests here. See the doc string at the top of tests/end_to_end_tests.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See my individual comments for details
n2y/notion.py
Outdated
headers = {**self.headers} | ||
del headers['Content-Type'] | ||
response = requests.delete( | ||
f"{self.base_url}blocks/{block_id}", headers=self.headers |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line should use headers
, not self.headers
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I had one small comment that needs to be addressed, but it looks good to merge otherwise.
No description provided.