perf: debounce repo search input in RepoAnalyticsExplorer#1922
Conversation
|
@neha2006-6 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. |
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.
Thanks for your first PR on DevTrack! 🎉
A maintainer will review it within 48 hours. While you wait:
- Make sure CI is passing (type-check + lint)
- Double-check the PR description is filled out and the issue is linked
- Feel free to ask questions in Discussions if you need help
If you find DevTrack useful, a ⭐ star on the repo is always appreciated — it helps the project grow and attract more contributors!
|
CI is failing because of a broken change in // Current (broken) — actual and target are not defined here
const pct = actual > 0 ? Math.max(1, Math.round((actual / target) * 100)) : 0;
// Should be:
const pct = goal.current > 0 ? Math.max(1, Math.round((goal.current / goal.target) * 100)) : 0;The debounce implementation in |
|
I've pushed the GoalTracker fix requested in the review (goal.current / goal.target). |
6f1b831
into
Priyanshu-byte-coder:main
|
🎉 Merged! Thanks for contributing to DevTrack. If the project has been useful to you, a ⭐ star on the repo is the easiest way to support it — it helps DevTrack get discovered by more developers. Keep an eye on open issues for your next contribution! |
Summary
Added debounce functionality to the repo search
input in RepoAnalyticsExplorer to prevent API
calls on every keystroke.
Closes #1894
Type of Change
Changes Made
How to Test
Checklist