Skip to content

fix: align wallet docs with canonical mainnet config - #38

Open
chriszhao1988 wants to merge 1 commit into
mainfrom
fix/dev-897-doc-network-canonical
Open

fix: align wallet docs with canonical mainnet config#38
chriszhao1988 wants to merge 1 commit into
mainfrom
fix/dev-897-doc-network-canonical

Conversation

@chriszhao1988

@chriszhao1988 chriszhao1988 commented May 9, 2026

Copy link
Copy Markdown
Contributor

Summary

  • align legacy MetaMask and Coinbase wallet docs with the canonical mainnet config from Wallet Setup
  • update the mainnet explorer reference to https://scan.treasurenet.io/
  • fix the featured networks testnet explorer typo in static config

Context

  • Linear: DEV-897
  • This PR only updates docs and static documentation config in the docs repo.

Validation

  • reviewed the updated docs against docs/12/Wallet Setup.md and docs/12/WalletSetup.mdx canonical values
  • no runtime code changes

Summary by CodeRabbit

  • Documentation

    • Updated Treasurenet Mainnet network configuration across wallet setup guides.
    • Corrected block explorer URLs in documentation and configuration files.
    • Updated documentation across English and Chinese language versions.
  • Chores

    • Updated featured network configuration files with current network parameters.

@coderabbitai

coderabbitai Bot commented May 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

This pull request updates Treasurenet network endpoints across documentation and configuration files. The RPC URL, Chain ID, and Block Explorer URL are changed to new Mainnet values, with the testnet explorer URL corrected. Changes are synchronized across JSON configuration, English documentation, and Chinese translations.

Changes

Treasurenet Network Endpoints Update

