Skip to content

docs: fix Accordion route spelling#571

Merged
vishxrad merged 2 commits into
thesysdev:mainfrom
Lucas-FManager:codex/fix-accordion-typo
May 26, 2026
Merged

docs: fix Accordion route spelling#571
vishxrad merged 2 commits into
thesysdev:mainfrom
Lucas-FManager:codex/fix-accordion-typo

Conversation

@Lucas-FManager
Copy link
Copy Markdown
Contributor

@Lucas-FManager Lucas-FManager commented May 24, 2026

Problem

The docs navigation and route naming use the misspelling accordian, while the component and user-facing concept are Accordion.

Root Cause

The typo exists in both docs navigation configs and in the block route folder name. Because navigation IDs and links reuse that misspelled value, users see Accordian in the sidebar and the generated route also carries the typo.

What Changed

  • Updated the navigation ID, label, and href to use accordion / Accordion.
  • Moved the docs block page from /components/blocks/accordian to /components/blocks/accordion.
  • Added a redirect from the old misspelled route so existing links do not break.

Why This Approach

This fixes the visible typo and the route naming, while preserving backwards compatibility for anyone who already has the old URL bookmarked or linked.

Validation

  • git diff --check
  • rg "Accordian|accordian" docs/app docs/shared -g '!node_modules'

Risk

Low. This is a docs route/navigation correction. The compatibility redirect keeps the old path working.

Closes #553

Lucas-FManager and others added 2 commits May 25, 2026 01:09
Replaces the 307 stub page with a config-level permanent redirect,
and adds the design-system path that was missed in thesysdev#571 but caught
in thesysdev#561.

Co-authored-by: zqleslie <17967998@qq.com>
Copy link
Copy Markdown
Contributor

@vishxrad vishxrad left a comment

Choose a reason for hiding this comment

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

Pushed 9c6295d on top of this branch, moved the redirect from the stub accordian/page.tsx to a config-level redirect in next.config.mjs, switched it to permanent: true (308 instead of 307, better for SEO/canonical URLs), and added the /docs/design-system/blocks/accordian path as well.

The redirect approach is lifted from @zqleslie's #561 (credited as co-author on the commit). With this and @Lucas-FManager's complete nav fix, both #553 paths are covered. Closing #561 in favor of this PR.

@vishxrad vishxrad merged commit 35061c4 into thesysdev:main May 26, 2026
1 check 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.

Fix Accordion typo in docs navigation and route naming

2 participants