Skip to content

fix(site): missing 404#7772

Merged
mhartington merged 1 commit intomainfrom
fix-site-404s
Apr 3, 2026
Merged

fix(site): missing 404#7772
mhartington merged 1 commit intomainfrom
fix-site-404s

Conversation

@mhartington
Copy link
Copy Markdown
Member

@mhartington mhartington commented Apr 3, 2026

Summary by CodeRabbit

  • Chores
    • Updated URL redirect routing to send /optimize requests to the home page.

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 3, 2026

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

Project Deployment Actions Updated (UTC)
blog Ready Ready Preview, Comment Apr 3, 2026 0:56am
docs Ready Ready Preview, Comment Apr 3, 2026 0:56am
eclipse Ready Ready Preview, Comment Apr 3, 2026 0:56am
site Ready Ready Preview, Comment Apr 3, 2026 0:56am

Request Review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 8294ca85-bcff-4d07-a6ab-a4b956d92bd2

📥 Commits

Reviewing files that changed from the base of the PR and between 52a04ba and 8fded63.

📒 Files selected for processing (1)
  • apps/site/vercel.json

Walkthrough

Added a new permanent redirect rule in Vercel configuration directing traffic from the /optimize path to the root path /, while maintaining the existing redirect for /serverless to /.

Changes

Cohort / File(s) Summary
Vercel Redirect Configuration
apps/site/vercel.json
Added a new permanent redirect rule from /optimize to /, alongside the pre-existing /serverless redirect configuration.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'fix(site): missing 404' is vague and doesn't clearly communicate the actual change—adding a Vercel redirect rule from /optimize to /. Consider a more specific title like 'fix(site): add redirect from /optimize to root' that accurately describes what the change does.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.


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.

@argos-ci
Copy link
Copy Markdown

argos-ci bot commented Apr 3, 2026

The latest updates on your projects. Learn more about Argos notifications ↗︎

Build Status Details Updated (UTC)
default (Inspect) ✅ No changes detected - Apr 3, 2026, 1:03 AM

@mhartington mhartington merged commit 0bb5c43 into main Apr 3, 2026
18 checks passed
@mhartington mhartington deleted the fix-site-404s branch April 3, 2026 01:45
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