fix: add per-widget error boundaries for dashboard widgets#1807
Conversation
|
@sakshiwankhade026-coder 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):
|
|
Closing: this PR mixes two unrelated concerns — adding WidgetErrorBoundary wrappers (the stated goal) and a large dashboard layout restructuring that moves widgets around, creates duplicate section IDs (two elements with id='streaks'), and leaves the file with broken structure (a dangling grid div where a closing section tag was removed). The WidgetErrorBoundary.tsx indentation is also broken. Please submit a focused PR that only adds error boundary wrappers to existing widgets without touching the layout. |
|
Thank you for the review and feedback. I understand the concern about mixing layout changes with the error boundary implementation. I'll create a new focused PR that only adds WidgetErrorBoundary wrappers without modifying the dashboard layout or structure. |
Summary
Fixes dashboard crash issue when any single widget API fails by adding per-widget error boundaries.
Closes #1671
Type of Change
Changes Made
WidgetErrorBoundaryWhy this change is needed
Previously, if any one API request failed (GitHub rate limits, Supabase timeout, etc.), the entire dashboard would crash or show a blank screen.
Now each widget fails independently without affecting the rest of the dashboard.
How to Test
Expected Behavior
Fallback Message
Unable to load data — please try refreshing or try again later