feat: add smart_shuffle parameter to CurrentlyPlayingContext (#430) #432
Conversation
|
Oh, it's my first pull request here and I just saw the contributing file. |
|
Yep, you're good! Creating another branch is just a suggestion for you to be able to create multiple pull requests, each with their own branch, instead of changing the main branch in your fork that you may want to synchronize with the upstream's main branch eventually. |
|
btw, you can link PRs to issues by using certain keywords on GitHub. That way the issue will be automatically closed once the PR is merged to the main branch. Just another idea though, I just linked the issue manually :) |
dargmuesli
left a comment
dargmuesli
left a comment
There was a problem hiding this comment.
Thank you for adding this!
I just aligned the order of code lines with the order of properties that is returned by Spotify's API, so it's easy to compare if anything's missing.
2 participants
I added smart_shuffle to CurrentlyPlayingContext and to the files for the tests.