Skip to content

fix(dashboard): decouple company dialog from menu#11

Merged
Vann-Dev merged 1 commit into
mainfrom
kaito/fix-company-switch-crash
May 12, 2026
Merged

fix(dashboard): decouple company dialog from menu#11
Vann-Dev merged 1 commit into
mainfrom
kaito/fix-company-switch-crash

Conversation

@Vann-Dev
Copy link
Copy Markdown
Owner

Summary

  • Fixes the production company switch/create-company crash by removing the nested Radix Dialog trigger from the company dropdown menu.
  • Opens the create-company dialog via controlled sidebar state instead, while preserving the polished dropdown switcher, settings link, long-name truncation, and create-company flow.
  • Keeps direct/dashboard create-company usage working by making CreateCompanyDialog support controlled mode.

Checks

  • PASS: corepack pnpm lint
  • PASS: DATABASE_URL=postgresql://user:pass@localhost:5432/db?schema=public corepack pnpm typecheck
  • PASS: DATABASE_URL=postgresql://user:pass@localhost:5432/db?schema=public corepack pnpm build

Note: build still emits the existing Next.js warning that the middleware file convention is deprecated in favor of proxy.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 12, 2026

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

Project Deployment Actions Updated (UTC)
agent-bridge Ready Ready Preview, Comment May 12, 2026 1:29am

@Vann-Dev Vann-Dev merged commit 8719718 into main May 12, 2026
2 checks 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.

1 participant