Skip to content

fix: update broken MeshSync documentation link in solutions page - #8296

Merged
hiyach28 merged 1 commit into
layer5io:masterfrom
tejaswiverma121-byte:fix/solutions-meshsync-docs-link
Nov 4, 2026
Merged

hiyach28 merged 1 commit into
layer5io:masterfrom
tejaswiverma121-byte:fix/solutions-meshsync-docs-link

Conversation

@tejaswiverma121-byte

@tejaswiverma121-byte tejaswiverma121-byte commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Description

This PR fixes #8294
On the Solutions page (/cloud-native-management/solutions), under the "Minimize complexity and automate workflows" section, clicking the MeshSync logo under "Layer5 Products and Tools Deployed" was pointing to an obsolete versioned URL (https://docs.meshery.io/v0.6/concepts/architecture/meshsync), resulting in a 404 Page Not Found error.

This PR updates the link to the active MeshSync documentation page: https://docs.meshery.io/concepts/architecture/meshsync

Screen.Recording.2026-11-03.at.5.37.15.PM.mov

Notes for Reviewers
Checked src/sections/Solutions/ and the broader repository to confirm there are no other outdated docs.meshery.io/v0.6/ links.
Tested locally on development server (localhost:8000/cloud-native-management/solutions) and verified the icon now redirects to the live MeshSync architecture doc with 200 OK.

Signed commits

  • Yes, I signed my commits.

Summary by CodeRabbit

Summary by CodeRabbit

  • Documentation
    • Updated the MeshSync documentation link to the current architecture guide.

@coderabbitai

coderabbitai Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review.

Walkthrough

The MeshSync product link on the Solutions page now points to the current architecture documentation URL instead of the versioned v0.6 URL.

Changes: Solutions section

Layer / File(s) Summary
MeshSync documentation link
src/sections/Solutions/minimize-complexity-section.js
The MeshSync link now uses the unversioned architecture documentation URL.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix · Severity of issue fixed: Low

Possibly related issues

Possibly related PRs

Suggested reviewers: mahakgupta108

Merge Risk: Minimal · up to c1ced

The MeshSync logo now targets the requested documentation page. No merge-blocking issue is identified for this localized link update.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title fix: update broken MeshSync documentation link in solutions page matches the single change in the changeset. The diff changes one line in src/sections/Solutions/minimize-complexity-section.j…
Linked Issues check Passed Issue #8294 is the only directly linked issue. Its single coding requirement is that the MeshSync logo under "Layer5 Products and Tools Deployed" in src/sections/Solutions/minimize-complexity-section.…
Out of Scope Changes check Passed The whole-PR diff contains one changed line: the href of the MeshSync anchor in src/sections/Solutions/minimize-complexity-section.js. The change removes the /v0.6 path segment from that URL. This is …
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Full details: Title check

Explanation

The title fix: update broken MeshSync documentation link in solutions page matches the single change in the changeset. The diff changes one line in src/sections/Solutions/minimize-complexity-section.js, replacing the versioned v0.6 MeshSync docs URL with the current architecture documentation URL. The title names the change type, the affected content, and the page, so a teammate scanning history understands the primary change.

Full details: Linked Issues check

Explanation

Issue #8294 is the only directly linked issue. Its single coding requirement is that the MeshSync logo under "Layer5 Products and Tools Deployed" in src/sections/Solutions/minimize-complexity-section.js link to https://docs.meshery.io/concepts/architecture/meshsync instead of the obsolete https://docs.meshery.io/v0.6/concepts/architecture/meshsync URL. The whole-PR diff (merge base to head) changes only line 239 of that file. The MeshSync anchor now has href="https://docs.meshery.io/concepts/architecture/meshsync", which matches the expected URL. The other icon links in the ProductIcons block are unchanged. The issue names no automated test requirement, and the remaining issue content is non-coding or background. The live 200 response of the external docs page was not verified in this review. That check is outside the code evidence and does not affect the code requirement.

Full details: Out of Scope Changes check

Explanation

The whole-PR diff contains one changed line: the href of the MeshSync anchor in src/sections/Solutions/minimize-complexity-section.js. The change removes the /v0.6 path segment from that URL. This is the exact change requested by #8294. No other files changed, and no unrelated refactoring, dependency, or documentation changes appear in the diff. The change is within scope.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 1 files.

Finishing Touches
Generate unit tests (beta)
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Warning

Some tools did not complete. Review the errors below.

ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install timed out. The project may have too many dependencies for the sandbox.

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

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.
Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f3fd5077-ca2f-4a61-96b9-1c9ac75ce4a2
Commits

Reviewing files that changed from the base of the PR and between acac51c and c1cedb4.

Files selected for processing: 1
  • src/sections/Solutions/minimize-complexity-section.js

Included review availability: This review used your included allowance. Your plan provides up to 4 included reviews per hour; 3 remain after this review.

@miacycle miacycle left a comment

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.

Thanks for catching this broken link, and for verifying the updated URL returns 200 and checking for other v0.6 links.

One request to keep the change easy to review: most of this diff is formatting reflow in src/sections/Solutions/minimize-complexity-section.js (indentation and line wrapping), with the actual fix being the MeshSync href change from the v0.6 URL to https://docs.meshery.io/concepts/architecture/meshsync. Could you revert the formatting-only hunks so the diff shows just the link change? That will make the fix clearer to review and reduce merge conflict risk.

Also, the description says "fixes #" without an issue number. If there is a linked issue, adding its number would help track this.

Otherwise the link update itself looks right.

@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Preview removed because the pull request was closed.

Signed-off-by: tejaswiverma <tejaswiverma121@gmail.com>
@tejaswiverma121-byte
tejaswiverma121-byte force-pushed the fix/solutions-meshsync-docs-link branch from acac51c to c1cedb4 Compare November 3, 2026 13:27
@tejaswiverma121-byte

Copy link
Copy Markdown
Contributor Author

Hi @miacycle, thanks for the review! I've reverted the formatting hunks and amended the commit so the diff now cleanly shows only the 1-line link change (+1 -1). I have also updated the description to link issue #8294.

@Mahakgupta108 Mahakgupta108 left a comment

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.

Minimal changes, lgtm!

@hiyach28 hiyach28 left a comment

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.

Verified: the v0.6 URL returns 404 and the unversioned docs.meshery.io/concepts/architecture/meshsync returns 200. Also confirmed docs.meshery.io/v0.6 appears only in this file, so there are no other versioned links to fix. Diff is the single href line, DCO and build green. LGTM.

@hiyach28
hiyach28 merged commit 5e0005f into layer5io:master Nov 4, 2026
5 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.

[Bug]: Broken MeshSync documentation link (404) on Solutions page

4 participants