Skip to content

Conversation

@hbjORbj
Copy link
Contributor

@hbjORbj hbjORbj commented Jan 26, 2026

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/features to apps/web/modules to support the vertical slice architecture.

Files moved:

  • useSegmentsapps/web/modules/data-table/hooks/
  • useAppsDataapps/web/modules/apps/hooks/
  • useAgentsDataapps/web/modules/ee/workflows/hooks/
  • onboardingStore (+ test) → apps/web/modules/ee/organizations/lib/
  • useWelcomeModalapps/web/modules/ee/organizations/hooks/

All import paths across 22 consuming files have been updated accordingly.

Updates since last revision

  • Fixed useSegments.ts imports 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)

  • 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 hooks.

How should this be tested?

This is a pure refactoring PR with no functional changes. Testing should verify:

  1. Run type checking: yarn type-check:ci --force
  2. Run lint: yarn lint
  3. Verify the app builds successfully

Items for reviewer attention

  1. Platform atoms dependency: packages/platform/atoms/event-types/wrappers/EventPaymentsTabPlatformWrapper.tsx now imports from @calcom/web/modules. Please verify this doesn't create issues if platform atoms is used independently.
  2. useSegments still depends on features package: The moved useSegments.ts hook still imports utilities from @calcom/features/data-table/lib/ since only the hook was moved, not the underlying lib files.

Checklist

  • My code follows the style guidelines of this project
  • I have checked if my changes generate no new warnings
  • My PR is appropriately sized (29 files, mostly import path updates)

Link to Devin run: https://app.devin.ai/sessions/22cb14b2f6554594b1e738631aa29184
Requested by: @hbjORbj

…rom 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

Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants