-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[BUG] Update Page does not support Status type #6310 #6372
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
base: master
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Hi @lcaresia , thanks for the fix, looks good to me! Just check if that console.log
should remain there.
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.
Hi @lcaresia , thanks for the fix, looks good to me! Just check if that console.log
should remain there.
Hello everyone, I have tested this PR and there're some test cases failed or needed improvement. Please check the test report below for more information |
Hello @lcaresia, I have tested again, but even though I added the variable following the description, the notion page Status field still not updated. I have the page with Status = In progress Then I updated it to Not started with the full object (I also tried with only the Would you mind testing on your side? |
WHAT
🤖 Generated by Copilot at 9d1ae4d
Improved the
notion
package by adding more options and fixing issues for the update page action. Updated thepnpm-lock.yaml
andpackage.json
files with the latest package versions and dependencies.🤖 Generated by Copilot at 9d1ae4d
WHY
HOW
🤖 Generated by Copilot at 9d1ae4d
pnpm-lock.yaml
(link, link, link)