feat: add refresh button to each dashboard widget#327
Conversation
|
@Tannuu18 is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
Priyanshu-byte-coder
left a comment
There was a problem hiding this comment.
Good refactor. Extracting fetch into useCallback is the right pattern — lets the button reuse it without duplicating logic. aria-busy, aria-label, disabled-during-load all correct. CSS vars throughout. Covers all 4 widgets (ContributionGraph, PRMetrics, StreakTracker, TopRepos). Approved.
|
PR is approved but has a conflict with the just-merged #288 (both modify ContributionGraph.tsx). Please rebase on latest main and resolve the conflict — should be straightforward to combine both changes. Then it'll merge automatically. |
|
The refactor to Push once rebased — will merge. |
Summary
Adds per-widget refresh controls to dashboard cards so users can reload individual widgets without refreshing the full page.
Closes #179
Type of Change
Changes Made
StreakTracker
ContributionGraph
PRMetrics
TopRepos
How to Test
Screenshots (if UI change)
Checklist
npm run lintpasses locallynpm run type-check)