music-assistant / hass-music-assistant

Turn your Home Assistant instance into a jukebox, hassle free streaming of your favorite media to Home Assistant media players.
Apache License 2.0
1.38k stars 52 forks source link

Discover weekly on spotify not available for multiple accounts #2861

Closed happydasch closed 2 months ago

happydasch commented 2 months ago

What version of Music Assistant has the issue?

2.2.2

What version of the Home Assistant Integration have you got installed?

2024.8.4

Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

The problem

When using the spotify provider for multiple accounts, the special playlist discover weekly is only available from the first account. At the bottom of the playlist screen, both playlists are listed. Seems to be the same as #2206

How to reproduce

Add the discover weekly playlist from multiple spotify accounts, only the first spotify account will be used

Music Providers

Spotify

Player Providers

Sonos

Full log output

No response

Additional information

No response

What version of Home Assistant Core are your running

2024.8.4

What type of installation are you running?

Home Assistant OS

On what type of hardware are you running?

Generic x86-64 (e.g. Intel NUC)

OzGav commented 2 months ago

At the bottom of the playlist screen, both playlists are listed

Please supply a screenshot of what you mean here

happydasch commented 2 months ago
Bildschirmfoto 2024-09-03 um 07 48 30

This is the discover weekly playlist which is generated for a user. Both playlists, mine and my wifes, are shown as sources at the bottom but the songs only come from the first playlist, the second is ignored.

marcelveldt commented 2 months ago

please test if this is fixed in 2.3.0b18

marcelveldt commented 2 months ago

You may have to remove the playlist from the library once to trigger the fix

happydasch commented 2 months ago

Tested this, works well. Now the discover weekly appears twice with the correct songs for every account.

marcelveldt commented 2 months ago

Tested this, works well. Now the discover weekly appears twice with the correct songs for every account.

We didnt release it yet so you couldnt test it yet

happydasch commented 2 months ago

tested using the dev release of music assistant, like i said, the fix works well, both playlists are now seperated

marcelveldt commented 2 months ago

tested using the dev release of music assistant, like i said, the fix works well, both playlists are now seperated

ah ok that makes sense - thanks for testing it on the dev build then!