Skip to content

refactor(mobile-manage): use v2 project service for project pinning#3

Open
chennemann wants to merge 1 commit intochore/mobile-manage-projects-ui-local-and-unified-listfrom
chore/mobile-manage-v2-project-service-pinning
Open

refactor(mobile-manage): use v2 project service for project pinning#3
chennemann wants to merge 1 commit intochore/mobile-manage-projects-ui-local-and-unified-listfrom
chore/mobile-manage-v2-project-service-pinning

Conversation

@chennemann
Copy link
Owner

Context:

  • Align Manage project favorite actions with the v2 project model and service layer.

Changes:

  • Add v2 ProjectService with observeProjects and togglePinnedById.
  • Source Manage project list from ProjectService instead of session-layer project state.
  • Switch project favorite toggles to use projectId end-to-end.
  • Wire ProjectService into DI and ManageViewModel.
  • Add and update unit tests for project service and manage viewmodel behavior.

Context:

- Align manage project favorite actions with the v2 project domain model and reduce reliance on session-layer project state.

Changes:

- Added v2 ProjectService with observeProjects and togglePinnedById operations.

- Wired ProjectServiceV2 into DI and injected it into ManageViewModel.

- Updated ManageViewModel to observe projects from ProjectServiceV2 and map pinned to favorite.

- Switched project favorite toggles to use projectId end-to-end.

- Added and updated unit tests for project service and manage viewmodel behavior.
@chennemann chennemann force-pushed the chore/mobile-manage-projects-ui-local-and-unified-list branch from bd0cc77 to ffab288 Compare February 28, 2026 08:05
@chennemann chennemann force-pushed the chore/mobile-manage-v2-project-service-pinning branch from 2733ad9 to 71db5b8 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