Skip to content

refactor: remove 10 trpc imports from features package by moving 16 tRPC-driven components to apps/web/modules#27336

Merged
keithwillcode merged 16 commits intomainfrom
devin/1769602252-integration-attribute-sync-migration
Jan 30, 2026
Merged

refactor: remove 10 trpc imports from features package by moving 16 tRPC-driven components to apps/web/modules#27336
keithwillcode merged 16 commits intomainfrom
devin/1769602252-integration-attribute-sync-migration

Conversation

@hbjORbj
Copy link
Contributor

@hbjORbj hbjORbj commented Jan 28, 2026

Summary

Moves 16 tRPC-driven UI components from packages/features to apps/web/modules, removing 10 @calcom/trpc imports from the features package.

Impact

Files Moved

Component New Location
Segment.tsx + test apps/web/modules/event-types/components/
BookerLayoutSelector.tsx apps/web/modules/settings/components/
CreateLicenseKeyForm.tsx apps/web/modules/ee/deployment/licensekey/
useBookingLocation apps/web/modules/bookings/hooks/
useBookerUrl apps/web/modules/bookings/hooks/
blocklist.tsx apps/web/modules/ee/admin/pages/settings/
BulkDeleteBlocklistEntries.tsx apps/web/modules/ee/admin/pages/settings/blocklist/
BulkDismissReports.tsx apps/web/modules/ee/admin/pages/settings/blocklist/
EmbedTabs.tsx apps/web/modules/embed/components/
ThemeLabel.tsx apps/web/modules/settings/components/
SectionBottomActions.tsx apps/web/modules/settings/components/
SettingsHeader.tsx apps/web/modules/settings/components/
SettingsHeaderWithBackButton.tsx apps/web/modules/settings/components/

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A - no documentation changes needed.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works. Existing tests cover these components.

…om packages/features to apps/web/modules

Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

devin-ai-integration bot and others added 3 commits January 28, 2026 12:23
Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
…ges/features to apps/web/modules

Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
…ps/web/modules

Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
@pull-request-size pull-request-size bot added size/M and removed size/S labels Jan 28, 2026
@pull-request-size pull-request-size bot added size/L and removed size/M labels Jan 30, 2026
@devin-ai-integration devin-ai-integration bot changed the title refactor: move Segment, BookerLayoutSelector, CreateLicenseKeyForm from packages/features to apps/web/modules refactor: remove 10 trpc imports from features package by moving 16 tRPC-driven components to apps/web/modules Jan 30, 2026
@hbjORbj hbjORbj marked this pull request as ready for review January 30, 2026 07:36
@hbjORbj hbjORbj requested review from a team as code owners January 30, 2026 07:36
@graphite-app graphite-app bot added foundation core area: core, team members only labels Jan 30, 2026
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 107 files

Note: This PR contains a large number of files. cubic only reviews up to 75 files per PR, so some files may not have been reviewed.

@keithwillcode keithwillcode merged commit b3430c8 into main Jan 30, 2026
59 checks passed
@keithwillcode keithwillcode deleted the devin/1769602252-integration-attribute-sync-migration branch January 30, 2026 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants