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 playlist create/edit support to the Subsonic provider #1335

Merged
merged 3 commits into from
Jun 9, 2024

Conversation

khers
Copy link
Contributor

@khers khers commented Jun 9, 2024

The first commit adds the ability to edit playlists from MA, partially addressing the feature request in https://github.com/orgs/music-assistant/discussions/2358 favorite syncing will come later.

The second commit explicitly sets the favorite field in the album and artists constructors to the value in the provider.

khers added 3 commits June 9, 2024 18:07
This is to be used when a provider rejects a request because of
permissions.

Signed-off-by: Eric B Munson <eric@munsonfam.org>
Editting playlists is easily supported by the API so add the plumbing to
allow the user to do it in MA.

Signed-off-by: Eric B Munson <eric@munsonfam.org>
This filed is supported for artists and albums but the song/track entity
is not complete in the spec. Tracks will be updated when the spec is
complete.

Signed-off-by: Eric B Munson <eric@munsonfam.org>
Copy link
Member

@marcelveldt marcelveldt left a comment

Choose a reason for hiding this comment

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

Nice work @khers !

@marcelveldt marcelveldt merged commit 8898ca5 into music-assistant:main Jun 9, 2024
4 of 5 checks passed
@marcelveldt marcelveldt changed the title Two updates to the Subsonic provider Add playlist create/edit support to the Subsonic provider Jun 9, 2024
@khers khers deleted the edit-playlists branch June 23, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants