docs(pinterest): board must be the numeric board id - #248
giladresisi wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe Pinterest settings reference now requires numeric board IDs, documents the validation error for board names, and provides updated instructions for finding board IDs. ChangesPinterest board ID documentation
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to Pinterest API documentation now tells users to provide numeric board IDs, explains the validation error for board names, and directs them to supported ID lookup paths. The change is ready to merge. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
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. Comment |
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
DO NOT MERGE until the postiz-app PR gitroomhq/postiz-app#2049 is deployed. The 400 validation error described here does not exist until then.
Updates the
boardsection of the Pinterest public API page: the value is the numeric board id, a board name is rejected with a 400 at scheduling time, and ids come from the channel'sboardstool (integration settings/trigger endpoints) or the Postiz UI. Examples already used numeric ids and are unchanged.Why: in the 7 days ending 2026-09-09, 152 Pinterest pins scheduled via the public API and MCP failed at publish because
boardheld a board name; the app PR rejects those at scheduling, and the docs need to say so.Merge order: postiz-app first, then this PR and the postiz-agent PR.
🤖 Generated with Claude Code
https://claude.ai/code/session_01Cb2afwuxqmE9PKj4qvNE2Q
Summary by CodeRabbit