feat: version history page display download count#2178
feat: version history page display download count#2178btea wants to merge 3 commits intonpmx-dev:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
📝 WalkthroughWalkthroughAdds per-version download metrics for npm packages. A new cached server API handler ( Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 1✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
app/pages/package/[[org]]/[name]/versions.vue (1)
295-302: Consider a shared download-count component.The same lookup/format/label block now exists in five places, and the copies already differ slightly (
divvsspan, width classes, etc.). Pulling this into a tiny component or shared helper would make later styling or accessibility tweaks much easier to keep consistent.Also applies to: 352-359, 443-450, 515-522, 563-570
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 15baadc8-c5b4-4bfe-827c-2d8460ab9ee2
📒 Files selected for processing (3)
app/pages/package/[[org]]/[name]/versions.vueserver/api/registry/npmjs-versions/[...pkg].get.tsserver/utils/npm-website-versions.ts

🔗 Linked issue
related to #2025
🧭 Context
The version history page displays the download count for each version over the past 7 days, consistent with the npmjs functionality, which seems very useful.
📚 Description