| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 221b5b45-bf59-49a2-a0dc-84039a66d3c2 📥 CommitsReviewing files that changed from the base of the PR and between a789ece and 52793a7. 📒 Files selected for processing (1)
📝 Walkthrough WalkthroughThe Pages resource now exposes workspace- and project-scoped page update endpoints. Both methods serialize UpdatePage data, issue PATCH requests, and return validated Page models. ChangesPages API updates
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5 ✅ Passed checks (5 passed)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. ❤️ ShareComment @coderabbitai help to get the list of available commands. |
Sorry, something went wrong.
|
Heads up — this is a dependency for makeplane/plane-mcp-server#190, which needs update_workspace_page()/update_project_page() to implement its update_page MCP tool. Would appreciate a look here so that PR can move forward. |
Sorry, something went wrong.
|
@akhil-vamshi-konam @sangeethailango I dont know if I am tagging you guys correctly but I think these features are important so we can use MCP properly, me and my team will be working heavily with wiki and it will be crucial to have these, any recommendations or anything so we can proceed with these faster? |
Sorry, something went wrong.
|
Hey @demetere @Galeforce1 , yes — I’ve raised the corresponding functionality in both the SDK and MCP:
The SDK PR includes the page update and archive/unarchive APIs, along with the relevant tests. For MCP, I raised #202 separately because we recently consolidated the tools, which changed how we define and expose them. We’ll get these reviewed and merged soon so the functionality can move forward smoothly. |
Sorry, something went wrong.
|
The required functionality is now supported in the Python SDK (v0.2.23). Closing this PR as it’s no longer needed. Thanks for the contribution! |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Overview
Adds update_workspace_page and update_project_page to the Pages resource.
Motivation
The UpdatePage model already exists in plane/models/pages.py, but no
resource method uses it — Pages currently supports list, retrieve,
create, and delete, but not update. This closes that gap, following the
exact pattern already used by Cycles.update(), Projects.update(),
and other resources (self._patch(...) + data.model_dump(exclude_none=True)).
Changes
No new models, no breaking changes — purely additive.
Summary by CodeRabbit