-
Notifications
You must be signed in to change notification settings - Fork 616
Conversation
Co-authored-by: Amy Blais <amyblais@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Prepares the Mattermost documentation site (docs.mattermost.com) for the Mobile v2.42 release by adding a new changelog section and updating the mobile releases matrix.
Changes:
- Added a new 2.42.0 section to the Mobile Apps Changelog (with compatibility + known issues).
- Added v2.42 to the "Latest releases" table and removed it from the "Future releases" table in the Mobile Releases page.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| source/product-overview/mobile-app-changelog.md | Adds the v2.42.0 release section (compatibility, placeholders, known issues). |
| source/product-overview/mattermost-mobile-releases.md | Updates the releases tables to include v2.42 as "Latest" and removes it from "Future". |
Newest code from mattermost has been published to preview environment for Git SHA 184aac8
📝 WalkthroughWalkthroughTwo documentation files are updated for the Mattermost mobile v2.42 release. The mobile releases table moves v2.42 from "Future releases" to "Latest releases", adds v2.48 as a new future entry, and adjusts v2.43. The mobile app changelog gains a new v2.42.0 section at the top with release date, compatibility requirements, improvements, bug fixes, and known issues. ChangesMobile v2.42 Release Documentation
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
🚥 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)
Warning Review ran into problems🔥 ProblemsStopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a 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 |
Newest code from mattermost has been published to preview environment for Git SHA db1c7cd
Newest code from mattermost has been published to preview environment for Git SHA 2a32b2b
Newest code from mattermost has been published to preview environment for Git SHA 362743a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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/product-overview/mobile-app-changelog.md`:
- Line 26: The bullet point describing Offline Persistence Timer support for
Mobile Ephemeral Mode contains awkward phrasing that reduces clarity. Rewrite
the bullet point to use clear, plain release-note language. Specifically,
replace the awkward phrase "honoring organizations to purge the mattermost app
data when needed" with clearer, more direct language that explains what the
feature actually does and how it benefits users. The rewritten text should be
easy for end users to understand without requiring multiple reads.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
i️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 8bc546b9-6502-4d81-91c4-031f9453194f
📒 Files selected for processing (1)
source/product-overview/mobile-app-changelog.md
Newest code from mattermost has been published to preview environment for Git SHA 1cea100
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To clarify, this is what we're advertising (to encourage updates), but I think we still "support" v9.11.0 as per mattermost/mattermost-mobile#9825? Do we want to distinguish that?
Closes #9051
Generated with Claude Code