Walkyst / lavaplayer-fork

Apache License 2.0
173 stars 68 forks source link

Expose playlist metadata in playlist response object #106

Closed Drapersniper closed 7 months ago

Drapersniper commented 1 year ago

Exposing things such as type, url, author, artwork for sources that support it would be a really nice addition.

Lavascr already does a this for it's playlists inside the pluginInfo object but it would be nice seeing a more wide support for all sources natively.

Walkyst commented 1 year ago

Did you check readme? You can use custom branch for artworks

Drapersniper commented 1 year ago

This is explicitly for Playlists objects, not the tracks; currently, the artwork is exposed only to Tracks, I was talking to @TopiSenpai about it, and they asked me to open a tracking issue for it.

topi314 commented 1 year ago

The idea is to extend the current AudioPlaylist interface with new fields like url type (playlist, album, top tracks, etc) artworkUrl

These fields would all be nullable

this change could be done backwards compatible via default methods in the interface

Walkyst commented 1 year ago

url and artworkUrl shouldn't be a problem but not sure about type and it's purpose

topi314 commented 1 year ago

sources like spotify, deezer & co don't only have playlists, but also albumns having that metadata somewhere would be nice so you can say Loaded Album x, Loaded Top Songs x instead of Loaded Playlist x for everything

playlist author would also be interesting to output