Skip to content

feat(mobile): sort users alphabetically#27906

Open
OdinOxin wants to merge 2 commits intoimmich-app:mainfrom
OdinOxin:feat/sort-users-alphabetically
Open

feat(mobile): sort users alphabetically#27906
OdinOxin wants to merge 2 commits intoimmich-app:mainfrom
OdinOxin:feat/sort-users-alphabetically

Conversation

@OdinOxin
Copy link
Copy Markdown
Contributor

@OdinOxin OdinOxin commented Apr 17, 2026

Description

Now displaying users in drift_user_selection and drift_album_options in alphabetically order, as already implemented in Web (#27731).
For large immich-instances it is now easier to find a specific user to share an album with.

Fixes # (issue)

How Has This Been Tested?

  • Connected Debug-App to public Immich-Demo-instance and opened the share page for an album.
  • Connected with own Immich-Instance and opened the album options page for an shared album.

Screenshots (if appropriate)

Unsorted

Unsorted

Sorted

Sorted

Checklist:

  • I have carefully read CONTRIBUTING.md
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable (not mentioned in documentation, therefore no changes were made)
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary. (none new dependencies)
  • I have written tests for new code (if applicable) (not 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. (unchanged)
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/) (unchanged)

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

@OdinOxin OdinOxin marked this pull request as ready for review April 17, 2026 18:49
@OdinOxin OdinOxin changed the title sort users alphabetically feat/sort users alphabetically Apr 17, 2026
@OdinOxin OdinOxin changed the title feat/sort users alphabetically feat(mobile): sort users alphabetically Apr 17, 2026
Copy link
Copy Markdown
Member

@shenlong-tanwen shenlong-tanwen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We want the users list to be sorted ideally in the SQL query. Can you update the query with the sort instead?

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