Skip to content

Test notice 11.6#476

Merged
amyblais merged 1 commit into
masterfrom
yasserfaraazkhan-patch-1
Apr 30, 2026
Merged

Test notice 11.6#476
amyblais merged 1 commit into
masterfrom
yasserfaraazkhan-patch-1

Conversation

@yasserfaraazkhan
Copy link
Copy Markdown
Contributor

Summary

[todo]

Jira ticket

  • MM-Txxxxx

Screenshots of the modals or screens in all target clients (required)

  • [] In Web
  • [] In Desktop app
  • [] In Mobile app, both Android and iOS

Test environment (required)

  • [] Server versions (ex. from v7.8 to v7.9)
  • [] Server configuration
  • [] Desktop app version
  • [] Mobile app version

Test steps and expectation (required)

[todo]

@yasserfaraazkhan yasserfaraazkhan changed the title Update notices.json Test notice 11.6 Apr 30, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 30, 2026

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: b6d066a9-95fd-4ea1-9597-f91aba0dc8e8

📥 Commits

Reviewing files that changed from the base of the PR and between fc25c6f and 9c608aa.

📒 Files selected for processing (1)
  • notices.json

📝 Walkthrough

Walkthrough

The notices.json file was updated to target newer software releases. The server upgrade notice was bumped from v11.5 to v11.6, and the patch upgrade notice was extended to cover additional patch versions (up to 10.11.14). Display dates and localized content were updated accordingly for each notice.

Changes

Cohort / File(s) Summary
Notice Definitions
notices.json
Updated two notice entries: server upgrade notice changed from server_upgrade_v11.5 to server_upgrade_v11.6 with version cutoff and display date updates; patch upgrade notice extended version range from <=10.11.12 to <=10.11.14 and updated security severity descriptions in English localization.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • Test notice 11.5 #473: Related notice version-bump PR that continues the same pattern of advancing server upgrade and patch notice entries to subsequent release versions with corresponding metadata updates.
🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Test notice 11.6' is vague and generic, using non-descriptive language that doesn't clearly convey the main purpose of the changeset. Replace with a more descriptive title that clearly indicates the primary change, such as 'Update notices for Mattermost 11.6 and 10.11.15 releases' or 'Update server upgrade and patch upgrade notices'.
Description check ❓ Inconclusive The description contains multiple [todo] placeholders and unchecked items, making it incomplete and vague about the changeset's actual purpose and testing requirements. Complete all [todo] sections with actual details, provide concrete test steps, specify target server versions, and verify all required checklists are properly filled out.
✅ Passed checks (3 passed)
Check name Status Explanation
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.

✏️ 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 yasserfaraazkhan-patch-1

Review rate limit: 4/5 reviews remaining, refill in 12 minutes.

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

@amyblais amyblais merged commit 2957ae2 into master Apr 30, 2026
2 checks passed
@amyblais amyblais deleted the yasserfaraazkhan-patch-1 branch April 30, 2026 09:58
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.

2 participants