Feature: Dashboard Suspense Streaming Implementation - #9024
Feature: Dashboard Suspense Streaming Implementation#9024armanpanigrahi59 wants to merge 14 commits into
Conversation
- Refactored dashboard data fetching to pass unawaited promises to client - Updated DashboardClient to use React.use() for suspense boundary integration - Removed blocking 3.5s opacity animation from DashboardPageWrapper - Leveraged Next.js layout streaming to instantly render dashboard shell
|
@armanpanigrahi59 is attempting to deploy a commit to the jhasourav07's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
This PR adds a significant feature to the dashboard, focusing on suspense streaming. It aligns with previous decisions, such as the addition of a dark/light preview background toggle, which enhances user experience by allowing customization. Additionally, it complements the enhancements made for tech stack contribution analysis with language-aware visualization. Please ensure all checklist items are completed before requesting a review. Thank you for your contribution! |
📦 Next.js Bundle Size Report (Gzipped Sizes)
📊 Summary of Totals
|
Aamod007
left a comment
There was a problem hiding this comment.
The suspense streaming implementation looks fantastic! However, just like your other recent PRs, this branch has accidentally included several unrelated commits from other branches (such as the validation schema fixes, timezone mocks, and webhook handler changes). Could you please rebase this branch or cherry-pick the suspense implementation into a fresh branch? Once the polluted commits are gone, I'll be happy to approve this!
Description
Fixes #71
Pillar
Visual Preview
Checklist before requesting a review:
CONTRIBUTING.mdfile.localhost:3000/api/streak?user=YOUR_USERNAME).npm run formatandnpm run lintlocally and resolved all errors (CI will fail otherwise).feat(themes): ...,fix(calculate): ...).README.mdif I added a new theme or URL parameter.