Skip to content

Added DM and GM restriction note for quarntine message feature - #9127

Merged
harshilsharma63 merged 3 commits into
masterfrom
quarntine_post_dm_gm_doc
Aug 11, 2026
Merged

Added DM and GM restriction note for quarntine message feature#9127
harshilsharma63 merged 3 commits into
masterfrom
quarntine_post_dm_gm_doc

Conversation

@harshilsharma63

Copy link
Copy Markdown
Member

Summary

Docs PR for complimenting mattermost/mattermost#37841

Ticket Link

Fixes https://mattermost.atlassian.net/browse/MM-70099

@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e3a23021-e4cd-4c16-954c-bd3a88fbfb48

📥 Commits

Reviewing files that changed from the base of the PR and between bb693ea and 78c81f0.

📒 Files selected for processing (1)
  • source/end-user-guide/collaborate/flag-messages.rst
🚧 Files skipped from review as they are similar to previous changes (1)
  • source/end-user-guide/collaborate/flag-messages.rst

📝 Walkthrough

Walkthrough

The flag messages guide states that message quarantine is available in public and private channels, but not in direct or group messages.

Changes

Message quarantine documentation

Layer / File(s) Summary
Document supported channel types
source/end-user-guide/collaborate/flag-messages.rst
The guide identifies public and private channels as supported. It excludes direct and group messages.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the added documentation about direct and group message restrictions for message quarantine.
Description check ✅ Passed The description identifies this as documentation for the quarantine message restrictions and links it to the related pull request and ticket.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch quarntine_post_dm_gm_doc

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the end-user documentation for the Quarantine for Review feature to clarify where the action is available, aligning the docs with the related Mattermost product change.

Changes:

  • Added a note stating that quarantining is available only for public/private channel messages (not DMs or GMs).

Comment thread source/end-user-guide/collaborate/flag-messages.rst Outdated
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actions Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

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

@harshilsharma63
harshilsharma63 requested review from Combs7th and marianunez and removed request for Combs7th August 6, 2026 05:32
@Combs7th
Combs7th self-requested a review August 7, 2026 01:12
@Combs7th

Combs7th commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

@harshilsharma63 - Thank you for the doc update! Before merge, can you please add the corresponding server release milestone if applicable? Thanks!

@harshilsharma63 harshilsharma63 added this to the v11.10.0 milestone Aug 10, 2026
@harshilsharma63

Copy link
Copy Markdown
Member Author

@Combs7th done. Added to the v11.11 milestone.

@amyblais amyblais modified the milestones: v11.10.0, v11.11.0 Aug 10, 2026
@harshilsharma63
harshilsharma63 merged commit ac56ddd into master Aug 11, 2026
5 checks passed
@harshilsharma63
harshilsharma63 deleted the quarntine_post_dm_gm_doc branch August 11, 2026 05:50
@github-actions

Copy link
Copy Markdown
Contributor

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants