V11.8 Docs#8974
Conversation
📝 WalkthroughWalkthroughThis PR updates documentation: it clarifies the Report a Problem Default behavior and fixes its env var name, adds mobile-app log troubleshooting with the ChangesDocumentation Updates
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 inconclusive)
✅ Passed checks (3 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.
Pull request overview
Updates the Sphinx documentation configuration to reflect the Mattermost v11.8 documentation version.
Changes:
- Bumps the commented
versionvalue from11.7to11.8insource/conf.py. - Bumps the commented
releasevalue from11.7to11.8insource/conf.py.
|
Newest code from mattermost has been published to preview environment for Git SHA 53db6b0 |
* MM-67771: Update Report a Problem docs to reflect email flow Update the "Default link" description to reflect the new behavior: licensed servers now open an email to Mattermost support, and free edition directs to the troubleshooting forums. Also fix typo in REPORTAPROBLMEMAIL env var name. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> * Update site-configuration-settings.rst --------- Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Co-authored-by: Eric Sethna <14333569+esethna@users.noreply.github.com> Co-authored-by: Amy Blais <29708087+amyblais@users.noreply.github.com>
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 `@source/conf.py`:
- Around line 529-531: Uncomment the Sphinx version settings by removing the
comment markers so the variables version and release are assigned (i.e., change
the commented lines for version = '11.8' and release = '11.8' to active
assignments) in source/conf.py so Sphinx can substitute |version| and |release|
in the docs.
🪄 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: a570f205-7bcb-411a-9a9b-7f041d5976a9
📒 Files selected for processing (2)
source/administration-guide/configure/site-configuration-settings.rstsource/conf.py
|
Newest code from mattermost has been published to preview environment for Git SHA 328fc1e |
|
Newest code from mattermost has been published to preview environment for Git SHA bec05d0 |
* docs: add /mobile-logs slash command Document the new /mobile-logs command introduced in mattermost/mattermost#35658, which toggles the attach_app_logs preference to enable attaching mobile app logs as a file in the Mattermost mobile client (v2.38+) for debugging. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * docs: refine /mobile-logs admonition and wording - Elevate security/audit note to .. important:: - Use bold instead of italics for the error message - Clarify the attachment applies to any message the user sends Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com> * address reviews * docs: refine mobile troubleshooting formatting * improve description --------- Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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 `@source/deployment-guide/mobile/mobile-troubleshooting.rst`:
- Line 168: Standardize the session menu label wording on this page by choosing
one of the two variants and updating all occurrences to match; replace either
"View and Log Out of Active Sessions" or "View and Logout of Active Sessions" so
the label is consistent (ensure both the line around the DeviceId/Sessions
explanation and the earlier reference use the same string), and follow
Mattermost's standard UI label casing/spacing when making the change.
🪄 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: c5f9c96c-384e-42aa-9b67-57184e0c79bf
📒 Files selected for processing (2)
source/deployment-guide/mobile/mobile-troubleshooting.rstsource/integrations-guide/built-in-slash-commands.rst
✅ Files skipped from review due to trivial changes (1)
- source/integrations-guide/built-in-slash-commands.rst
|
Newest code from mattermost has been published to preview environment for Git SHA 89c8271 |
|
Newest code from mattermost has been published to preview environment for Git SHA 91d11f7 |
|
Newest code from mattermost has been published to preview environment for Git SHA 3ebc52d |
|
Newest code from mattermost has been published to preview environment for Git SHA e9ffec4 |
|
Newest code from mattermost has been published to preview environment for Git SHA aee36a1 |
No description provided.