refactor: move data-table, apps, organizations, and workflows hooks from packages/features to apps/web/modules #27228
+16
−20
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR is part of a larger effort to break down PR #26571 into smaller, reviewable chunks. It moves several hooks and stores from
packages/featurestoapps/web/modulesto support the vertical slice architecture.Files moved:
useSegments→apps/web/modules/data-table/hooks/useAppsData→apps/web/modules/apps/hooks/useAgentsData→apps/web/modules/ee/workflows/hooks/onboardingStore(+ test) →apps/web/modules/ee/organizations/lib/useWelcomeModal→apps/web/modules/ee/organizations/hooks/All import paths across 22 consuming files have been updated accordingly.
@calcom/features[3] #26571Updates since last revision
useSegments.tsimports to use absolute paths (@calcom/features/data-table/lib/...) instead of relative paths, since the relative paths no longer resolve after moving the file to its new location.Mandatory Tasks (DO NOT REMOVE)
How should this be tested?
This is a pure refactoring PR with no functional changes. Testing should verify:
yarn type-check:ci --forceyarn lintItems for reviewer attention
packages/platform/atoms/event-types/wrappers/EventPaymentsTabPlatformWrapper.tsxnow imports from@calcom/web/modules. Please verify this doesn't create issues if platform atoms is used independently.useSegments.tshook still imports utilities from@calcom/features/data-table/lib/since only the hook was moved, not the underlying lib files.Checklist
Link to Devin run: https://app.devin.ai/sessions/22cb14b2f6554594b1e738631aa29184
Requested by: @hbjORbj