Skip to content

docs: show the #5 Product of the Day rank on the site and in the README - #1123

Merged
EtienneLescot merged 1 commit into
mainfrom
chore/ph-5th-badge
Oct 12, 2026
Merged

EtienneLescot merged 1 commit into
mainfrom
chore/ph-5th-badge

Conversation

@EtienneLescot

@EtienneLescot EtienneLescot commented Oct 12, 2026 •

Copy link
Copy Markdown
Collaborator

OpenScreen 2 finished #5 Product of the Day on Product Hunt (October 11, 2026). This replaces the "Featured" badge added in #1108.

  • Site: the rank is a pill beside "OpenScreen v2 is here", same style, with Product Hunt's mark. Product Hunt's own embed was a white 250x54 card next to the download button and pulled the eye away from it. The download button now stands alone. Translated in the 7 site locales (i18n:sync --accept, i18n:check clean).
  • README: a shields.io pill in the badge row, in place of the large embed.

Checked locally on the docs dev server, desktop and 375 px wide (the two pills wrap, 10 px apart, no horizontal scroll).

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
  • Updates
    • Grouped the Product Hunt and release badges together in the homepage call-to-action area.
    • Updated the README badges to include the Product Hunt ranking.

The site shows the rank as a pill beside the v2 announcement, styled like it and translated. Product Hunt's own embed was a white 250x54 card that outshone the download button beside it. The README gets a shields.io pill in its badge row instead of the large embed.
@coderabbitai

coderabbitai Bot commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: e9d0daf5-d8a0-410a-8474-9ae887a38280

📥 Commits

Reviewing files that changed from the base of the PR and between cb8f89c and 310e44a.


📒 Files selected for processing (12)
  • README.md
  • website/i18n/code.source.json
  • website/i18n/de/code.json
  • website/i18n/es/code.json
  • website/i18n/fr/code.json
  • website/i18n/ja/code.json
  • website/i18n/pt-BR/code.json
  • website/i18n/zh-CN/code.json
  • website/i18n/zh-TW/code.json
  • website/src/components/ReleaseBadge/index.tsx
  • website/src/pages/index.module.css
  • website/src/pages/index.tsx

Included review availability: This review used your included allowance. Your plan provides up to 8 included reviews per hour; 7 remain after this review.



📝 Walkthrough

Walkthrough

The README and homepage now display a Product Hunt “#5 Product of the Day” badge. The homepage groups it with the release badge, and localization files add the ranking announcement in seven languages.

Changes

Product Hunt ranking badge

Layer / File(s) Summary
Ranking label and badge component
website/i18n/code.source.json, website/i18n/*/code.json, website/src/components/ReleaseBadge/index.tsx
The source messages and seven locale files add the Product Hunt ranking announcement. The exported ProductHuntBadge component links to the Product Hunt page and displays the translated ranking label.
Badge placement and layout
README.md, website/src/pages/index.tsx, website/src/pages/index.module.css
The homepage groups the release and Product Hunt badges in a responsive row and removes the previous embedded badge. The README places the Product Hunt and Trendshift badges together.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other


Merge Risk: ⚪ Minimal · up to 310e4

The Product Hunt ranking badge is added to the site and README with localized text and responsive placement. No material merge-blocking risk is evident in the supplied review context.

Pre-merge checks | Passed 3 | Failed 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Description check Warning The description explains the main change and testing performed, but it omits the required template sections and checklist selections for the related issue, change type, release impact, desktop impact,… Add the required template headings and complete the applicable fields. Include the related issue status, select the change type, release impact, and desktop impact, and provide screenshots or video for the visual changes.
Docstring Coverage Warning Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 2 files. (10 skipped: … Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (3 passed)
Check name Status Explanation
Title check Passed The title clearly summarizes the primary change: showing the #5 Product of the Day rank on the site and in the README.
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.

Full details: Description check

Explanation

The description explains the main change and testing performed, but it omits the required template sections and checklist selections for the related issue, change type, release impact, desktop impact, and screenshots or video.


Full details: Docstring Coverage

Explanation

Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 2 files. (10 skipped: 10 unsupported.)


  • Fix all pre-merge checks with AI
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Commit to this branch
  • Create a new PR

🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@EtienneLescot
EtienneLescot merged commit fe8c702 into main Oct 12, 2026
21 checks passed
@EtienneLescot
EtienneLescot deleted the chore/ph-5th-badge branch October 12, 2026 08:29
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