Skip to content

Fix all broken URL redirects#2950

Merged
yashovardhan merged 1 commit into
mainfrom
redirect-url
Jun 8, 2026
Merged

Fix all broken URL redirects#2950
yashovardhan merged 1 commit into
mainfrom
redirect-url

Conversation

@yashovardhan

@yashovardhan yashovardhan commented Jun 8, 2026

Copy link
Copy Markdown
Member

Multiple URLs were breaking due to the trailing slash issue in the docs. Fixing that plus adding cursor rule so that this never happens again.


Note

Medium Risk
Wide changes to production URL routing for docs; incorrect rules could break bookmarks or external links, though the edits follow documented Vercel normalization behavior.

Overview
Aligns vercel.json redirects with Vercel’s trailingSlash: true behavior so legacy URLs stop 404ing after normalization.

Redirect rules now use slash-terminated source and destination for normal doc paths, drop redundant no-slash duplicate sources (e.g. /whats-new vs /whats-new/), and fix capture-group destinations (wallet RPC aliases, Services section defaults, Embedded Wallets camelCase SDK URLs, migration guides, Agent Wallet quickstart).

Adds .cursor/rules/vercel-redirects.mdc documenting the platform behavior, good/bad examples, and curl verification. contributor-workflow.mdc now points contributors at those rules when adding redirects.

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

@yashovardhan yashovardhan requested review from a team as code owners June 8, 2026 20:01
@vercel

vercel Bot commented Jun 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
metamask-docs Building Building Preview, Comment Jun 8, 2026 8:01pm

Request Review

@yashovardhan yashovardhan merged commit bfedf29 into main Jun 8, 2026
18 of 19 checks passed
@yashovardhan yashovardhan deleted the redirect-url branch June 8, 2026 20:05
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