Skip to content

Add JWT auth page to sidebar#6271

Merged
Blargian merged 3 commits into
mainfrom
add-jwt-sidebar-entry
May 26, 2026
Merged

Add JWT auth page to sidebar#6271
Blargian merged 3 commits into
mainfrom
add-jwt-sidebar-entry

Conversation

@kcmannem
Copy link
Copy Markdown
Contributor

@kcmannem kcmannem commented May 25, 2026

Fixes "floating page" build error for
docs/operations/external-authenticators/jwt.md, added in ClickHouse/ClickHouse#103825.

Summary

Checklist


Note

Low Risk
Docs navigation and build-validation config only; no product or security runtime changes.

Overview
Registers operations/external-authenticators/jwt in the External authenticators sidebar (with HTTP and Kerberos) so the JWT doc is reachable in the docs nav.

Removes operations/external-authenticators/jwt.md from floating-pages-exceptions.txt, so the page is no longer exempt from the floating-pages check and must stay linked via sidebars.js.

Reviewed by Cursor Bugbot for commit 3957b9d. Bugbot is set up for automated code reviews on this repo. Configure here.

Fixes "floating page" build error for
`docs/operations/external-authenticators/jwt.md`, added in
ClickHouse/ClickHouse#103825.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@kcmannem kcmannem requested a review from a team as a code owner May 25, 2026 15:50
@vercel
Copy link
Copy Markdown

vercel Bot commented May 25, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clickhouse-docs Ready Ready Preview May 26, 2026 6:20pm
4 Skipped Deployments
Project Deployment Actions Updated (UTC)
clickhouse-docs-jp Ignored Ignored May 26, 2026 6:20pm
clickhouse-docs-ko Ignored Ignored Preview May 26, 2026 6:20pm
clickhouse-docs-ru Ignored Ignored Preview May 26, 2026 6:20pm
clickhouse-docs-zh Ignored Ignored Preview May 26, 2026 6:20pm

Request Review

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 25, 2026

Docs PR opened: ClickHouse/mintlify-docs-dev#54

Added a JWT entry under External authenticators in the sidebar with a placeholder page pending upstream content sync.

kcmannem and others added 2 commits May 26, 2026 14:07
The temporary exception added in #6274 is no longer needed since
`operations/external-authenticators/jwt` is wired into `sidebars.js`.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@Blargian Blargian merged commit 1e1bff6 into main May 26, 2026
18 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.

2 participants