Skip to content

fix: use PH embed badge with live upvotes#49

Merged
rohitg00 merged 1 commit intomainfrom
fix/ph-badge-embed
Feb 7, 2026
Merged

fix: use PH embed badge with live upvotes#49
rohitg00 merged 1 commit intomainfrom
fix/ph-badge-embed

Conversation

@rohitg00
Copy link
Owner

@rohitg00 rohitg00 commented Feb 7, 2026

Summary

  • Replace broken custom PH upvote fetch (GraphQL API needs auth) with the official PH embed badge image
  • The badge auto-shows live upvote count from Product Hunt's CDN
  • Sized to 28px height to fit the stats bar cleanly
  • Removed unused phUpvotes from useStats hook

Test plan

  • Verify PH badge renders in stats bar with upvote count visible
  • Verify badge links to correct PH page
  • Verify hidden on mobile

Open with Devin

Summary by CodeRabbit

  • Refactor
    • Simplified the Product Hunt badge display by replacing custom styling with a standard embedded image widget.
    • Removed Product Hunt upvote indicators from the interface and underlying statistics tracking.

@vercel
Copy link

vercel bot commented Feb 7, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
skillkit Ready Ready Preview, Comment Feb 7, 2026 1:48pm
skillkit-docs Ready Ready Preview, Comment Feb 7, 2026 1:48pm

@coderabbitai
Copy link

coderabbitai bot commented Feb 7, 2026

📝 Walkthrough

Walkthrough

The pull request removes Product Hunt upvotes tracking from the skillkit application. The custom SVG badge in App.tsx is replaced with a simple image badge. The phUpvotes field is removed from the Stats interface in useStats.ts, along with all associated API calls and data processing logic.

Changes

Cohort / File(s) Summary
Product Hunt Badge Refactoring
docs/skillkit/App.tsx
Replaces custom inline SVG Product Hunt badge with upvote indicators with a simple image badge. Updates responsive styling from hidden sm:flex to hidden sm:inline-flex with hover opacity effects.
Stats Interface Cleanup
docs/skillkit/hooks/useStats.ts
Removes phUpvotes field from the Stats interface. Eliminates Product Hunt API call from data fetching, removes related state variables and initialization logic, and simplifies the stats object to only include version, downloads, and stars.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

🐰 No more PH upvote chasing,
The badge grows lean and clean,
Stats simplified, no more tracing,
Simplest interface you've seen! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: use PH embed badge with live upvotes' clearly describes the main change: replacing the custom Product Hunt upvote implementation with an official embed badge that displays live upvote counts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/ph-badge-embed

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.

Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

@rohitg00 rohitg00 merged commit 2c05d08 into main Feb 7, 2026
8 of 9 checks passed
@rohitg00 rohitg00 deleted the fix/ph-badge-embed branch February 7, 2026 13:52
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