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

[chromecast] Added support for next command #11510

Merged
merged 2 commits into from
Dec 16, 2021

Conversation

boc-tothefuture
Copy link
Contributor

[chromecast] Next Command Support

Add support for next command to chromecast binding, this is implemented by seeking to the end of the current track as there is no native next command. This is the same implementation for next that the python library uses for chromecast, which is used by home assistant to achieve this functionality.

@wborn wborn changed the title feat(chromecast): added support for next command [chromecast] Added support for next command Nov 4, 2021
@wborn wborn added the enhancement An enhancement or new feature for an existing add-on label Nov 4, 2021
@lolodomo lolodomo added the awaiting feedback Awaiting feedback from the pull request author label Nov 15, 2021
@lolodomo
Copy link
Contributor

@boc-tothefuture : no time to answer to my comment ?

@boc-tothefuture
Copy link
Contributor Author

@boc-tothefuture : no time to answer to my comment ?

Didn't see the notification. Sorry.

@lolodomo lolodomo removed the awaiting feedback Awaiting feedback from the pull request author label Dec 12, 2021
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@lolodomo
Copy link
Contributor

Please sign property your last commit.

@lolodomo
Copy link
Contributor

Let's make an exception as the initial PR was correctly signed.

@lolodomo lolodomo merged commit edec4c7 into openhab:main Dec 16, 2021
@lolodomo lolodomo added this to the 3.2 milestone Dec 16, 2021
@boc-tothefuture
Copy link
Contributor Author

@lolodomo sorry for the delay.. For some reason I am not seeing notifications from this repo, and only this repo. I will recheck my setting.

NickWaterton pushed a commit to NickWaterton/openhab-addons that referenced this pull request Dec 30, 2021
* feat(chromecast): added support for next command

Signed-off-by: Brian O'Connell <[email protected]>
Signed-off-by: Nick Waterton <[email protected]>
mischmidt83 pushed a commit to mischmidt83/openhab-addons that referenced this pull request Jan 9, 2022
* feat(chromecast): added support for next command

Signed-off-by: Brian O'Connell <[email protected]>
Signed-off-by: Michael Schmidt <[email protected]>
nemerdaud pushed a commit to nemerdaud/openhab-addons that referenced this pull request Jan 28, 2022
* feat(chromecast): added support for next command

Signed-off-by: Brian O'Connell <[email protected]>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
* feat(chromecast): added support for next command

Signed-off-by: Brian O'Connell <[email protected]>
andan67 pushed a commit to andan67/openhab-addons that referenced this pull request Nov 6, 2022
* feat(chromecast): added support for next command

Signed-off-by: Brian O'Connell <[email protected]>
andrasU pushed a commit to andrasU/openhab-addons that referenced this pull request Nov 12, 2022
* feat(chromecast): added support for next command

Signed-off-by: Brian O'Connell <[email protected]>
Signed-off-by: Andras Uhrin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement or new feature for an existing add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants