Skip to content

chore: merge upstream immich-app/immich (2026-03-10)#64

Merged
Deeds67 merged 13 commits intomainfrom
worktree-upstream-merge
Mar 10, 2026
Merged

chore: merge upstream immich-app/immich (2026-03-10)#64
Deeds67 merged 13 commits intomainfrom
worktree-upstream-merge

Conversation

@Deeds67
Copy link
Collaborator

@Deeds67 Deeds67 commented Mar 10, 2026

Summary

Merges 26 upstream commits from immich-app/immich (2026-03-01 to 2026-03-09).

Key upstream changes

Conflicts resolved

  • server/test/small.factory.ts — upstream refactored test factories into dedicated files (test/factories/). Kept fork-specific SharedSpace and SharedSpaceMember imports and factory methods. Removed stale Stack import (moved to StackFactory).

Fork compatibility fixes

  • Updated factory.asset()AssetFactory.create() and factory.memory()MemoryFactory.create() in fork-specific test code (asset.service.spec.ts, memory.service.spec.ts, plugin.service.spec.ts) to match upstream's new factory pattern.

Fork features verified intact

  • Shared Spaces: schema, services, controllers, DTOs, web routes, mobile pages, E2E tests ✓
  • Storage Migration: service, S3 backend, E2E tests ✓
  • Pet Detection: ML model, service, test suite ✓
  • Image Editing: web components, server extensions, factories ✓
  • Docker: ghcr.io/open-noodle/gallery-* image refs preserved ✓
  • AWS S3 deps: all three @aws-sdk/* packages retained ✓

Test plan

  • CI passes: TypeScript check, lint, unit tests, E2E tests
  • Server unit tests pass with refactored factory imports
  • Fork features (spaces, editing, pet detection) still work end-to-end

🤖 Generated with Claude Code

danieldietzler and others added 9 commits March 10, 2026 20:58
It's not being used right now; can always add it back :P
Videos have recently been changed to support zooming, but this can make
the controls in the centre of the screen unergonomic as they will either
stay in the centre when dismissing, or stick to the video when zooming.
Neither is great. We should align the behaviour with other apps which
has the play/pause toggle at the bottom of the screen with the seeker
bar instead.

Co-authored-by: Alex <alex.tran1502@gmail.com>
immich-app#26807)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
After migrating factory.asset() to AssetFactory.create(), the factory
import from small.factory was no longer used in this file.
@Deeds67 Deeds67 force-pushed the worktree-upstream-merge branch from 737ca77 to e0a1155 Compare March 10, 2026 19:59
@Deeds67 Deeds67 merged commit 163621d into main Mar 10, 2026
10 checks passed
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.

6 participants