redxef/jellyfin-smart-playlist
1
0
Fork 0
Code Issues 3 Pull requests Projects Releases 12 Packages Wiki Activity Actions

Think about how different item kinds should be handled. #2

New issue
Closed
opened 2025-01-09 18:09:20 +01:00 by redxef · 1 comment
redxef commented 2025-01-09 18:09:20 +01:00
Owner
Copy link

Currently items can be BaseItemKind.Audio, BaseItemKind.MusicAlbum and BaseItemKind.Playlist.

This is cumbersome, since nearly every smart playlist has to specifically exclude other playlists. Additionally more kinds of items would be nice.

I have two ideas how to proceed:

  1. Keep the implementation as is and just add more BaseItemKinds, maybe all
  2. A select multiple field like for users.
Currently items can be `BaseItemKind.Audio`, `BaseItemKind.MusicAlbum` and `BaseItemKind.Playlist`. This is cumbersome, since nearly every smart playlist has to specifically exclude other playlists. Additionally more kinds of items would be nice. I have two ideas how to proceed: 1. Keep the implementation as is and just add more BaseItemKinds, maybe all 2. A `select multiple` field like for users.
redxef added the
enhancement
 label 2025-01-09 18:09:21 +01:00
redxef added reference enable-all-item-kinds 2025-01-16 18:56:42 +01:00
redxef commented 2025-01-16 19:00:28 +01:00
Author
Owner
Copy link

Since the official recommendation is to always specify the kind in the filter I'm for now fine with option 1. Lets see how we fare with this. I can see performance becoming an issue though.

Since the official recommendation is to always specify the kind in the filter I'm for now fine with option 1. Lets see how we fare with this. I can see performance becoming an issue though.
Sign in to join this conversation.
enable-all-item-kinds
Branches Tags
Clear current reference
main
manifest
collections
enable-all-item-kinds
yaml
Clear current reference
v0.5.3.0
v0.5.2.0
v0.5.1.0
v0.5.0.0
v0.4.1.0
v0.4.0.0
v0.3.0.0
v0.2.2.0
v0.2.1.0
v0.2.0.0
v0.1.1.0
v0.1.0.0
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
wontfix

This won't be fixed

No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: redxef/jellyfin-smart-playlist#2
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?