Skip to content

docs(unichain): remove mainnet-readonly.unichain.org RPC references - #1166

Closed
claude[bot] wants to merge 1 commit into
mainfrom
remove-mainnet-readonly-rpc
Closed

claude[bot] wants to merge 1 commit into
mainfrom
remove-mainnet-readonly-rpc

Conversation

@claude

@claude claude Bot commented Oct 8, 2026

Copy link
Copy Markdown

Requested by Joshua Gutow · Slack thread

Summary

Before: The Unichain "Submitting Transactions from L1" guide (content/unichain/technical-information/submitting-transactions-from-l1.mdx) set L2_RPC=https://mainnet-readonly.unichain.org in both the relay and finalize withdrawal snippets.

After: All references to mainnet-readonly.unichain.org are removed. Because the withdrawer command needs an L2 RPC (--l2-rpc=$L2_RPC), both snippets now use the public mainnet RPC https://mainnet.unichain.org, which the Unichain docs already use elsewhere (for example advanced-txn.mdx).

Type of change

  • Fix (typo, broken link, incorrect or outdated content)
  • New content (guide, page, code example)
  • Update to existing content
  • Other

How has this been verified?

How: replaced the two L2_RPC values; grep -rn mainnet-readonly across the repo now returns nothing. The same change is open against Uniswap/unichain-docs.

Applicable screenshots

N/A

Anything else reviewers should know?

mainnet.unichain.org is rate limited and documented as not for production; it is used here as the fallback since the snippet needs some L2 RPC.

🤖 Generated with Claude Code

https://claude.ai/code/session_01S8BPKxAYcnUpgKrPMaAiYv


Generated by Claude Code

Replace the mainnet-readonly.unichain.org L2_RPC value in the L1
withdrawal guide with the public mainnet RPC (mainnet.unichain.org)
already used elsewhere in the Unichain docs.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01S8BPKxAYcnUpgKrPMaAiYv
@github-actions

github-actions Bot commented Oct 8, 2026

Copy link
Copy Markdown

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.

@claude

claude Bot commented Oct 9, 2026

Copy link
Copy Markdown
Author

Closing — the docs source is Uniswap/universe apps/dev-portal/content/docs; the change moved to https://github.com/Uniswap/universe/pull/42957.


Generated by Claude Code

@claude claude Bot closed this Oct 9, 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.

1 participant