Skip to content

fix: Do not self import @calcom/ui#20050

Merged
emrysal merged 4 commits intomainfrom
bugfix/do-not-self-import-calcom-ui
Mar 13, 2025
Merged

fix: Do not self import @calcom/ui#20050
emrysal merged 4 commits intomainfrom
bugfix/do-not-self-import-calcom-ui

Conversation

@emrysal
Copy link
Copy Markdown
Contributor

@emrysal emrysal commented Mar 13, 2025

What does this PR do?

  • Fixes color picker Button mocks
  • Don't use the bucket index.ts file internally in @calcom/ui
  • Moves about the wrapping for AppListCard
  • Moves CredentialOwner from app-store/types to calcom/ui
  • Moves some index.ts files to dedicated files (prevent misuse)

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 13, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Visit Preview Mar 13, 2025 5:04pm
calcom-web-canary ⬜️ Ignored (Inspect) Visit Preview Mar 13, 2025 5:04pm

@graphite-app
Copy link
Copy Markdown

graphite-app bot commented Mar 13, 2025

Graphite Automations

"Add foundation team as reviewer" took an action on this PR • (03/13/25)

1 reviewer was added to this PR based on Keith Williams's automation.

@socket-security
Copy link
Copy Markdown

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@types/luxon@3.4.2 None 0 120 kB types
npm/memory-cache@0.2.0 None 0 36.7 kB ptarjan
npm/qs-stringify@1.2.1 None 0 5.65 kB goto-bus-stop

View full report↗︎

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 13, 2025

E2E results are ready!

@emrysal emrysal merged commit be40443 into main Mar 13, 2025
60 of 61 checks passed
@emrysal emrysal deleted the bugfix/do-not-self-import-calcom-ui branch March 13, 2025 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core area: core, team members only foundation ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants