Skip to content

fix(web): stack updates require unstack-restack #20348

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

xCJPECKOVERx
Copy link
Collaborator

@xCJPECKOVERx xCJPECKOVERx commented Jul 28, 2025

Description

Adds a new action to refresh a stack (change primary asset, remove asset from). Optionally will add a primary asset to the timeline manager (needed for change-primary-asset). Then updates the stack asset and makes sure all non-primary assets are removed from timeline.

Suggested by jrasm in #19703

How Has This Been Tested?

Create stack then change primary asset or remove assets and verify the stack and counts are correct, and any unstacked assets are now in the timeline

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services/ uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/)

   - optionally adds a primary asset
   - updates the asset and its stack
   - removes the non-primary assets
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