fix: show repository subscriber counts - #3355
Conversation
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
There was a problem hiding this comment.
🟢 Ready to approve
The change is small, directly addresses the reported mapping bug, and is consistent with GitHub’s repo field semantics for subscribers/watchers.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
This PR fixes Repository Comparison’s “Watchers” metric by mapping it to GitHub’s subscribers_count field (the actual watcher/subscriber total) instead of the legacy watchers_count field that mirrors stars.
Changes:
- Update repository comparison data mapping to use
subscribers_countfor the Watchers value.
File summaries
| File | Description |
|---|---|
| src/app/dashboard/repo-comparison/page.tsx | Maps the displayed Watchers metric to GitHub subscribers_count so Stars and Watchers can differ as expected. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
Summary
Map the repository comparison Watchers metric to GitHub's subscribers_count field instead of the legacy watchers_count field that mirrors stars.
Closes #3350
Type of Change
What Changed
How to Test
Expected result: Stars and Watchers can display different values.
Checklist
Additional Context