Comments (7)
Hey @jeromegit, thanks for opening the issue!
Indeed, we should add in_trash
. Rlated changelog from Notion: https://developers.notion.com/page/changelog#changes-for-april-2024
PR welcome!
from notion-sdk-py.
Hey @jeromegit @ramnes , thanks for your comments on this issue!
I've changed archived
to in_trash
in notion_client/api_endpoints.py
https://github.com/digzect/notion-sdk-py/blob/6b1eb8f9972bb851fc6585aa80a184b397bfd2d5/notion_client/api_endpoints.py#L235 , and the pytest
coverage is now 100%.
Am I ready to create a PR at this point? This is my first time contributing, so I would appreciate your guidance.
from notion-sdk-py.
Why removing archived
? The changelog entry above says that both are still supported. Did it get removed in the meantime?
Otherwise yes, just push your commit to a branch on your repository and open a PR!
from notion-sdk-py.
Thank you for your response.
I've reviewed the Notion API documentation:
- https://developers.notion.com/reference/patch-page uses
in_trash
instead ofarchived
- https://developers.notion.com/reference/update-a-block still uses
archived
The commit in question is specifically for the PagesEndpoint
, which is why I changed archived to in_trash.
Additionally, When using both archived
and in_trash
together in the code, the pytest
coverage remains at 100%.
Given this information, how would you suggest we proceed? Should we keep only in_trash for the PagesEndpoint, or include both for maximum compatibility?
from notion-sdk-py.
I'd say let's take a look at notion-sdk-js and do whatever the Notion developers did there. :)
from notion-sdk-py.
I've looked at notion-sdk-js and using both "archived"
and "in_trash"
together.
https://github.com/makenotion/notion-sdk-js/blob/7950edc034d3007b0612b80d3f424baef89746d9/src/api-endpoints.ts#L10514
Thank you for your guidance!
from notion-sdk-py.
Fixed in #236. Thanks!
from notion-sdk-py.
Related Issues (20)
- Logging configuration does not work HOT 2
- notion_client does not work in a class under multiprocessing HOT 2
- notion.databases.create: Is it possible to rearrange database fields according to some defined order? HOT 2
- Using asyncio for notion.pages.create requests HOT 1
- httpx 0.25.x is not compatible with notion client: "TypeError: ConnectionPool.__init__() got an unexpected keyword argument 'socket_options'" HOT 4
- Image upload error when create a new notion page HOT 1
- Alternative for `DatabasesEndpoint.list()` HOT 1
- Create database entry with `select` property HOT 1
- Deleting an icon or a cover is not possible.
- Switch to Ruff HOT 1
- iterate_paginated_api and api_endpoints#query seems to be broken in 2.2.0 HOT 3
- notion.blocks.append not accessible HOT 1
- Database query malformed schema HOT 1
- Wrong response when querying database rollup field with ascending/descending sort HOT 2
- Add Type Hints to Response and Arguments of a function HOT 1
- Add a GitHub CI cron job that open issues here for new Notion API changelog entries HOT 6
- Failed to insert block before first block on page
- How to query a Notion view like a relational database? HOT 1
- Just let us able to move the block
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from notion-sdk-py.