Layer / File(s) Summary
Network Configuration Data
static/img/featuredNetworks.json, static/img/featuredNetworks_testnet.json
Mainnet explorer URL changed to https://scan.treasurenet.io/ and Testnet explorer URL corrected to https://evmexplorer.testnet.treasurenet.io/ in both featured network configuration files.
English Wallet Documentation
docs/fundamentals/wallets/coinbase.md, docs/fundamentals/wallets/metamask.md
Coinbase and MetaMask manual network addition guides updated with new RPC URL (https://rpc.treasurenet.io), Chain ID (5570), and Block Explorer URL (https://scan.treasurenet.io/).
Chinese Wallet and Explorer Documentation
i18n/zh-Hans/docusaurus-plugin-content-docs/current/blockExplorers/intro.md, i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/coinbase.md, i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/metamask.md
Chinese translations of wallet setup guides and block explorer introduction updated with corresponding new Mainnet endpoint values and corrected testnet explorer URLs.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Network paths now gleam so bright,
From scan to rpc, all is right,
Five thousand five-seventy's the chain,
Documentation updates rain,
Documentation updates rain!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: align wallet docs with canonical mainnet config' directly and accurately describes the main change—updating wallet documentation to align with canonical mainnet configuration parameters.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/dev-897-doc-network-canonical

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/coinbase.md (1)

38-38: ⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

Update screenshot to reflect Chain ID 5570.

The screenshot filename addnetwork35002little.png contains the old chain ID (5002), but the documentation specifies Chain ID 5570 for the Treasurenet mainnet. Either replace this screenshot with an updated image showing the new configuration (Chain ID 5570, RPC URL: https://rpc.treasurenet.io, Explorer: https://scan.treasurenet.io/), or if the image already displays the updated values, rename it to match the new chain ID (e.g., addnetwork35570little.png) for consistency.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/coinbase.md`
at line 38, The referenced screenshot filename addnetwork35002little.png in the
Coinbase doc should be updated to reflect Chain ID 5570: either replace the
image with a new screenshot that shows Chain ID 5570, RPC URL
https://rpc.treasurenet.io and Explorer https://scan.treasurenet.io/, or if the
current image already shows those values, rename the file to something like
addnetwork35570little.png and update the image reference in coinbase.md
accordingly so the filename and displayed Chain ID are consistent.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In
`@i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/metamask.md`:
- Line 34: The screenshot reference in the markdown uses addmainnet5002.png but
the doc text expects Chain ID 5570; update the image reference to
addmainnet5570.png (or replace the file contents) and verify that the image
referenced by addmainnet5570.png clearly shows Chain ID 5570 and the correct
mainnet values (RPC URL, chain name, symbol, decimals) so the screenshot matches
the written instructions in metamask.md.

---

Outside diff comments:
In
`@i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/coinbase.md`:
- Line 38: The referenced screenshot filename addnetwork35002little.png in the
Coinbase doc should be updated to reflect Chain ID 5570: either replace the
image with a new screenshot that shows Chain ID 5570, RPC URL
https://rpc.treasurenet.io and Explorer https://scan.treasurenet.io/, or if the
current image already shows those values, rename the file to something like
addnetwork35570little.png and update the image reference in coinbase.md
accordingly so the filename and displayed Chain ID are consistent.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 200d1a65-e37f-44dd-9d24-b1e7fb1914de

📥 Commits

Reviewing files that changed from the base of the PR and between b9f2752 and 4e64eb9.

📒 Files selected for processing (7)
  • docs/fundamentals/wallets/coinbase.md
  • docs/fundamentals/wallets/metamask.md
  • i18n/zh-Hans/docusaurus-plugin-content-docs/current/blockExplorers/intro.md
  • i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/coinbase.md
  • i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/metamask.md
  • static/img/featuredNetworks.json
  • static/img/featuredNetworks_testnet.json

- Block Explorer URL (optional):https://evmexplorer.treasurenet.io/
- Block Explorer URL (optional):https://scan.treasurenet.io/

![addmainnet](/img/docs/addmainnet5002.png)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟡 Minor | ⚡ Quick win

🧩 Analysis chain

🏁 Script executed:

fd -t f "addmainnet.*\.png" -i

Repository: treasurenetprotocol/docs

Length of output: 134


🏁 Script executed:

find . -name "*5570*" -o -name "*addmainnet*" | grep -i png

Repository: treasurenetprotocol/docs

Length of output: 138


🏁 Script executed:

git ls-files | grep -E "(addmainnet|img/docs)" | head -20

Repository: treasurenetprotocol/docs

Length of output: 865


🏁 Script executed:

cat -n i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/metamask.md | head -50

Repository: treasurenetprotocol/docs

Length of output: 1745


🏁 Script executed:

file static/img/docs/addmainnet.png static/img/docs/addmainnet5002.png

Repository: treasurenetprotocol/docs

Length of output: 113


🏁 Script executed:

ls -lh static/img/docs/addmainnet*.png

Repository: treasurenetprotocol/docs

Length of output: 232


🏁 Script executed:

git log --oneline -10 -- i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/metamask.md

Repository: treasurenetprotocol/docs

Length of output: 129


🏁 Script executed:

git log --oneline -10 -- static/img/docs/addmainnet*.png

Repository: treasurenetprotocol/docs

Length of output: 129


🏁 Script executed:

git diff HEAD~5..HEAD -- i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/metamask.md | grep -A 5 -B 5 "addmainnet"

Repository: treasurenetprotocol/docs

Length of output: 103


🏁 Script executed:

git show 4e64eb9 --stat

Repository: treasurenetprotocol/docs

Length of output: 34735


🏁 Script executed:

git show 4e64eb9 -- i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/metamask.md

Repository: treasurenetprotocol/docs

Length of output: 2643


Update the screenshot reference to match Chain ID 5570.

The image filename addmainnet5002.png references the old chain ID (5002), but the instructions specify Chain ID 5570. Either the screenshot filename should be updated to addmainnet5570.png, or the referenced image needs to be verified to ensure it displays the correct Chain ID and mainnet values (5570, RPC URL, etc.) to avoid confusing users.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In
`@i18n/zh-Hans/docusaurus-plugin-content-docs/current/fundamentals/wallets/metamask.md`
at line 34, The screenshot reference in the markdown uses addmainnet5002.png but
the doc text expects Chain ID 5570; update the image reference to
addmainnet5570.png (or replace the file contents) and verify that the image
referenced by addmainnet5570.png clearly shows Chain ID 5570 and the correct
mainnet values (RPC URL, chain name, symbol, decimals) so the screenshot matches
the written instructions in metamask.md.

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