Skip to content

Conversation

pedrosimao
Copy link
Contributor

@pedrosimao pedrosimao commented Sep 3, 2025

Description

What:
Adds the option to add a selection of photos to an Album when picking items from existing Albums. I have implemented the feature in both legacy and beta timelines.

Why:
This feature currently exists in the web app and is missing in the mobile app. It is very useful because selecting photos from the main "Photos" timeline tab can be a lot of work. For instance people who sync Instagram, Whatsapp and Screenshots albums, will have everything mixed on the main timeline/tab, while the "Recents" album on the "Albums" tab only have photos that were taken with the phone camera. Other than that users can always want to create/improve albums out of other existing albums.

Github Issue:
I did not find a Github issue. But I just had my girlfriend complaining that she does not use the Immich app to share photos with me, because she finds it too time consuming to have to deselect photos from the timeline, before adding to our shared Albums. So, I am basically fixing her complaints. Also, it adds feature parity with web app, so it should be intuitive to web app users anyway. It's not a weird flow that people are not used to already.

How Has This Been Tested?

  • Everything was tested on an iOS simulator (iPhone 16 / iOS 18.6)
  • I have checked that the option appears on the bottom menu of an Album when selecting pictures (legacy timeline)
  • I have checked that the option appears on the bottom menu of an Album when selecting pictures (beta timeline)
  • I have tested adding pictures to an album that were not already added and it gives the correct success message
  • I have tested adding pictures to an album that were already added and it gives the correct error message
Simulator Screenshot - iPhone 16 - 2025-09-03 at 10 43 21 Simulator Screenshot - iPhone 16 - 2025-09-03 at 10 43 24 Simulator Screenshot - iPhone 16 - 2025-09-03 at 10 43 27 Simulator Screenshot - iPhone 16 - 2025-09-03 at 10 43 32 Simulator Screenshot - iPhone 16 - 2025-09-03 at 10 44 25

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/)

@pedrosimao
Copy link
Contributor Author

quick update, it seems I missed running the format script. I added a commit with the format (which added one empty line).

@alextran1502 alextran1502 merged commit b82e29f into immich-app:main Sep 4, 2025
43 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.

3 participants