fix: show error state when repository status fails to load#1434
fix: show error state when repository status fails to load#1434The-AarushiSingh wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Walkthrough
ChangesRepository indexing guard
Estimated code review effort: 2 (Simple) | ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ 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 |
|
Please include a screenshot of what this looks like in the UI |
|
@msukkari Thanks for the feedback! I tried to run the project locally to capture a screenshot, but I'm currently blocked by my local development setup on Windows (I'm hitting environment/setup issues unrelated to this change). I'm working on getting it running and will update this PR with a screenshot as soon as I can. |
What changed
Replaced the placeholder
return nullerror branch inRepoIndexedGuardwith an inline error state using the sharedAlertcomponent.This provides users with feedback when the repository status request fails instead of rendering a blank page.
Testing
Verified the component builds successfully.
Confirmed the change does not affect the existing loading, polling, or notification behavior.
Note
Low Risk
UI-only error handling in AskGH; no changes to API calls, auth, or indexing logic.
Overview
When the
repo-statusquery inRepoIndexedGuardfails, users now see a centered destructiveAlert(“Unable to load repository status”) instead of a blank page from the previousreturn nullplaceholder.Imports add the shared
Alertcomponents andAlertCircleicon; indexing spinner, polling, and ready notifications are unchanged.Reviewed by Cursor Bugbot for commit 0879eff. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit