search

wernight / docker-mopidy

Containerized Mopidy MPD (Music Player Daemon) along with Spotify, Google Music... plugins.
https://hub.docker.com/r/wernight/mopidy/
MIT License
160 stars 86 forks source link

Fix spotify #16

Closed sammerry closed 5 years ago

sammerry commented 5 years ago

spotify playlists are broken in Mopidy-Spotify. This uses a fixed fork that hasn't been merged.

wernight commented 5 years ago

udpsink branch is for testing and it's almost completely merged into master.

I don't feel very confortable to have an experimental fork on the main. What I suggest is to create a fork/branch to have a stable and latest, or a latest and snapshot. Depending what people generally prefer as the default.

Could also add a branch minimal having only Mopidy as a kind of build base.

Also could you point to the PR of that fix in mopidy-spotify? I kind of see many there https://github.com/mopidy/mopidy-spotify/pulls and I see people replying recently (days ago).

Then you could rebase your changes (preferably in a single commit) on that. What do you think?

sammerry commented 5 years ago

sounds reasonable. I'll pull this PR back then.