Skip to content

fix: replace broken bridge-token link with Optimism standard bridge docs#1271

Closed
ygd58 wants to merge 2 commits intobase:masterfrom
ygd58:fix/bridge-token-broken-link
Closed

fix: replace broken bridge-token link with Optimism standard bridge docs#1271
ygd58 wants to merge 2 commits intobase:masterfrom
ygd58:fix/bridge-token-broken-link

Conversation

@ygd58
Copy link
Copy Markdown

@ygd58 ygd58 commented Apr 3, 2026

Problem

The link Bridging an L1 Token to Base in bridges.mdx points to a page that does not exist, causing a 404 error reported in #1159.

A redirect exists in docs.json pointing this path to /base-chain/network-information/bridges (the same page), which creates a self-referencing loop.

Fix

Replaced the broken internal link with the official Optimism standard bridge documentation at https://docs.optimism.io/builders/app-developers/bridging/standard-bridge, which covers the exact topic (deploying ERC-20 tokens on Base using standard bridge contracts).

Closes #1159

@cb-heimdall
Copy link
Copy Markdown
Collaborator

cb-heimdall commented Apr 3, 2026

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/2
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 1
Sum 2

@ygd58
Copy link
Copy Markdown
Author

ygd58 commented Apr 14, 2026

@roethke @soheimam Could you please review this PR? It fixes a broken link in the bridge documentation. Thanks!

@ygd58
Copy link
Copy Markdown
Author

ygd58 commented Apr 14, 2026

@soheimam Is this PR also not needed? The bridge-token link returns 404 on the live docs site. Happy to close if not required.

@ygd58 ygd58 force-pushed the fix/bridge-token-broken-link branch 2 times, most recently from ddf660c to c2ff389 Compare April 14, 2026 08:17
@ygd58 ygd58 force-pushed the fix/bridge-token-broken-link branch from c2ff389 to 166f48d Compare April 14, 2026 11:30
@ygd58
Copy link
Copy Markdown
Author

ygd58 commented Apr 14, 2026

Closing this PR. The page is not live as noted by @soheimam, and the Optimism link I added was incorrect. Thanks for the review!

@ygd58 ygd58 closed this Apr 14, 2026
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.

Bug, Not Found: Base Chain -> Bridge Token

2 participants