Skip to content

Mobile Feat: AI rewrite#8872

Merged
esethna merged 5 commits intov11.6-documentationfrom
claude/issue-8871-20260409-2145
Apr 13, 2026
Merged

Mobile Feat: AI rewrite#8872
esethna merged 5 commits intov11.6-documentationfrom
claude/issue-8871-20260409-2145

Conversation

@Combs7th
Copy link
Copy Markdown
Contributor

@Combs7th Combs7th commented Apr 9, 2026

Expands the Mobile tab under "Rewrite messages with AI" in send-messages.rst with a full step-by-step workflow mirroring the existing Web/Desktop section.

Closes #8871

Generated with Claude Code

sadohert and others added 2 commits April 9, 2026 16:19
* Improve PostgreSQL database setup page discoverability in search

Searching for "postgres" or "postgres setup" in the docs didn't surface the
database preparation instructions on the preparations page. Renamed the section
heading, added search metadata, and included PostgreSQL in the intro paragraph
to improve Sphinx search ranking.

Also fixes a sphinx-autobuild rebuild loop caused by generated agent docs
being rewritten during each build cycle.

Closes #8864

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

* Address CodeRabbit review feedback

- Use double quotes for --ignore glob in Makefile for Windows cmd.exe
  compatibility
- Add legacy anchor label to preserve existing deep links to
  #database-preparation after heading rename

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
Replace the one-liner Mobile tab under 'Rewrite messages with AI' with
a full step-by-step workflow mirroring the existing Web/Desktop section,
covering: opening rewrite from the composer, selecting an agent, choosing
rewrite options (including custom prompt), the processing state, and
reviewing/sending the result.

Closes #8871

Co-authored-by: Combs7th <[email protected]>
@Combs7th Combs7th changed the base branch from master to v11.6-documentation April 9, 2026 21:57
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 9, 2026

📝 Walkthrough

Walkthrough

The Mobile section of the "Rewrite messages with AI" documentation was expanded from a single-sentence statement into detailed step-by-step instructions covering the complete workflow for using AI message rewrites on mobile devices, including agent selection and all available rewrite options.

Changes

Cohort / File(s) Summary
Mobile AI Rewrite Documentation
source/end-user-guide/collaborate/send-messages.rst
Expanded the Mobile subsection under "Rewrite messages with AI" to include detailed procedural steps: tapping the Rewrite option, selecting an AI agent, choosing from predefined rewrite actions (Improve writing, Fix spelling, Shorten, Elaborate, Simplify, Summarize), using a custom prompt, reviewing AI-generated suggestions with Regenerate/Discard controls, and sending the message.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'Mobile Feat: AI rewrite' directly summarizes the main change—expanding the Mobile section's AI rewrite workflow documentation.
Description check ✅ Passed The description accurately relates to the changeset, explaining the expansion of Mobile tab content with step-by-step workflow and referencing the linked issue.
Linked Issues check ✅ Passed The PR successfully expands the Mobile subsection with step-by-step instructions covering all required objectives: opening AI rewrite, selecting agents, choosing rewrite actions, processing state, and reviewing results before sending.
Out of Scope Changes check ✅ Passed All changes are scoped to the required file (send-messages.rst) and directly address the Mobile AI rewrite workflow documentation as specified in issue #8871.
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-8871-20260409-2145

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

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

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

🧹 Nitpick comments (2)
source/end-user-guide/collaborate/send-messages.rst (2)

70-70: Consider using "Choose from" to match Web/Desktop.

The Web/Desktop section uses "Choose from available rewrite options" (line 52), while Mobile uses "Select from." For consistency and to maintain parallel structure, consider aligning the verb choice.

📝 Proposed fix for verb consistency
-  4. Select from available rewrite options:
+  4. Choose from available rewrite options:
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@source/end-user-guide/collaborate/send-messages.rst` at line 70, Replace the
verb "Select from" with "Choose from" in the sentence "Select from available
rewrite options" so it matches the Web/Desktop wording "Choose from available
rewrite options" (ensure the phrase to edit is the one that currently reads
"Select from available rewrite options" in send-messages.rst).

69-69: Inconsistent terminology: "agent" vs. "bot".

The Web/Desktop section (line 51) uses "Select a bot from the list of available AI agents," but the Mobile section uses "Select an agent." For consistency across platforms, consider using the same phrasing unless there's a genuine mobile-specific UI difference.

📝 Proposed fix for terminology consistency
-  3. Select an agent from the list of available AI agents.
+  3. Select a bot from the list of available AI agents.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@source/end-user-guide/collaborate/send-messages.rst` at line 69, The
documentation uses inconsistent terminology between the Web/Desktop and Mobile
sections—replace the Web/Desktop phrasing "Select a bot from the list of
available AI agents" so it matches the Mobile phrasing by using "agent" instead
(i.e., update the sentence to "Select an agent from the list of available AI
agents"); locate this text in the Web/Desktop section of
source/end-user-guide/collaborate/send-messages.rst and make the wording
identical across both sections.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@source/end-user-guide/collaborate/send-messages.rst`:
- Line 70: Replace the verb "Select from" with "Choose from" in the sentence
"Select from available rewrite options" so it matches the Web/Desktop wording
"Choose from available rewrite options" (ensure the phrase to edit is the one
that currently reads "Select from available rewrite options" in
send-messages.rst).
- Line 69: The documentation uses inconsistent terminology between the
Web/Desktop and Mobile sections—replace the Web/Desktop phrasing "Select a bot
from the list of available AI agents" so it matches the Mobile phrasing by using
"agent" instead (i.e., update the sentence to "Select an agent from the list of
available AI agents"); locate this text in the Web/Desktop section of
source/end-user-guide/collaborate/send-messages.rst and make the wording
identical across both sections.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: fc59ef3c-277c-4436-a977-f450c74753c2

📥 Commits

Reviewing files that changed from the base of the PR and between 73e24f9 and ff13858.

📒 Files selected for processing (1)
  • source/end-user-guide/collaborate/send-messages.rst

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 9, 2026

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

@Combs7th Combs7th requested a review from fmartingr April 9, 2026 22:02
@Combs7th Combs7th added 1: Dev Review Requires review by a core commiter 2: Editor Review Requires review by an editor Guidance labels Apr 9, 2026
@Combs7th Combs7th added this to the v11.6.0 milestone Apr 9, 2026
@Combs7th
Copy link
Copy Markdown
Contributor Author

Combs7th commented Apr 9, 2026

@fmartingr - Do the above additions look accurate to you?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 9, 2026

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

@fmartingr
Copy link
Copy Markdown
Contributor

@fmartingr - Do the above additions look accurate to you?

@Combs7th Yes! Looking good, thanks for taking care of this.

Copy link
Copy Markdown
Contributor

@esethna esethna left a comment

Choose a reason for hiding this comment

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

@Combs7th curious why this change is needed at all? I might be missing something but aren't we carbon copying the desktop instructions right above it and putting them in a separate tab? Why not just remove the tabs all together and just state that it works the same on desktop and mobile?

@esethna
Copy link
Copy Markdown
Contributor

esethna commented Apr 13, 2026

Updated it. @Combs7th please check it out as this is a good example of where we can simplify

@esethna esethna removed 1: Dev Review Requires review by a core commiter 2: Editor Review Requires review by an editor labels Apr 13, 2026
@esethna esethna merged commit 6b35d15 into v11.6-documentation Apr 13, 2026
6 checks passed
@esethna esethna deleted the claude/issue-8871-20260409-2145 branch April 13, 2026 22:23
@github-actions
Copy link
Copy Markdown
Contributor

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

amyblais added a commit that referenced this pull request Apr 16, 2026
* Update conf.py

* [MM-65701] Document 7zip file blocking in file content search (#8836)

Adds notes to end-user search docs and admin config docs clarifying
that 7zip (.7z) files are not supported for file content search and
are skipped for security reasons. Only standard ZIP files are supported
when ZIP file search is enabled.

Ref: mattermost/mattermost#34983

Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>

* Add docs for preferred_username claim support in OpenID Connect (#8838)

Adds Use preferred username setting documentation to each OpenID
Connect provider section (GitLab, Google, Entra ID, OpenID Connect
other) in the authentication configuration settings reference, and a
corresponding note in the OpenID Connect SSO setup guide.

Closes #8835

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Combs7th <[email protected]>

* docs(auth): Clarify Google SSO username/email sync behavior (#8832)

* Desktop v5.13.5 docs (#8828)

* Update mattermost-desktop-releases.md

* Update desktop-app-changelog.md

* Update desktop-app-changelog.md

* Update desktop-app-changelog.md

* docs(auth): clarify Google SSO username/email sync behavior

Username and email address changes made in Google Workspace are not
automatically synced to Mattermost. Adds a note to the existing
user-attribute warning in the Google SSO setup guide to prevent
readers from incorrectly inferring that the IdP is an automatic sync
source. Aligns with mattermost/mattermost#34475.

Co-authored-by: Combs7th <[email protected]>

* Update desktop-app-changelog.md

* Update desktop-app-changelog.md

* Update mattermost-desktop-releases.md

---------

Co-authored-by: Amy Blais <[email protected]>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Combs7th <[email protected]>

* Update mattermost-desktop-releases.md

* Update desktop-app-changelog.md

* Add four newly sanitized fields to Support Packet documentation (#8874)

* Improve PostgreSQL database setup discoverability in search (#8865)

* Improve PostgreSQL database setup page discoverability in search

Searching for "postgres" or "postgres setup" in the docs didn't surface the
database preparation instructions on the preparations page. Renamed the section
heading, added search metadata, and included PostgreSQL in the intro paragraph
to improve Sphinx search ranking.

Also fixes a sphinx-autobuild rebuild loop caused by generated agent docs
being rewritten during each build cycle.

Closes #8864

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

* Address CodeRabbit review feedback

- Use double quotes for --ignore glob in Makefile for Windows cmd.exe
  compatibility
- Add legacy anchor label to preserve existing deep links to
  #database-preparation after heading rename

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>

* docs: add four newly sanitized fields to Support Packet documentation

From Mattermost v11.6.0 (mattermost/mattermost#35346), the following
fields are now automatically redacted during Support Packet generation:
- FileSettings.ExportAmazonS3SecretAccessKey
- ElasticsearchSettings.ClientKey
- ServiceSettings.GoogleDeveloperKey
- ServiceSettings.GiphySdkKey

Co-authored-by: Combs7th <[email protected]>

* Update Makefile

* Update preparations.rst

---------

Co-authored-by: Stuart Doherty <[email protected]>
Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Combs7th <[email protected]>

* Add popout window documentation for search results (#8868)

* Add popout window documentation for search results (v11.6)

Documents the new Search RHS popout feature introduced in Mattermost v11.6,
covering: opening search results in a popout window, query preservation,
thread navigation within the popout, back button behavior, and Jump
opening the source channel in the main window.

Co-authored-by: Combs7th <[email protected]>

* Update search-for-messages.rst

* Update search-for-messages.rst

---------

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Combs7th <[email protected]>
Co-authored-by: Eric Sethna <[email protected]>

* Mobile Feat: AI rewrite (#8872)

* Improve PostgreSQL database setup discoverability in search (#8865)

* Improve PostgreSQL database setup page discoverability in search

Searching for "postgres" or "postgres setup" in the docs didn't surface the
database preparation instructions on the preparations page. Renamed the section
heading, added search metadata, and included PostgreSQL in the intro paragraph
to improve Sphinx search ranking.

Also fixes a sphinx-autobuild rebuild loop caused by generated agent docs
being rewritten during each build cycle.

Closes #8864

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

* Address CodeRabbit review feedback

- Use double quotes for --ignore glob in Makefile for Windows cmd.exe
  compatibility
- Add legacy anchor label to preserve existing deep links to
  #database-preparation after heading rename

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>

* docs: expand mobile AI rewrite workflow in send-messages.rst

Replace the one-liner Mobile tab under 'Rewrite messages with AI' with
a full step-by-step workflow mirroring the existing Web/Desktop section,
covering: opening rewrite from the composer, selecting an agent, choosing
rewrite options (including custom prompt), the processing state, and
reviewing/sending the result.

Closes #8871

Co-authored-by: Combs7th <[email protected]>

* Update preparations.rst

* Update Makefile

* Simplify

---------

Co-authored-by: Stuart Doherty <[email protected]>
Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Combs7th <[email protected]>
Co-authored-by: Eric Sethna <[email protected]>

* Rename Content Flagging to Data Spillage Handling (#8870)

* Improve PostgreSQL database setup discoverability in search (#8865)

* Improve PostgreSQL database setup page discoverability in search

Searching for "postgres" or "postgres setup" in the docs didn't surface the
database preparation instructions on the preparations page. Renamed the section
heading, added search metadata, and included PostgreSQL in the intro paragraph
to improve Sphinx search ranking.

Also fixes a sphinx-autobuild rebuild loop caused by generated agent docs
being rewritten during each build cycle.

Closes #8864

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

* Address CodeRabbit review feedback

- Use double quotes for --ignore glob in Makefile for Windows cmd.exe
  compatibility
- Add legacy anchor label to preserve existing deep links to
  #database-preparation after heading rename

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>

* docs: rename Content Flagging to Data Spillage Handling (v11.6.0)

Update user-visible and admin-visible terminology throughout the
Content Flagging docs to reflect the v11.6.0 rename:

- Rename page title and all feature-name references from
  'Content Flagging' to 'Data Spillage Handling'
- Update System Console navigation path to
  'System Console > Site Configuration > Data Spillage Handling'
- Replace 'flag/flagged' with 'quarantine/quarantined' where the
  product UI now uses that wording
- Replace 'content-review bot' with 'Data Spillage Bot' in all
  user-facing notification and moderation queue descriptions
- Update 'Flag message' menu action to 'Quarantine for Review'

Internal RST cross-reference labels and API/config identifiers
left unchanged per scope constraints.

Closes #8869

Co-authored-by: Combs7th <[email protected]>

* Update preparations.rst

* Update Makefile

* Update Data Spillage Handling reasons to new default categories

Co-authored-by: Combs7th <[email protected]>

---------

Co-authored-by: Stuart Doherty <[email protected]>
Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Combs7th <[email protected]>

* Allow System Admins to view and update User AuthData and Username in System Console (#8834)

* Desktop v5.13.5 docs (#8828)

* Update mattermost-desktop-releases.md

* Update desktop-app-changelog.md

* Update desktop-app-changelog.md

* Update desktop-app-changelog.md

* docs: add System Admin ability to update username and AuthData in System Console

Reflects the new capability from mattermost/mattermost#33550 where System
Admins can view and update user AuthData and username directly in the System
Console user detail workflow.

Co-authored-by: Combs7th <[email protected]>

* Update user-management-configuration-settings.rst

* Update desktop-app-changelog.md

* Update mattermost-desktop-releases.md

---------

Co-authored-by: Amy Blais <[email protected]>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Combs7th <[email protected]>

* Update desktop-app-changelog.md

* Update mattermost-desktop-releases.md

* v11.6 Changelog (#8821)

* Prepare v11.6 feature release docs (April 16, 2026)

Co-authored-by: Amy Blais <[email protected]>

* Update important-upgrade-notes.rst

* Update deprecated-features.rst

* Update release-policy.md

* Update mattermost-v11-changelog.md

* Update software-hardware-requirements.rst

* Update open-source-components.rst

* Update mattermost-v11-changelog.md

* Update ui-ada-changelog.rst

* Update ui-ada-changelog.rst

* Update mattermost-v11-changelog.md

* Update mattermost-desktop-releases.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update source/product-overview/mattermost-v11-changelog.md

Co-authored-by: Doug Lauder <[email protected]>

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update ui-ada-changelog.rst

* Update mattermost-v11-changelog.md

* Update important-upgrade-notes.rst

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update ui-ada-changelog.rst

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update mattermost-v11-changelog.md

* Update ui-ada-changelog.rst

* Update mattermost-v11-changelog.md

* Update ui-ada-changelog.rst

---------

Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Amy Blais <[email protected]>
Co-authored-by: Doug Lauder <[email protected]>

---------

Co-authored-by: Christopher Poile <[email protected]>
Co-authored-by: Claude Opus 4.6 (1M context) <[email protected]>
Co-authored-by: Combs7th <[email protected]>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Combs7th <[email protected]>
Co-authored-by: Stuart Doherty <[email protected]>
Co-authored-by: Eric Sethna <[email protected]>
Co-authored-by: Amy Blais <[email protected]>
Co-authored-by: Doug Lauder <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mobile Feat: AI rewrite

4 participants