Skip to content

Fix Agents plugin upgrade guide link in v11.7 changelog#8982

Merged
esethna merged 1 commit into
mattermost:masterfrom
rohithchandran-mattermost:patch-2
May 18, 2026
Merged

Fix Agents plugin upgrade guide link in v11.7 changelog#8982
esethna merged 1 commit into
mattermost:masterfrom
rohithchandran-mattermost:patch-2

Conversation

@rohithchandran-mattermost
Copy link
Copy Markdown
Contributor

Updated the Agents plugin upgrade guide link referenced in the v11.7 release notes.

Updated the Agents plugin upgrade guide link referenced in the v11.7 release notes.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 18, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request corrects a malformed hyperlink in the Mattermost v11 changelog documentation. Specifically, the URL for the Agents plugin upgrade guide from v1.x to v2.0.0 is fixed by removing an erroneous ".md" file extension appended to the hyperlink.

Changes

v11.7 Changelog Documentation Link Fix

Layer / File(s) Summary
Agents plugin upgrade hyperlink correction
source/product-overview/mattermost-v11-changelog.md
The URL for the Agents plugin v1.x to v2.0.0 upgrade documentation is corrected by removing an extraneous ".md" file extension.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • mattermost/docs#8975: Updates hyperlink URLs in the same v11.7 changelog section to correct and insert markdown links.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: fixing a hyperlink for the Agents plugin upgrade guide in the v11.7 changelog.
Description check ✅ Passed The description is directly related to the changeset, describing the update to the Agents plugin upgrade guide link in the v11.7 release notes.
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

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.

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/mattermost-v11-changelog.md`:
- Line 24: The markdown link to the Agents upgrade guide contains a hidden
Unicode character at the end of the URL
("https://github.com/mattermost/mattermost-plugin-agents/blob/master/docs/upgrading_to_2.0.md‎");
edit the text in source/product-overview/mattermost-v11-changelog.md to remove
the invisible trailing character so the URL ends exactly with ".md", save the
change, and verify the link resolves (or run the repo link checker) to ensure
the malformed character was removed.
🪄 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: 3b6be57f-8b2c-493b-8421-568ebc854807

📥 Commits

Reviewing files that changed from the base of the PR and between 6140523 and 4a64694.

📒 Files selected for processing (1)
  • source/product-overview/mattermost-v11-changelog.md

**Breaking Changes**
- FIPS builds require a minimum of 14 characters for passwords, atmos/camo proxy configuration, and shared channel secrets. Shorter passwords for existing users will no longer be valid and require a password reset. Non-FIPS builds are unaffected.
- v11.7 includes Agents plugin v2. Please see [this guide](https://github.com/mattermost/mattermost-plugin-agents/blob/master/docs/upgrading_to_2.0.md‎.md) on how to upgrade the Mattermost Agents plugin from a v1.x release to v2.0.0. It covers the supported version path, the migrations that run automatically on first start of v2.0.0, the breaking changes and default-behavior flips that admins should know about before the upgrade window, and the verification steps to confirm the upgrade succeeded.
- v11.7 includes Agents plugin v2. Please see [this guide](https://github.com/mattermost/mattermost-plugin-agents/blob/master/docs/upgrading_to_2.0.md‎) on how to upgrade the Mattermost Agents plugin from a v1.x release to v2.0.0. It covers the supported version path, the migrations that run automatically on first start of v2.0.0, the breaking changes and default-behavior flips that admins should know about before the upgrade window, and the verification steps to confirm the upgrade succeeded.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Remove the hidden Unicode character from the upgrade guide URL.

The link includes an invisible trailing character after .md, which can make the URL malformed and fail to resolve.

🔧 Minimal fix
- - v11.7 includes Agents plugin v2. Please see [this guide](https://github.com/mattermost/mattermost-plugin-agents/blob/master/docs/upgrading_to_2.0.md‎) on how to upgrade the Mattermost Agents plugin from a v1.x release to v2.0.0. It covers the supported version path, the migrations that run automatically on first start of v2.0.0, the breaking changes and default-behavior flips that admins should know about before the upgrade window, and the verification steps to confirm the upgrade succeeded.  
+ - v11.7 includes Agents plugin v2. Please see [this guide](https://github.com/mattermost/mattermost-plugin-agents/blob/master/docs/upgrading_to_2.0.md) on how to upgrade the Mattermost Agents plugin from a v1.x release to v2.0.0. It covers the supported version path, the migrations that run automatically on first start of v2.0.0, the breaking changes and default-behavior flips that admins should know about before the upgrade window, and the verification steps to confirm the upgrade succeeded.  

As per coding guidelines "When a docs link is broken, brittle, or non-standard, explain the issue briefly and provide the smallest correct replacement" and "Flag and fix broken links, incorrect relative paths, outdated anchors, malformed URLs, or link text that hides the destination".

🤖 Prompt for 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.

In `@source/product-overview/mattermost-v11-changelog.md` at line 24, The markdown
link to the Agents upgrade guide contains a hidden Unicode character at the end
of the URL
("https://github.com/mattermost/mattermost-plugin-agents/blob/master/docs/upgrading_to_2.0.md‎");
edit the text in source/product-overview/mattermost-v11-changelog.md to remove
the invisible trailing character so the URL ends exactly with ".md", save the
change, and verify the link resolves (or run the repo link checker) to ensure
the malformed character was removed.

@esethna esethna merged commit cf048c8 into mattermost:master May 18, 2026
7 checks passed
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