user-guide: add warning for permissions additiveness#11
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughThis PR adds a clarifying warning to the Permissions section of the experiments user guide. The new callout explains that permissions are additive relative to the base permission and directs users to select a restrictive base permission, then explicitly grant access to teams, teamgroups, or users as required. ChangesPermissions Documentation Update
Estimated Code Review Effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/usage/user-guide/experiments.md`:
- Around line 215-217: The admonition block's delimiters are mismatched: it
opens with ":::warning" but closes with "::::" causing a Docusaurus render
error; update the closing delimiter to use three colons (":::") so the warning
block's opening ":::warning" and closing "::: " match (look for the ":::warning"
/ "::::" pair in the experiments.md content around the permissions note).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 27e24c20-76c0-46ba-ad28-fc33a369c990
📒 Files selected for processing (1)
docs/usage/user-guide/experiments.md
:)
Summary by CodeRabbit