Skip to content

fix: assemble dashboard page#83

Merged
codebestia merged 5 commits into
ShadeProtocol:mainfrom
manueldezman:fix/dashboard-preview-unblock
Jun 23, 2026
Merged

fix: assemble dashboard page#83
codebestia merged 5 commits into
ShadeProtocol:mainfrom
manueldezman:fix/dashboard-preview-unblock

Conversation

@manueldezman

Copy link
Copy Markdown
Contributor

Summary

Assembles the dashboard route into the intended overview layout and unblocks local previewing by fixing the missing chart dependency and local icon shim gaps. Also removes the nested dashboard shell that was causing duplicate navigation on large screens.

Closes #17

Changes

  • assembled src/app/(dashboard)/dashboard/page.tsx with stat cards, RevenueChart, and RecentActivity
  • added seeded recent activity data for the dashboard preview state
  • removed the nested dashboard shell wrapper from src/app/(dashboard)/dashboard/layout.tsx to prevent duplicate navbars
  • expanded the local lucide-react shim with the missing icons needed by the register flow and dashboard shell
  • added recharts to project dependencies so RevenueChart can build locally

Verification

  • npm run typecheck
  • npm run format:check

Notes

  • npm run typecheck was not fully clean before this change due to unrelated missing test/dev dependencies in the repo.

Screenshot

Desktop screen:
image
Mobile screen:
image

@drips-wave

drips-wave Bot commented Jun 23, 2026

Copy link
Copy Markdown

@manueldezman 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

@manueldezman

Copy link
Copy Markdown
Contributor Author

@codebestia kindly review and merge

@manueldezman manueldezman changed the title fix: unblock dashboard preview fix: assemble dashboard page Jun 23, 2026

@codebestia codebestia left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!
Thank you for your contribution.

@codebestia codebestia merged commit 973aa9c into ShadeProtocol:main Jun 23, 2026
1 check passed
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.

Assemble Main Dashboard Page

2 participants