Skip to content

fix(dashboard): migrate tables to moonshine#3004

Open
alx-xo wants to merge 7 commits into
mainfrom
alexm/fix-update-tables-to-moonshine
Open

fix(dashboard): migrate tables to moonshine#3004
alx-xo wants to merge 7 commits into
mainfrom
alexm/fix-update-tables-to-moonshine

Conversation

@alx-xo
Copy link
Copy Markdown
Contributor

@alx-xo alx-xo commented May 22, 2026

Summary

  • Migrate insights Employee and Cost tables to Moonshine Table, including sortable columns and preserved default ordering. Also removed the extra border container around these
  • Converted Triggers and Policy Center tables to Moonshine Table
  • Removed the legacy @/components/ui/table wrapper.
  • Update frontend skill guidance so future dashboard tables use Moonshine and do not recreate the removed wrapper.

Test Plan

  • git diff --check origin/main...HEAD
  • pnpm --filter ./client/dashboard lint

@alx-xo alx-xo requested review from a team as code owners May 22, 2026 18:29
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review to trigger a review and subscribe this PR to future pushes, or @claude review once for a one-time review.

Tip: disable this comment in your organization's Code Review settings.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
gram-docs-redirect Ready Ready Preview, Comment May 22, 2026 8:58pm

Request Review

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented May 22, 2026

🦋 Changeset detected

Latest commit: 96028fc

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
dashboard Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@alx-xo
Copy link
Copy Markdown
Contributor Author

alx-xo commented May 22, 2026

@alx-xo alx-xo force-pushed the alexm/fix-update-tables-to-moonshine branch from 718427e to dcda4bf Compare May 22, 2026 20:49
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 22, 2026

DES-421

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.

1 participant