Skip to content

fix(docs): redirect docs root to /overview#3935

Merged
saddlepaddle merged 1 commit intomainfrom
fix-superset-docs-redirec
May 1, 2026
Merged

fix(docs): redirect docs root to /overview#3935
saddlepaddle merged 1 commit intomainfrom
fix-superset-docs-redirec

Conversation

@saddlepaddle
Copy link
Copy Markdown
Collaborator

@saddlepaddle saddlepaddle commented May 1, 2026

Summary

Test plan

  • Visit https://docs.superset.sh/ on the deploy preview and confirm it lands on /overview
  • Visit https://docs.superset.sh/docs and confirm it also lands on /overview

Summary by CodeRabbit

  • Chores
    • Updated documentation site navigation to redirect the root path to the overview page.
    • Adjusted search and sitemap priorities to reflect the new documentation structure.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 1, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: af8a8b63-e8c3-4e24-8d32-60463032781b

📥 Commits

Reviewing files that changed from the base of the PR and between a8c0810 and 4acefa2.

📒 Files selected for processing (2)
  • apps/docs/next.config.mjs
  • apps/docs/src/app/sitemap.ts

📝 Walkthrough

Walkthrough

The documentation site's primary landing page is being redirected from /installation to /overview. Both redirect rules and sitemap priority configurations are updated to reflect this change across two configuration files.

Changes

Cohort / File(s) Summary
Documentation Navigation Configuration
apps/docs/next.config.mjs, apps/docs/src/app/sitemap.ts
Redirect rules updated to point root path and /docs to /overview instead of /installation; sitemap priority reassigned so /overview receives priority 1.0 (previously assigned to /installation).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A hop through the docs, a path refined,
From installation's door to overview's find,
The priority shifts, the redirects align,
Now travelers start where all paths intertwine!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-superset-docs-redirec

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
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@saddlepaddle saddlepaddle merged commit 567ef46 into main May 1, 2026
7 of 9 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

🚀 Preview Deployment

🔗 Preview Links

Service Status Link
Neon Database (Neon) View Branch
Vercel API (Vercel) Open Preview
Vercel Web (Vercel) Open Preview
Vercel Marketing (Vercel) Open Preview
Vercel Admin (Vercel) Open Preview
Vercel Docs (Vercel) Open Preview

Preview updates automatically with new commits

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented May 1, 2026

Greptile Summary

Fixes a broken docs root redirect introduced when /installation was removed in #3926 — both / and /docs now correctly redirect to /overview, and the sitemap priority is updated to match.

Confidence Score: 5/5

Safe to merge — minimal, targeted fix with no logic concerns.

Two-line changes updating a stale string reference in a redirect config and sitemap priority check. No new logic introduced, no P0/P1 findings.

No files require special attention.

Important Files Changed

Filename Overview
apps/docs/next.config.mjs Updated / and /docs redirects from /installation to /overview to fix the 404 broken by #3926.
apps/docs/src/app/sitemap.ts Updated sitemap priority check from /installation to /overview so the new landing page receives the highest SEO priority of 1.0.

Sequence Diagram

sequenceDiagram
    participant Browser
    participant Next.js
    participant Overview as /overview

    Browser->>Next.js: GET /
    Next.js-->>Browser: 307 Redirect → /overview
    Browser->>Overview: GET /overview
    Overview-->>Browser: 200 OK

    Browser->>Next.js: GET /docs
    Next.js-->>Browser: 307 Redirect → /overview
    Browser->>Overview: GET /overview
    Overview-->>Browser: 200 OK
Loading

Reviews (1): Last reviewed commit: "fix(docs): redirect docs root to /overvi..." | Re-trigger Greptile

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.

1 participant