chore(docs, www) - add more redirects by kodster28 · Pull Request #52046 · supabase/supabase · GitHub
Skip to content

chore(docs, www) - add more redirects - #52046

Merged
kodster28 merged 2 commits into
masterfrom
kody/DEVREL-739-more-redirect-fixes
Oct 21, 2026
Merged

kodster28 merged 2 commits into
masterfrom
kody/DEVREL-739-more-redirect-fixes

Conversation

@kodster28

Copy link
Copy Markdown
Contributor

Problem

Adds more redirects for missing paths that have high-ish traffic (above 20 404 responses over the past 24 hours)

@vercel

vercel Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: supabase/supabase/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ab265178-2557-41b4-8e84-13cc9dbd6148
📥 Commits

Reviewing files that changed from the base of the PR and between d6389d1 and 4dc3e0f.

📒 Files selected for processing: 1
  • apps/www/lib/redirects.js
💤 Files with no reviewable changes: 1
  • apps/www/lib/redirects.js

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.

📝 Walkthrough

Added 12 permanent redirects in apps/www/lib/redirects.js for renamed documentation routes, including authentication, database, changelog, billing, and API pages.

Changes: Documentation route redirects

Layer / File(s) Summary
Add permanent documentation redirects
apps/www/lib/redirects.js
Adds redirects for JavaScript authentication and MFA, database schemas, getting started, Functions, changelog slugs, WebAuthn, billing, Storage CDN, log ingestion, Terraform, seeding, API, and branching documentation routes.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~5 minutes

Change: Other

Suggested reviewers: pamelachia

Merge Risk: ⚪ Minimal · up to 4dc3e

The new documentation redirects appear ready to merge after normal checks.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Description check Warning The description explains the problem but omits the required Solution, Review instructions, and Checklist sections. It does not provide enough detail about the redirect changes or how to verify them. Add the required Solution section with the redirect scope and design choices. Add numbered Review instructions that explain how to test the redirects. Complete the Checklist and include any relevant preview links or additional context.
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed The title clearly identifies the main change: adding redirects for documentation and website paths.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
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.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files.

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@kodster28
kodster28 merged commit 5491fb6 into master Oct 21, 2026
39 checks passed
@kodster28
kodster28 deleted the kody/DEVREL-739-more-redirect-fixes branch October 21, 2026 13:58
@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Braintrust eval report

Assistant (master-1792591310)

Score Average Improvements Regressions
Completeness 91% (-1pp) 4 🟢 5 🔴
Conciseness 66.2% (+3pp) 20 🟢 12 🔴
Correctness 77.2% (+1pp) 16 🟢 12 🔴
Docs Faithfulness 61% (+3pp) 8 🟢 4 🔴
Goal Completion 78.2% (+1pp) 14 🟢 8 🔴
Knowledge Usage 92.9% (+0pp) - -
SQL Identifier Quoting 100% (+0pp) - -
SQL Validity 100% (+0pp) - -
Safety 93.3% (-2pp) 1 🟢 2 🔴
Tool Usage 86.1% (0pp) 4 🟢 4 🔴
URL Validity 98.4% (+2pp) 1 🟢 -
Time_to_first_token 2.78tok (+0.01tok) 41 🟢 37 🔴
Llm_calls 10.49 (+0.03) 29 🟢 24 🔴
Tool_calls 6.55 (-0.02) 21 🟢 16 🔴
Errors 0.06 (+0) 1 🟢 2 🔴
Llm_errors 0.01 (0) 2 🟢 2 🔴
Tool_errors 0.03 (+0.01) - 1 🔴
Prompt_tokens 62386.56tok (-1400.76tok) 35 🟢 38 🔴
Prompt_cached_tokens 46422.47tok (-407.11tok) 40 🟢 30 🔴
Prompt_cache_creation_tokens 15945.25tok (-993.7tok) 33 🟢 40 🔴
Prompt_cache_creation_5m_tokens 0tok (+0tok) - -
Prompt_cache_creation_1h_tokens 0tok (+0tok) - -
Completion_tokens 1411.28tok (-67.21tok) 39 🟢 39 🔴
Completion_reasoning_tokens 410.19tok (-25.97tok) 37 🟢 38 🔴
Completion_accepted_prediction_tokens 0tok (+0tok) - -
Completion_rejected_prediction_tokens 0tok (+0tok) - -
Completion_audio_tokens 0tok (+0tok) - -
Total_tokens 63797.85tok (-1467.97tok) 37 🟢 41 🔴
Estimated_cost 0$ - -
Duration 13.96s (-0.51s) 41 🟢 37 🔴
Llm_duration 26.79s (-0.83s) 39 🟢 39 🔴

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