docs: Update Partial Items settings note - #1090
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Team 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 Partial Items API documentation now explains that the API URL and required headers can be configured under Coop Settings. It removes the previous limitation and code-management instructions. ChangesPartial Items API documentation
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This localized documentation update removes outdated Partial Items setup guidance and points users to the current Settings instructions; no actionable merge-blocking risk remains. Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.) ✨ Finishing Touches🧪 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 |
There was a problem hiding this comment.
🟢 Approval recommended
Documentation change is consistent with the current Settings docs (anchor exists) and cleanly removes outdated guidance.
Pull request overview
Updates the Partial Items API documentation to reflect that Partial Items configuration is now available in the Coop UI (via Settings), removing an outdated warning that referenced a now-closed issue.
Changes:
- Removed the “not yet possible to configure from the Coop UI” IMPORTANT note and the link to issue #378.
- Added a direct link to the Settings → Partial Items section where the endpoint URL/headers can be configured.
File summaries
| File | Description |
|---|---|
| docs/api/partial-items.md | Replaces the outdated configuration warning with a link to the Settings documentation for configuring Partial Items. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Context & Requests for Reviewers
Documentation for Partial Items currently states that it's not possible to configure the API details from the UI and points to issue #378 for additional context.
This note is now outdated as PR #709 introduced a section on the UI to configure the Partial Items API, and #378 is now closed.
PR removes the note from the docs and adds a link to the Settings section where the Partial Items setup screenshot is currently shown.
Tests
Docs build successfully with
mdbook build docsCurrent docs:

Updated docs:

Checklist
Only check items that apply to this PR; leave the rest unchecked.
If you changed anything user-facing (i.e. user interface or APIs):
Did you update related docs?: Yes, PR is a documentation change.
If the change is notable (refer to Keep a Changelog conventions):
Did you update CHANGELOG.md?
If you changed
server/models/**/{ContentTypeModel,ActionModel,RuleModel,PolicyModel}.ts:Did you update the corresponding history tables and their triggers?
If you changed
db/src/scripts/**and usedCREATE TABLE,ADD COLUMN, orALTER COLUMN:Are as many columns marked
NOT NULLas possible? If some columns can sometimes be null depending on other columns, are thereCHECKconstraints capturing those relationships, and are these also reflected using unions in the associated Kysely types?If you added a new signal in
server/services/signalsService/signals/**:Did you classify every error case as a permanent error (
SignalPermanentError, no retry) or a normal error (retryable)? Any case where the signal can't determine a score should be aSignalPermanentError.Summary by CodeRabbit