Skip to content

docs(start): fix authenticated routes doc URL#7214

Merged
schiller-manuel merged 1 commit intoTanStack:mainfrom
pavan-sh:docs/fix-authenticated-routes-link-v2
Apr 17, 2026
Merged

docs(start): fix authenticated routes doc URL#7214
schiller-manuel merged 1 commit intoTanStack:mainfrom
pavan-sh:docs/fix-authenticated-routes-link-v2

Conversation

@pavan-sh
Copy link
Copy Markdown
Contributor

@pavan-sh pavan-sh commented Apr 17, 2026

Fixes one docs link in authentication-overview.md by removing the .md suffix from the authenticated-routes URL so it resolves correctly on the TanStack docs site (no more 404)

Summary by CodeRabbit

  • Documentation
    • Fixed authentication guide documentation reference link.

@github-actions github-actions Bot added the documentation Everything documentation related label Apr 17, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 17, 2026

📝 Walkthrough

Walkthrough

Updated a documentation link by removing the .md file extension from a Router Authentication Guide reference in the authentication overview guide. This is a minor documentation link correction.

Changes

Cohort / File(s) Summary
Documentation Link Update
docs/start/framework/react/guide/authentication-overview.md
Removed .md file extension from Router Authentication Guide URL reference.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A link that wandered far astray,
With .md extension leading astray,
A bunny fixed it with care today,
The path now clean, hip-hip-hooray!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately describes the primary change: fixing a documentation URL by removing the .md file extension from the authenticated routes link.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Apr 17, 2026

View your CI Pipeline Execution ↗ for commit 7d0f6f3

Command Status Duration Result
nx affected --targets=test:eslint,test:unit,tes... ✅ Succeeded <1s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 6s View ↗

☁️ Nx Cloud last updated this comment at 2026-04-17 14:26:10 UTC

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.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
docs/start/framework/react/guide/authentication-overview.md (1)

239-239: ⚠️ Potential issue | 🟡 Minor

Remove .md extension from line 239 for consistency with cross-project link format.

Line 239 uses /router/latest/docs/framework/react/how-to/README.md#authentication, which includes the .md extension before the anchor. This is inconsistent with line 230 (/router/latest/docs/framework/react/guide/authenticated-routes) and the standard pattern for all other cross-project links in the Start documentation (which omit the .md extension). Change line 239 to: /router/latest/docs/framework/react/how-to/README#authentication

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docs/start/framework/react/guide/authentication-overview.md` at line 239,
Update the cross-project link on the page by removing the `.md` extension from
the URL string
`/router/latest/docs/framework/react/how-to/README.md#authentication`; replace
it with `/router/latest/docs/framework/react/how-to/README#authentication` so it
matches the other links (e.g.,
`/router/latest/docs/framework/react/guide/authenticated-routes`) and the
project's standard link format.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@docs/start/framework/react/guide/authentication-overview.md`:
- Line 239: Update the cross-project link on the page by removing the `.md`
extension from the URL string
`/router/latest/docs/framework/react/how-to/README.md#authentication`; replace
it with `/router/latest/docs/framework/react/how-to/README#authentication` so it
matches the other links (e.g.,
`/router/latest/docs/framework/react/guide/authenticated-routes`) and the
project's standard link format.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4a67d731-3cc8-4a2e-8fd2-435f7d52def2

📥 Commits

Reviewing files that changed from the base of the PR and between 3485a88 and 7d0f6f3.

📒 Files selected for processing (1)
  • docs/start/framework/react/guide/authentication-overview.md

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 17, 2026

More templates

@tanstack/arktype-adapter

npm i https://pkg.pr.new/@tanstack/arktype-adapter@7214

@tanstack/eslint-plugin-router

npm i https://pkg.pr.new/@tanstack/eslint-plugin-router@7214

@tanstack/eslint-plugin-start

npm i https://pkg.pr.new/@tanstack/eslint-plugin-start@7214

@tanstack/history

npm i https://pkg.pr.new/@tanstack/history@7214

@tanstack/nitro-v2-vite-plugin

npm i https://pkg.pr.new/@tanstack/nitro-v2-vite-plugin@7214

@tanstack/react-router

