Skip to content

docs: fix underscores#2608

Merged
vinckr merged 2 commits into
masterfrom
una-fix-underscore
Jun 5, 2026
Merged

docs: fix underscores#2608
vinckr merged 2 commits into
masterfrom
una-fix-underscore

Conversation

@unatasha8

@unatasha8 unatasha8 commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

Fixed underscore instances

Summary by CodeRabbit

  • Chores
    • Standardized solution documentation routes to hyphenated names across navigation and sidebars.
    • Updated navigation links for CIAM, B2B IAM, and Agentic IAM to use the new paths.
    • Added redirects so old underscore-style URLs forward to the new hyphenated pages.

@coderabbitai

coderabbitai Bot commented Jun 5, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 1ac8500b-0a37-458f-b20c-5fb4146356b3

📥 Commits

Reviewing files that changed from the base of the PR and between ad054fd and a5889a0.

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

📝 Walkthrough

Walkthrough

Path references for solution documentation entries (CIAM, B2B IAM, Agentic IAM) are standardized from underscore-based naming to hyphenated naming across sidebar and navbar configurations, and corresponding Vercel redirects were added.

Changes

Solution routes standardization

Layer / File(s) Summary
Solution sidebar paths
sidebars-agentic.ts, sidebars-b2b.ts, sidebars-ciam.ts
Sidebar category items entries updated to use hyphenated path format: solution_agenticsolution-agentic, solution_B2Bsolution-B2B, solution_CIAMsolution-CIAM.
Solution navbar dropdown paths
src/navbar.ts
Solutions dropdown menu docId references updated to match hyphenated path format across CIAM, B2B IAM, and Agentic IAM entries.
Vercel redirects
vercel.json
Added non-permanent redirects mapping old underscore-separated /docs/solutions/* slugs to new hyphen-separated destinations for each solution page.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • zepatrik
  • aeneasr
  • piotrmsc

Poem

🐰 With hyphens neat and underscores shorn,

I hopped through docs at early morn.
CIAM, B2B, Agentic too,
Now march in line — a tidy crew.
The routes align, the rabbit's through! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete and lacks required template sections including the 'Related Issue or Design Document' section and missing the checklist items that are required per the repository template. Add the 'Related Issue or Design Document' section explaining why this change was needed, and complete the checklist to confirm the contributor has read guidelines and performed necessary checks.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'docs: fix underscores' is concise and directly describes the main change - converting underscores to hyphens in documentation route paths across multiple sidebar and navbar configuration files.
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 una-fix-underscore

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.

@vinckr vinckr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this also needs redirects because the URL changes

e.g.
https://www.ory.com/docs/solutions/solution_CIAM
becomes
https://www.ory.com/docs/solutions/solution-CIAM

@vinckr vinckr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@vinckr vinckr merged commit 247ab85 into master Jun 5, 2026
13 checks passed
@vinckr vinckr deleted the una-fix-underscore branch June 5, 2026 15:01
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.

3 participants