Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ability to sort performers by updated_at #866

Merged
merged 1 commit into from
Dec 19, 2024

Conversation

ikmckenz
Copy link
Contributor

@ikmckenz ikmckenz commented Dec 13, 2024

Resolves #830

Quick one line change since the backend infrastructure is already there. Not sure how performant the underlying query will be on a big stashbox, or if it needs an index, but it looks like the other sort options don't currently have indexes other than name.

Copy link
Collaborator

@InfiniteStash InfiniteStash left a comment

Choose a reason for hiding this comment

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

There probably should be an index, but the impact is very small. Thanks!

@InfiniteStash InfiniteStash merged commit 1ebd890 into stashapp:master Dec 19, 2024
3 checks passed
@ikmckenz ikmckenz deleted the sort-performers-updated-at branch December 20, 2024 05:39
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.

Possibility to sort performers with updated_at
2 participants