Add skeleton loaders and empty state handling to tracker dashboard#559
Add skeleton loaders and empty state handling to tracker dashboard#559priya434960 wants to merge 3 commits into
Conversation
✅ Deploy Preview for github-spy ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
Warning Review limit reached
More reviews will be available in 34 minutes and 30 seconds. Learn how PR review limits work. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more reviews become available, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available. Please see our Fair Usage Limits Policy for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Tracker component now provides visual feedback during data loading and when no results are found. A MUI ChangesLoading and Empty-State UX Improvements
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
🎉 Thank you @priya434960 for your contribution. Please make sure your PR follows https://github.com/GitMetricsLab/github_tracker/blob/main/CONTRIBUTING.md#-pull-request-guidelines
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/pages/Tracker/Tracker.tsx`:
- Line 376: The empty-state message in the Tracker component contains incorrect
product casing: replace the string "No Github activity found with current
filters." with "No GitHub activity found with current filters." in the
JSX/return of the Tracker component (look for the exact literal in Tracker.tsx)
so user-facing copy uses the official "GitHub" casing.
- Around line 373-379: The empty-state message is shown based only on
currentFilteredData.length === 0; update the condition to also require that a
fetch has been executed (e.g., check a "hasFetched" or similar flag) so the
message only displays after at least one fetch completes; if no flag exists, add
a local state like hasFetched in the Tracker component (set it true when the
fetch action or the function that loads data completes) and change the render
condition around the TableRow/TableCell to currentFilteredData.length === 0 &&
hasFetched (or equivalent) to avoid showing the message on initial load.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 8cbb99c2-e3ee-4bbd-8b44-46c19493a2dd
📒 Files selected for processing (1)
src/pages/Tracker/Tracker.tsx
|
Hi, I have addressed the review comments and pushed the required fixes. If there are any additional changes needed, please let me know. Thank you! |
Related Issue
Description
Implemented improved loading and empty-state UI for the tracker dashboard.
How Has This Been Tested?
-Tested locally on localhost
-Verified Skeleton loader appears during data fetch
-Verified empty-state message appears correctly
-Verified tracker page still works normally
Screenshots
Type of Change
Summary by CodeRabbit