Skip to content

docs: fix broken links in custom plugin guides - #1156

Merged
dengliming merged 1 commit into
apache:mainfrom
kakala778:docs/fix-custom-plugin-links
Oct 12, 2026
Merged

dengliming merged 1 commit into
apache:mainfrom
kakala778:docs/fix-custom-plugin-links

Conversation

@kakala778

Copy link
Copy Markdown
Contributor

Summary

The English and Chinese custom plugin guides link to WASM design pages under /docs/next/, which return 404. The current docs are served under /docs/ (lastVersion: "current", labeled 2.7.1). The English guide also has a section link containing spaces that does not match the rendered heading ID.

  • Point both WASM design links to the current documentation routes, preserving their locales.
  • Correct the English section anchor to #single-responsibility-plugin-in-multiple-languages.
  • Leave the existing Chinese section anchor unchanged.

Validation

  • Both old WASM URLs returned HTTP 404; both replacement URLs returned HTTP 200 with the expected English/Chinese page titles and version 2.7.1.

  • Confirmed the English section ID in the live rendered page.

  • git diff --check passed.

  • The CI-equivalent full repository Markdown lint passed using markdownlint-cli@0.25.0 and .github/.markdownlint.yml.

  • NODE_OPTIONS=--max-old-space-size=6144 npm run build passed for both English and Chinese. Inspected the generated guides to confirm the replacement links and anchors.

  • The initial default-memory build failed with JavaScript heap exhaustion (exit 134); the retry above exited 0. The build reports existing warnings in unchanged historical docs and other pages; this PR does not address those warnings.

Related to #1138. This PR addresses only the custom plugin guide links.

AI assistance: ChatGPT Codex helped check routes, apply the three specified link replacements, and run validation.

Related to apache#1138

Generated-by: ChatGPT Codex
@dengliming
dengliming merged commit 753860b into apache:main Oct 12, 2026
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