Skip to content

Feature - Allow sorting of albums by date of included photos.#1525

Open
m-reiter wants to merge 12 commits into
pulsejet:masterfrom
m-reiter:sort_albums_by_photo_date
Open

Feature - Allow sorting of albums by date of included photos.#1525
m-reiter wants to merge 12 commits into
pulsejet:masterfrom
m-reiter:sort_albums_by_photo_date

Conversation

@m-reiter
Copy link
Copy Markdown

@m-reiter m-reiter commented Sep 1, 2025

Added the options to sort albums by date of their newest or oldest photo.

If this actually works, it should fix #1087 and #1123 and be a partial solution for #558.

This is my first PR, so apologies if I did anything wrong here - I'll be happy to learn and correct.

Signed-off-by: m-reiter <m-reiter@users.noreply.github.com>
Signed-off-by: m-reiter <m-reiter@users.noreply.github.com>
…opilot). Fingers crossed.

Signed-off-by: m-reiter <m-reiter@users.noreply.github.com>
Signed-off-by: m-reiter <m-reiter@users.noreply.github.com>
Signed-off-by: m-reiter <m-reiter@users.noreply.github.com>
Signed-off-by: m-reiter <m-reiter@users.noreply.github.com>
…or missing values. Fixed.

Signed-off-by: m-reiter <m-reiter@users.noreply.github.com>
Signed-off-by: m-reiter <m-reiter@users.noreply.github.com>
@m-reiter
Copy link
Copy Markdown
Author

m-reiter commented Sep 9, 2025

@pulsejet I don't want to rush but do you already have an idea whether you'll find time to look at this in the foreseeable future? If there's anything you don't like about it, it would be good to know and maybe try fixing while my memory is still fresh. Thanks!

@pulsejet
Copy link
Copy Markdown
Owner

pulsejet commented Nov 2, 2025

I'm a bit concerned about the performance impact of this. The query needs to be studied carefully for effects on large albums (folks are running this with millions)

@m-reiter
Copy link
Copy Markdown
Author

m-reiter commented Nov 2, 2025

Thanks for looking! If I can do anything to help, let me know.

@fherreror
Copy link
Copy Markdown

Why not add a Date field to album description, manually filled by the user. In this way, it would not have to make a query for each album every time. It would sort all albums by their manually filled description date.
Just an idea. I think this feature would be great and I think this could be an intermediate solution

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sort albums view by its content date, not only album creation date

3 participants