Repository navigation
docs(unichain): fix dead Optimism, Infura and QuickNode links - #1177
ninglinLiu wants to merge 1 commit into
Conversation
|
Thanks for contributing to Uniswap Docs! 🦄 Quick heads up on how this repo works: the docs you see on developers.uniswap.org are built from a separate source repo. When we accept a change here, we port it over there, so your fix might show up on the live site before it lands in this repo. Updates to this repo come in batches on a regular cadence. When yours is included, you'll be credited as a co-author on the commit, so the contribution counts on your GitHub profile and in Uniswap/docs. We review every PR but can't merge everything. Either way, someone from the docs team will follow up here once we've taken a look. One legal bit: by submitting a PR you agree your contribution is licensed under the repo's MIT license. |
| - [How to Run a Superchain Node](https://docs.optimism.io/node-operators/overview) | ||
| - [Node Operation Tutorials](https://docs.optimism.io/node-operators/tutorials/node-from-docker) |
There was a problem hiding this comment.
Link-swap looks correct: docs.optimism.io/builders/node-operators/rollup-node is Optimism's old docs path and does 404 after their docs restructure; node-operators/overview and node-operators/tutorials/node-from-docker are the current OP Stack node-operator docs. I couldn't reach the network from this review sandbox to re-verify live, so please do a quick manual click-through before merging — otherwise this matches Optimism's current doc structure.
|
|
||
| - **Alchemy:** https://www.alchemy.com/docs/reference/unichain-flashblocks-api-quickstart | ||
| - **Quicknode:** https://www.quicknode.com/docs/Unichain/flashblocks/overview | ||
| - **Quicknode:** https://www.quicknode.com/docs/unichain/flashblocks/overview |
There was a problem hiding this comment.
Lowercase unichain path matches the QuickNode link already used elsewhere in this repo (content/unichain/tools/node-providers.mdx line 79: quicknode.com/docs/unichain), so this casing fix is consistent with the rest of the docs. Flagging per review policy since it's a URL change — worth a quick manual check that the page still loads.
| [Infura](https://docs.infura.io/reference/unichain/) offers highly available JSON-RPC access to Unichain Sepolia in partnership with the [Decentralized Infrastructure Network (DIN)](https://din.build) protocol. Get started building on Unichain with the Infura free tier today. | ||
|
|
||
| See the [quickstart](https://docs.metamask.io/services/reference/unichain/) to get started! | ||
| See the [quickstart](https://docs.infura.io/reference/unichain/) to get started! | ||
|
|
||
| ### Supported networks | ||
| - [Unichain Mainnet](https://docs.metamask.io/services/reference/unichain/) | ||
| - [Unichain Sepolia Testnet](https://docs.metamask.io/services/reference/unichain/) | ||
| - [Unichain Mainnet](https://docs.infura.io/reference/unichain/) | ||
| - [Unichain Sepolia Testnet](https://docs.infura.io/reference/unichain/) |
There was a problem hiding this comment.
Migrating docs.metamask.io/services/reference/unichain/ → docs.infura.io/reference/unichain/ is plausible (Infura docs have been consolidating under infura.io, away from the metamask.io services path), and the PR description notes the old path now redirects to the new one anyway. I couldn't independently verify liveness over the network in this sandbox, so please confirm the new URL resolves and still documents Unichain JSON-RPC access before merging.
Review summaryThis is a tightly-scoped, content-only PR: 9 lines across 3 Unichain pages, all URL swaps replacing dead external links with their current equivalents. No addresses, chain IDs, code samples, or frontmatter are touched, and no MDX syntax issues were introduced. Checked:
Note: my review sandbox has no outbound network access, so I could not independently re-fetch the new URLs to confirm 200s/content live. Left inline notes on each file flagging the URL changes for a quick human click-through before merge, per the usual policy of flagging any URL change — nothing here looks technically wrong or inconsistent with the rest of the docs. |
Summary
Fixes 9 dead external links on three Unichain pages (all currently return 404), found by checking every non-GitHub external link under
content/:docs.optimism.io/builders/node-operators/rollup-nodedocs.optimism.io/node-operators/overview(OP Stack "Node operators" overview, which lists the tutorials/clients/reference)docs.optimism.io/builders/node-operators/rollup-node#node-operator-tutorialsdocs.optimism.io/node-operators/tutorials/node-from-docker(first entry of the current Tutorials section)docs.metamask.io/services/reference/unichain/docs.infura.io/reference/unichain/("Unichain | Infura documentation";docs.metamask.io/services/reference/itself now redirects todocs.infura.io/reference/)www.quicknode.com/docs/Unichain/flashblocks/overviewwww.quicknode.com/docs/unichain/flashblocks/overview(path is now lowercase; "Flashblocks Overview | Quicknode Docs")Type of change
How has this been verified?
curl -Lreturns 404 for every old URL and 200 for every new one, and each new page's title matches the linked topic.Anything else reviewers should know?
Docs-only, 9 lines across 3 files. Separate from #1173–#1176. Prepared with AI assistance; I reviewed the change and checked every link listed above.