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

Backport playlist refresh #385

Merged
merged 6 commits into from
Apr 12, 2024

Conversation

kingosticks
Copy link
Member

No description provided.

@kingosticks kingosticks self-assigned this Apr 12, 2024
djmattyg007 and others added 3 commits April 12, 2024 13:43
This change prevents loading of Spotify playlists from blocking other
common Mopidy operations. These include startup and browsing, as well as
refreshing of playlists in general. Loading Spotify playlists will
always happen in a dedicated thread, and only one refresh thread will
run at a time.

The playlists_loaded event is manually re-dispatched when Spotify is
done loading playlists, as a signal that clients should refresh their
list of playlists. This is necessary, as otherwise clients relying
solely on websocket notifications may never discover the newly-loaded
playlists.
@kingosticks kingosticks force-pushed the port-playlist-refresh branch from b908950 to 9624a47 Compare April 12, 2024 12:50
@kingosticks kingosticks force-pushed the port-playlist-refresh branch from 9624a47 to 4ed6a24 Compare April 12, 2024 13:11
@kingosticks kingosticks merged commit 1b93bd3 into mopidy:v5.0.0alpha Apr 12, 2024
5 checks passed
@kingosticks kingosticks deleted the port-playlist-refresh branch April 12, 2024 13:14
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

Successfully merging this pull request may close these issues.

2 participants