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

Expose playlist metadata in playlist response object #106

Closed
Drapersniper opened this issue May 25, 2023 · 5 comments
Closed

Expose playlist metadata in playlist response object #106

Drapersniper opened this issue May 25, 2023 · 5 comments

Comments

@Drapersniper
Copy link

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
Copy link
Owner

Walkyst commented May 25, 2023

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

@Drapersniper
Copy link
Author

Drapersniper commented May 25, 2023

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
Copy link
Contributor

topi314 commented May 25, 2023

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
Copy link
Owner

Walkyst commented May 25, 2023

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

@topi314
Copy link
Contributor

topi314 commented May 25, 2023

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

@Walkyst Walkyst closed this as completed Jan 23, 2024
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

No branches or pull requests

3 participants