fix(goals): surface GitHub rate limit error in goal sync#1347
Conversation
|
@mallya-m 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):
|
|
@Priyanshu-byte-coder could you please add |
674e77d
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! |
What does this PR do?
Fixes silent failure when GitHub Search API rate limit is hit during goal sync. Now returns proper 429 with reset time, and the UI surfaces a user-friendly message.
Related issue
Closes #935
Changes made
/api/goals/syncX-RateLimit-Resetheader and include reset time in error message{ error, rateLimited: true }with status 429How to test