npm i https://pkg.pr.new/@tanstack/react-router@7214

@tanstack/react-router-devtools

npm i https://pkg.pr.new/@tanstack/react-router-devtools@7214

@tanstack/react-router-ssr-query

npm i https://pkg.pr.new/@tanstack/react-router-ssr-query@7214

@tanstack/react-start

npm i https://pkg.pr.new/@tanstack/react-start@7214

@tanstack/react-start-client

npm i https://pkg.pr.new/@tanstack/react-start-client@7214

@tanstack/react-start-rsc

npm i https://pkg.pr.new/@tanstack/react-start-rsc@7214

@tanstack/react-start-server

npm i https://pkg.pr.new/@tanstack/react-start-server@7214

@tanstack/router-cli

npm i https://pkg.pr.new/@tanstack/router-cli@7214

@tanstack/router-core

npm i https://pkg.pr.new/@tanstack/router-core@7214

@tanstack/router-devtools

npm i https://pkg.pr.new/@tanstack/router-devtools@7214

@tanstack/router-devtools-core

npm i https://pkg.pr.new/@tanstack/router-devtools-core@7214

@tanstack/router-generator

npm i https://pkg.pr.new/@tanstack/router-generator@7214

@tanstack/router-plugin

npm i https://pkg.pr.new/@tanstack/router-plugin@7214

@tanstack/router-ssr-query-core

npm i https://pkg.pr.new/@tanstack/router-ssr-query-core@7214

@tanstack/router-utils

npm i https://pkg.pr.new/@tanstack/router-utils@7214

@tanstack/router-vite-plugin

npm i https://pkg.pr.new/@tanstack/router-vite-plugin@7214

@tanstack/solid-router

npm i https://pkg.pr.new/@tanstack/solid-router@7214

@tanstack/solid-router-devtools

npm i https://pkg.pr.new/@tanstack/solid-router-devtools@7214

@tanstack/solid-router-ssr-query

npm i https://pkg.pr.new/@tanstack/solid-router-ssr-query@7214

@tanstack/solid-start

npm i https://pkg.pr.new/@tanstack/solid-start@7214

@tanstack/solid-start-client

npm i https://pkg.pr.new/@tanstack/solid-start-client@7214

@tanstack/solid-start-server

npm i https://pkg.pr.new/@tanstack/solid-start-server@7214

@tanstack/start-client-core

npm i https://pkg.pr.new/@tanstack/start-client-core@7214

@tanstack/start-fn-stubs

npm i https://pkg.pr.new/@tanstack/start-fn-stubs@7214

@tanstack/start-plugin-core

npm i https://pkg.pr.new/@tanstack/start-plugin-core@7214

@tanstack/start-server-core

npm i https://pkg.pr.new/@tanstack/start-server-core@7214

@tanstack/start-static-server-functions

npm i https://pkg.pr.new/@tanstack/start-static-server-functions@7214

@tanstack/start-storage-context

npm i https://pkg.pr.new/@tanstack/start-storage-context@7214

@tanstack/valibot-adapter

npm i https://pkg.pr.new/@tanstack/valibot-adapter@7214

@tanstack/virtual-file-routes

npm i https://pkg.pr.new/@tanstack/virtual-file-routes@7214

@tanstack/vue-router

npm i https://pkg.pr.new/@tanstack/vue-router@7214

@tanstack/vue-router-devtools

npm i https://pkg.pr.new/@tanstack/vue-router-devtools@7214

@tanstack/vue-router-ssr-query

npm i https://pkg.pr.new/@tanstack/vue-router-ssr-query@7214

@tanstack/vue-start

npm i https://pkg.pr.new/@tanstack/vue-start@7214

@tanstack/vue-start-client

npm i https://pkg.pr.new/@tanstack/vue-start-client@7214

@tanstack/vue-start-server

npm i https://pkg.pr.new/@tanstack/vue-start-server@7214

@tanstack/zod-adapter

npm i https://pkg.pr.new/@tanstack/zod-adapter@7214

commit: 7d0f6f3

@schiller-manuel schiller-manuel merged commit e2332b0 into TanStack:main Apr 17, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Everything documentation related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants