Skip to content

Lazy-load the transactions dashboard - #569

Merged
Calebux merged 2 commits into
zintarh:mainfrom
nasirudeenbadirudeen87-cloud:fix/issue-466
Oct 1, 2026
Merged

Calebux merged 2 commits into
zintarh:mainfrom
nasirudeenbadirudeen87-cloud:fix/issue-466

Conversation

@nasirudeenbadirudeen87-cloud

@nasirudeenbadirudeen87-cloud nasirudeenbadirudeen87-cloud commented Sep 2, 2026 •

Copy link
Copy Markdown

Summary

  • Keep the heavy TransactionsOfFury client dashboard behind Next.js dynamic() so it is split from the route’s initial bundle.
  • Replace the short, generic placeholder with a full-viewport fallback matching the dashboard canvas, reserving its screen height to avoid a large layout shift.
  • Expose loading feedback as an accessible status and respect reduced-motion preferences.

Verification

  • Focused ESLint on app/[locale]/transactions-of-fury/page.tsx — passed.
  • Prettier check and git diff --check — passed.
  • pnpm typecheck and pnpm build remain blocked by pre-existing errors in unrelated files; neither reported an error in the changed route. The build currently fails on duplicate declarations/missing modules elsewhere in the repository.
  • The PR’s Vercel status is externally blocked with “Authorization required to deploy”; a Vercel team member must authorize the GitHub integration.

Fixes #466

@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown

@zeemscript is attempting to deploy a commit to the zintarh Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Sep 2, 2026

Copy link
Copy Markdown

@nasirudeenbadirudeen87-cloud Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@nasirudeenbadirudeen87-cloud

Copy link
Copy Markdown
Author

@zintarh @nasirudeenbadirudeen87-cloud Pushed commit 4cc5729 to fix/issue-466. The lazy-loaded dashboard now has a full-height, dark-theme loading status with reduced-motion support, reducing the fallback-to-content layout shift. Focused ESLint, Prettier, and git diff --check pass. Repository-wide typecheck/build still fail on unrelated existing errors; the Vercel check specifically requires team authorization. PR body now documents the issue and verification. Please authorize Vercel and review.

@Calebux
Calebux merged commit 7f70a50 into zintarh:main Oct 1, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lazy load Dashboard components to reduce bundle size

3 participants