-
Notifications
You must be signed in to change notification settings - Fork 15
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
[Feature Request] Next/Previous Track Action #13
Comments
Should be doable… let me peek at the official Mopidy API. |
YAY… Should be really simple, as it's the same playback controller class as the https://docs.mopidy.com/en/latest/api/core/#mopidy.core.PlaybackController.next |
I'll give it a try with a PR in the next days 👍 |
No worries, I already have it :) |
I'll probably refactor the code a bit to keep compliant with DRY |
I created a new PR for my new What I can say so far:
Again: I only changed the code on the fly without testing. I'd be surprised if that change already works ^^ |
Also refactor the code to get rid of separate action functions & tag classes. Now there's only action classes. See also #13
@tobru I created a new release. You can use the tag Can you test if everything works as expected and let me know? :) |
@domibarton I can confirm that it just works ™️ 🎉 I'm even impressed that you could read my mind about the new unregister tag possibility which was a thing I was thinking about. Thank you very much, I owe you a 🍺 once it's possible again! |
lolz :D after the corona thingy, we'll have some 🍻 ;) Yeah the "unregister" feature was in my mind for some time. But I'm a lazy person, so it took some time. Glad it worked :) |
When working with playlists it would also be great to have a tag to skip forward to the next track or back to the previous one in the playlist. This "Class" is currently not available and it would be fantastic to have it available.
The text was updated successfully, but these errors were encountered: