Skip to content

Conversation

jamarzka
Copy link

@jamarzka jamarzka commented Oct 5, 2025

Description

  • Added a new orderBy option to the search metadata route
  • Added a new utility to show assets sorted by asset.createdAt

I'm bulk uploading old photos and want to be able to review them without having to assign them to a temporary album.

Fixes #8716

How Has This Been Tested?

Tested on my local dev instance

  • Uploaded photos in a random date order and confirmed they were displayed in the upload order rather than the date the photo was taken.
  • Manually tested the selection appbar buttons and menu options.

Screenshots

image image

API Changes

Added a new orderBy option to the search metadata endpoint.

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

Please describe to which degree, if any, an LLM was used in creating this pull request.

A LLM was not used at all for this

@adrianjost
Copy link
Contributor

That's nice, I think this added sort would also make implementing #10365 much easier.

@jamarzka
Copy link
Author

jamarzka commented Oct 5, 2025

That's nice, I think this added sort would also make implementing #10365 much easier.

Trash uses the timeline bucket endpoints so the API changes made here wouldn't directly help. I did an initial proof of concept using the timeline but ran into some issues. I'll take another look now that I've spent some time in the code.

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