docs(README.zh-CN): sync Product Hunt badge + install URL with English#2033
Conversation
Two upstream README.md commits landed after the Simplified Chinese README was created (tinyhumansai#1890) and were not mirrored into README.zh-CN.md: - tinyhumansai#1895 (bd5c2ed): Product Hunt badge migrated from `badge-featured` / `featured.svg` to `badge-top-post-badge` / `top-post-badge.svg`, and the two `<p>` blocks holding the Trendshift and Product Hunt badges were merged into a single centred block with ` ` spacing. - tinyhumansai#1903 (df542e7): the download-instructions code block dropped the `?utm_source=github&utm_medium=readme` query parameters from the comment so the visible URL matches what users would type. This PR applies both diffs verbatim to README.zh-CN.md so the Chinese edition matches the English source. README.ja-JP.md is already in sync with both changes — no edit needed there.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughUpdated the Chinese README header by replacing the Product Hunt featured widget with a top-post widget and simplified the installation instruction comment to use a direct link without tracking parameters. ChangesDocumentation Updates
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
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. Comment |
tinyhumansai#2033) Co-authored-by: 李冠辰 <liguanchen@xiaomi.com>
Summary
README.md(Update README.md #1895, Update README.md #1903) were never mirrored intoREADME.zh-CN.md.README.zh-CN.mdso the Chinese edition matches the English source.README.ja-JP.mdis already in sync with both changes — no edit needed there.Problem
README.zh-CN.mdis the user-facing landing page for Chinese-speaking visitors arriving via the language badge / Trendshift / Product Hunt referrals. Two upstream changes drifted away from it:Product Hunt badge (Update README.md #1895, commit
bd5c2ed3): the English README migrated the embed frombadge-featured/featured.svgtobadge-top-post-badge/top-post-badge.svg, and merged the two separate<p>blocks (one for Trendshift, one for Product Hunt) into a single centred row with spacing. The Chinese README still shows the olderfeaturedbadge in its own block, so visitors see a stale "OpenHuman is Featured" widget instead of the current "Top Post" widget — and the layout differs between editions.Install URL (Update README.md #1903, commit
df542e72): the English README dropped?utm_source=github&utm_medium=readmefrom the URL inside the install code block's comment, so the URL shown to users matches what they would actually type. The Chinese README still carries the tracking parameters in the comment, which is both inconsistent and slightly confusing (the prose link above it kept the params; only the in-code comment was simplified).README.ja-JP.mdwas added later (#2005) and already includes both fixes, so it does not need touching in this PR.Solution
Apply the two upstream diffs to
README.zh-CN.md:featuredProduct Hunt block with thetop-post-badgemarkup and collapse the two<p>blocks into one, matchingREADME.mdline 7-16 byte-for-byte (apart from the surrounding Chinese prose).?utm_source=github&utm_medium=readmefrom the comment line inside the install code block (line 48), matchingREADME.mdline 49.No other content is touched. This is the minimal diff that re-aligns the Chinese edition to the English source as of
main.Submission Checklist
.mdlines changed;diff-coverdoes not score Markdown.## Related— N/A: documentation correction.Closes #NNN— N/A: Suggestion: Add a Chinese version of README (README.zh-CN.md) #1886 (the original request to add a Chinese README) is already fulfilled by the merged feat: add Simplified Chinese README translation #1890; this PR is sync maintenance, not the original delivery. Happy to also close Suggestion: Add a Chinese version of README (README.zh-CN.md) #1886 in a separate comment if maintainers prefer.Impact
Related
AI Authored PR Metadata (required for Codex/Linear PRs)
Linear Issue
Commit & Branch
docs/sync-zh-cn-readme-driftd535fe13Validation Run
pnpm --filter openhuman-app format:check— N/A: noapp/*files changed.pnpm typecheck— N/A: no TypeScript changed..rschanged.Validation Blocked
command:N/Aerror:N/Aimpact:N/ABehavior Changes
Parity Contract
Duplicate / Superseded PR Handling
Summary by CodeRabbit