Add Edit Attachments permission documentation

[Combs7th]

Documents the new Edit Attachments permission introduced in Mattermost v11.8.0.

Adds a Restrict who can edit post attachments subsection under Message management in source/administration-guide/onboard/advanced-permissions.rst, directly after Restrict who can edit messages.

Docs for Mattermost PR mattermost/mattermost#36227

Resolves #9022

Generated with Claude Code

[Combs7th]

@harshilsharma63 - Do the above doc updates look good to you?

[coderabbitai]

📝 Walkthrough

Walkthrough

This pull request adds documentation for the Edit Attachments permission introduced in Mattermost v11.8. The new subsection under Message management explains how this permission controls attachment editing during post edits, its independence from text-edit permissions, default behaviour, administrative configuration steps, and the resulting user-facing error message.

Changes

Edit Attachments Permission Documentation

Layer / File(s)	Summary
Edit Attachments permission subsection source/administration-guide/onboard/advanced-permissions.rst	New subsection documents the Edit Attachments permission, explaining its scope for adding/removing attachments, separation from post text editing, default grant behaviour, step-by-step System Console instructions to disable attachment editing, and the user error message when permission is denied.

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely summarizes the main change: adding documentation for the Edit Attachments permission.
Description check	✅ Passed	The description is directly related to the changeset, explaining the new Edit Attachments permission documentation and referencing the linked issue and Mattermost PR.
Linked Issues check	✅ Passed	The PR fully satisfies the requirements in issue #9022: it adds the 'Restrict who can edit post attachments' section to the correct file with complete content covering all specifications.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to the objective of documenting the Edit Attachments permission; no out-of-scope modifications are present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/issue-9022-20260612-0406

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

🧹 Nitpick comments (1)

source/administration-guide/onboard/advanced-permissions.rst (1)

219-219: ⚡ Quick win

Specify which role panel(s) should be modified for clarity and consistency.

Line 219 uses "In the applicable role panel" without naming the specific role panel(s), which is inconsistent with similar configuration examples throughout this document. Every other recipe in this section explicitly names the role panels—for example:

• Line 119: "In the All Members panel"
• Line 193: "In the All Members and Channel Administrators panels"
• Line 203: "In the All Members, Channel Administrators, and Team Administrators panels"

For the scenario described (allowing users to edit post text without changing attachments), clarify which panel(s) should be modified. For example:

• "In the All Members panel, go to Manage Posts." (if applying system-wide to all users)
• Or: "In the appropriate role panel(s) (such as All Members), go to Manage Posts." (if the choice depends on deployment policy)

This would match the established pattern and help novice administrators understand exactly which panels to modify.

🤖 Prompt for 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.

In `@source/administration-guide/onboard/advanced-permissions.rst` at line 219,
Replace the ambiguous phrase "In the applicable role panel, go to **Manage
Posts**." with a specific panel name or clear choice guidance to match the rest
of the document; for example, change it to "In the **All Members** panel, go to
**Manage Posts**." if this setting is system-wide, or to "In the appropriate
role panel(s) (such as **All Members**), go to **Manage Posts**." if it depends
on deployment policy, ensuring consistency with lines that reference **All
Members**, **Channel Administrators**, and **Team Administrators**.

Source: Coding guidelines

🤖 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.

Nitpick comments:
In `@source/administration-guide/onboard/advanced-permissions.rst`:
- Line 219: Replace the ambiguous phrase "In the applicable role panel, go to
**Manage Posts**." with a specific panel name or clear choice guidance to match
the rest of the document; for example, change it to "In the **All Members**
panel, go to **Manage Posts**." if this setting is system-wide, or to "In the
appropriate role panel(s) (such as **All Members**), go to **Manage Posts**." if
it depends on deployment policy, ensuring consistency with lines that reference
**All Members**, **Channel Administrators**, and **Team Administrators**.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 58363387-88ec-485b-9c11-fd632a07a0b2

📥 Commits

Reviewing files that changed from the base of the PR and between eb35cda and 53bc6d1.

📒 Files selected for processing (1)

• source/administration-guide/onboard/advanced-permissions.rst

[github-actions]

Newest code from mattermost has been published to preview environment for Git SHA 53bc6d1

[github-actions]

Newest code from mattermost has been published to preview environment for Git SHA a0ffbb4

[github-actions]

Newest code from mattermost has been published to preview environment for Git SHA bf31b54

[github-actions]

Newest code from mattermost has been published to preview environment for Git SHA dfe7533

[github-actions]

Newest code from mattermost has been published to preview environment for Git SHA 10ec3e9

[github-actions]

Newest code from mattermost has been published to preview environment for Git SHA c27728d

[github-actions]

Newest code from mattermost has been published to preview environment for Git SHA ef0d13a

[Combs7th]

@harshilsharma63 - Please ignore the unintended commits pulled in for Azure Blob storage, and anything in the "Configure" folder. I'll get those removed.

Only relevant update here is to the advanced-permissions.rst page.

[harshilsharma63]

LGTM!

[github-actions]

Newest code from mattermost has been published to preview environment for Git SHA 7aceaf3

[Combs7th]

@wiersgallak - Can you help with the editor review on this one so we can get it across the finish line by Tuesday?