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

Adds ability to configure sort order for DLNA videos #3645

Merged
merged 8 commits into from
May 3, 2023

Conversation

cj12312021
Copy link
Collaborator

@cj12312021 cj12312021 commented Apr 6, 2023

This pull request adds the ability to configure the default sort order videos I'm DLNA.

Fixes #2166.

@cj12312021 cj12312021 added feature Pull requests that add a new feature ui Issues related to UI backend Pull requests that update Go code labels Apr 6, 2023
@cj12312021
Copy link
Collaborator Author

cj12312021 commented Apr 6, 2023

Once saved filters are integrated into DLNA, I imagine the breakdown would look like this:

Scenes
 |- All
 |- Saved Filter 1
 |- Saved Filter 2
Movies
 |- All
     |- Movie 1
     |- Movie 2
 |- Saved Filter 1
     ....
 |- Saved Filter 2
     ....
Performers
 |- All
     |- Performer 1
     |- Performer 2
 |- Saved Filter 1
      ....
 |- Saved Filter 2
      ....
Studios
 |- All
     |- Studio 1
     |- Studio 2
 |- Saved Filter 1
      ....
 |- Saved Filter 2
      ....
Tags
 |- All
     |- Tag 1
     |- Tag 2
 |- Saved Filter 1
      ....
 |- Saved Filter 2
      ....

When this work is done, the save filter sort order would clearly override the default sort order for videos under the scenes saved filter folders. The default sort order would come into play for videos under folders such as movies, performers, studios, and tags since the saved filters would only apply to the order of the movies, performers, studios, and tags displayed and not the videos under each of those folders.

Copy link
Collaborator

@kermieisinthehouse kermieisinthehouse left a comment

Choose a reason for hiding this comment

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

Code reviewed but not tested.

@WithoutPants WithoutPants added this to the Version 0.21.0 milestone May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Pull requests that update Go code feature Pull requests that add a new feature ui Issues related to UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] Add DLNA "Random" and "Last Added (Updated at)" categories
3 participants