Move authentication overview to enterprise docs - #6858
Open
masenf wants to merge 2 commits into
Open
Conversation
…overview The authentication overview docs are superseded by the enterprise auth docs. Remove the old page so its route is free for the redirect, point the docs landing "Auth" card (already marked Enterprise-only) at the enterprise auth overview, and drop the now-empty Authentication section from the backend sidebar. Old links keep working via the redirect. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PdytUN4KVXsMb8KxJwtCR7
Contributor
Greptile SummaryThe PR relocates the authentication overview into the enterprise documentation while preserving access through a redirect.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| docs/app/reflex_docs/pages/docs_landing/views/framework.py | Updates the authentication landing-page card to use the generated enterprise overview route. |
| docs/app/reflex_docs/reflex_docs.py | Adds a redirect from the removed authentication overview route to its enterprise location. |
| docs/app/reflex_docs/templates/docpage/sidebar/sidebar.py | Allows cross-reference leaves to be omitted from the previous/next navigation sequence. |
| docs/app/reflex_docs/templates/docpage/sidebar/sidebar_items/learn.py | Replaces the removed backend authentication page with an enterprise documentation cross-reference. |
| docs/app/reflex_docs/templates/docpage/sidebar/state.py | Adds explicit sidebar-item metadata for exclusion from previous/next navigation. |
| docs/app/tests/test_routes.py | Verifies that the old document route is removed and the enterprise route is registered. |
| docs/app/tests/test_sidebar.py | Verifies the authentication cross-reference and enterprise previous/next sequence. |
| docs/authentication/authentication_overview.md | Removes the superseded general authentication overview. |
Reviews (2): Last reviewed commit: "Keep Authentication in the backend sideb..." | Re-trigger Greptile
Merging this PR will not alter performance
Comparing Footnotes
|
Restore the Authentication entry removed with the old page, now pointing at /enterprise/auth/overview/. The entry is a cross-reference into the enterprise section, so it's excluded from the prev/next footer chain via a new exclude_from_prev_next flag — otherwise it would shadow the enterprise occurrence and replace that page's footer links with backend neighbors. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01PdytUN4KVXsMb8KxJwtCR7
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Type of change
Description
Relocates the authentication overview documentation from the general backend docs section to the enterprise documentation section, reflecting that authentication is an enterprise-only feature in Reflex.
Changes:
docs/authentication/authentication_overview.md/enterprise/auth/overview/)/authentication/authentication-overview/) to new path (/enterprise/auth/overview/)This change ensures users are directed to the correct documentation location and maintains backward compatibility through URL redirects.
Testing
test_authentication_overview_moved_to_enterpriseto verify the route migrationhttps://claude.ai/code/session_01PdytUN4KVXsMb8KxJwtCR7