Skip to content

refactor(mobile-v2-sync): delegate sync domain logic to project and session services#4

Open
chennemann wants to merge 1 commit intochore/mobile-manage-v2-project-service-pinningfrom
chore/mobile-v2-sync-delegate-project-session-services
Open

refactor(mobile-v2-sync): delegate sync domain logic to project and session services#4
chennemann wants to merge 1 commit intochore/mobile-manage-v2-project-service-pinningfrom
chore/mobile-v2-sync-delegate-project-session-services

Conversation

@chennemann
Copy link
Owner

Context:

  • Reduce SynchronizationService to orchestration and move domain persistence rules into dedicated v2 services.

Changes:

  • Add project-domain sync API in ProjectService and return synced projects.
  • Add session-domain per-project sync API in SessionService with explicit params.
  • Refactor SynchronizationService to delegate project sync, then session sync per returned project.
  • Update DI wiring for new service constructors and synchronization dependencies.
  • Rework v2 tests to validate delegation and move persistence assertions to service-specific tests.

…ession services

Context:

- Reduce SynchronizationService to orchestration and move domain persistence rules into dedicated v2 services.

Changes:

- Added project-domain sync API in ProjectService and made it return synced projects.

- Added session-domain per-project sync API in SessionService with explicit params.

- Refactored SynchronizationService to delegate project sync then call session sync per returned project.

- Updated DI wiring for the new service constructors and synchronization dependencies.

- Reworked v2 tests to validate delegation and moved persistence assertions to service-specific tests.
@chennemann chennemann force-pushed the chore/mobile-manage-v2-project-service-pinning branch from 2733ad9 to 71db5b8 Compare February 28, 2026 08:05
@chennemann chennemann force-pushed the chore/mobile-v2-sync-delegate-project-session-services branch from 292f940 to f370a43 Compare February 28, 2026 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant