Skip to content

Comments

refactor: move AdditionalCalendarSelector, TeamsFilter, FilterResults from packages/features to apps/web/modules#27338

Closed
hbjORbj wants to merge 0 commit intomainfrom
devin/1769603552-booker-hooks-migration
Closed

refactor: move AdditionalCalendarSelector, TeamsFilter, FilterResults from packages/features to apps/web/modules#27338
hbjORbj wants to merge 0 commit intomainfrom
devin/1769603552-booker-hooks-migration

Conversation

@hbjORbj
Copy link
Contributor

@hbjORbj hbjORbj commented Jan 28, 2026

What does this PR do?

This PR is part of the effort to break down #26571 into smaller, reviewable PRs.

Moves filter and calendar selector components from packages/features/ to apps/web/modules/ and updates all import paths in consuming files.

Files moved:

  • AdditionalCalendarSelector.tsxapps/web/modules/calendars/components/
  • TeamsFilter.tsxapps/web/modules/filters/components/
  • FilterResults.tsxapps/web/modules/filters/components/

Import updates in:

  • apps/web/app/(use-page-wrapper)/apps/routing-forms/forms/[[...pages]]/Forms.tsx
  • apps/web/modules/calendars/components/SelectedCalendarsSettingsWebWrapper.tsx
  • apps/web/modules/ee/workflows/views/WorkflowsPage.tsx
  • apps/web/modules/insights/components/filters/OrgTeamsFilter.tsx

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - No documentation changes needed for file moves.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

This is a pure file move with import path updates. CI type checks and unit tests should verify correctness.

  1. Type checks should pass (verifies all imports are correct)
  2. Unit tests should pass (verifies components work correctly from new locations)

Human Review Checklist

  • Verify all files importing these components have been updated
  • Confirm no circular dependencies are introduced (these components are only imported by apps/web)

Checklist

  • My code follows the style guidelines of this project
  • I have checked if my changes generate no new warnings
  • My PR is not too large

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

@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 devin-ai-integration bot force-pushed the devin/1769603552-booker-hooks-migration branch from c99c5fe to 5bea18f Compare January 28, 2026 12:43
@devin-ai-integration devin-ai-integration bot changed the title refactor: move calendars DatePicker and NoAvailabilityDialog from packages/features to apps/web/modules refactor: move AdditionalCalendarSelector, TeamsFilter, FilterResults from packages/features to apps/web/modules Jan 28, 2026
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.

1 participant