Conversation
* 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]>
📝 WalkthroughWalkthroughThe 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
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 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)
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.
🧹 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
📒 Files selected for processing (1)
source/end-user-guide/collaborate/send-messages.rst
|
Newest code from mattermost has been published to preview environment for Git SHA ff13858 |
|
@fmartingr - Do the above additions look accurate to you? |
|
Newest code from mattermost has been published to preview environment for Git SHA 7e578eb |
@Combs7th Yes! Looking good, thanks for taking care of this. |
esethna
left a comment
There was a problem hiding this comment.
@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?
|
Updated it. @Combs7th please check it out as this is a good example of where we can simplify |
|
Newest code from mattermost has been published to preview environment for Git SHA 44f1563 |
* 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]>
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