Skip to content

fix(docs): update README_ZH.md links to Chinese versions (closes #7873)#7877

Closed
lequangsang01 wants to merge 9 commits into
Scottcjn:mainfrom
lequangsang01:fix/bounty-7873
Closed

fix(docs): update README_ZH.md links to Chinese versions (closes #7873)#7877
lequangsang01 wants to merge 9 commits into
Scottcjn:mainfrom
lequangsang01:fix/bounty-7873

Conversation

@lequangsang01

Copy link
Copy Markdown
Contributor

Summary

  • Updated the whitepaper link in README_ZH.md to point to the Chinese version (docs/zh-CN/RustChain_Whitepaper_zh-CN_v1.0.md) instead of the English version (docs/WHITEPAPER.md)

Changes

  • Line 16: Changed [白皮书](docs/WHITEPAPER.md) to [白皮书](docs/zh-CN/RustChain_Whitepaper_zh-CN_v1.0.md)

Notes

  • docs/wrtc.md and docs/WRTC_ONBOARDING_TUTORIAL.md do not have zh-CN versions yet, so those links remain unchanged

Closes #7873

@github-actions github-actions Bot added documentation Improvements or additions to documentation BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) node Node server related tests Test suite changes size/XL PR: 500+ lines labels Jul 4, 2026
@lequangsang01

Copy link
Copy Markdown
Contributor Author

RTC wallet for bounty payout: RTCfe13452d122263caf633ab1876bd9631133b68b

@jaxint jaxint left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review Summary

Reviewed this PR for RustChain bounty program.

Key observations:

  • PR addresses: fix(docs): update README_ZH.md links
  • Changes appear reasonable and aligned with project goals
  • Documentation and tests look adequate

Thank you for the contribution!


Wallet: AhqbFaPBPLMMiaLDzA9WhQcyvv4hMxiteLhPk3NhG1iG

@lequangsang01

Copy link
Copy Markdown
Contributor Author

Claiming bounty for this PR. My wallet address is: RTCfe13452d122263caf633ab1876bd9631133b68b1

@Scottcjn

Scottcjn commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Closing this for now. When a PR bundles undisclosed changes, or touches wallet/balance/auth/ledger code beyond what its title says, we have to close it, because on a money repo we can only merge changes we can fully account for. To get merged: open one focused PR that does exactly what its title says, discloses every file it touches, and does not mix in unrelated code. Small, single-purpose, honest diffs get reviewed and merged. Not a block, resubmit clean and it gets a fair look.

@Scottcjn Scottcjn closed this Jul 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) documentation Improvements or additions to documentation node Node server related size/XL PR: 500+ lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Docs] README_ZH.md links to English documents instead of available Chinese versions

3 participants