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

config option for shuffle #797

Closed
SilPan opened this issue Jan 25, 2021 · 5 comments
Closed

config option for shuffle #797

SilPan opened this issue Jan 25, 2021 · 5 comments
Labels
blocked by: librespot Issues that are blocked by librespot's upstream enhancement A new feature that would improve Spotifyd good first issue An easy to implement issue, good for first time contributors or people new to open source help wanted Issues that need help since the assigned person has little to no knowledge about the topic wontfix Issues that will not be fixed under any circumstances

Comments

@SilPan
Copy link

SilPan commented Jan 25, 2021

spotifyd disables shuffle (at least) on every restart.

I usually have shuffle turned on.

So it would be cool if spotifyd could either preserve the status of if shuffle is enabled, or have a config/commandline option to set an initial value if shuffle should be enabled.

From what I have seen documented here https://developer.spotify.com/documentation/web-api/reference/#endpoint-toggle-shuffle-for-users-playback, this should be easy to implement.

I am aware of #626, but the issue got closed and tagged with wontfix due to inactivity, as far as I can see.
It sais something sbout a todo list there?

@SilPan SilPan added the enhancement A new feature that would improve Spotifyd label Jan 25, 2021
@robinvd
Copy link
Contributor

robinvd commented Jan 25, 2021

I see there is currently no option for this on librespot, but it should not be too hard to add

places:
https://github.com/librespot-org/librespot/blob/a2f832dff171914935f7e1a98a5da3d7c8a0b789/protocol/proto/spirc.proto#L34 this is the command to send for shuffle
https://github.com/librespot-org/librespot/blob/a2f832dff171914935f7e1a98a5da3d7c8a0b789/connect/src/spirc.rs#L70 a new variant should be added for shuffle
https://github.com/librespot-org/librespot/blob/a2f832dff171914935f7e1a98a5da3d7c8a0b789/connect/src/spirc.rs#L324 a new helper function for sending the shuffle command
https://github.com/librespot-org/librespot/blob/a2f832dff171914935f7e1a98a5da3d7c8a0b789/connect/src/spirc.rs#L487 handle the new enum variant

and then a config option in spotifyd has to be added, but we can add that if support is added in librespot

@robinvd robinvd added blocked by: librespot Issues that are blocked by librespot's upstream good first issue An easy to implement issue, good for first time contributors or people new to open source help wanted Issues that need help since the assigned person has little to no knowledge about the topic labels Jan 25, 2021
@jackvstrickland
Copy link

I have created a PR for the above work in librespot: librespot-org/librespot#666

@robinvd
Copy link
Contributor

robinvd commented Mar 9, 2021

Looks good! Im not involved in librespot so i cant merge/approve it. But when that is merged/released we can add it to spotifyd!

@sashahilton00
Copy link

sashahilton00 commented May 1, 2021

Merged. Will be available in the next release, I'm hoping to do that this weekend after a couple of weeks of delay.

@stale
Copy link

stale bot commented Jul 30, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix Issues that will not be fixed under any circumstances label Jul 30, 2021
@stale stale bot closed this as completed Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked by: librespot Issues that are blocked by librespot's upstream enhancement A new feature that would improve Spotifyd good first issue An easy to implement issue, good for first time contributors or people new to open source help wanted Issues that need help since the assigned person has little to no knowledge about the topic wontfix Issues that will not be fixed under any circumstances
Projects
None yet
Development

No branches or pull requests

4 participants