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

add support for mutiple commands in mqtt message #55

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mhdawson
Copy link

harmony-api is a great project. I use mqtt to glue together a lot
of different home automation pieces and it let me quickly add the
ability to connect/issue commands to my tv setup.

I'm current working on Amazon Alexa control for some additional
features (set channel, volume up/down, mute etc.) that don't
seem to be built in with the existing logitech/alexa integration. The
project is https://github.com/mhdawson/AlexaMqttBridge (still a work in
progress so no README.md etc. yet).

Along similar lines to the existing support to be able repeat a command
multiple times, this adds the ability to specify multiple commands in a
single mqtt message. This allows me to create a single mqtt message
from Alexa that changes the TV to a particular channel. For example, by
posting "1 7" on the appropriate mqtt topic the commands for 1 and
7 are sent, changing the channel to 17.

It looks like there are no spaces in the slugs (at least for my devices)
so using spaces to separate the commands seemed safe. If you
think there is a better choice, just let me know and I'll update it.

Along similar lines to the existing support to be
able repeat a command multiple times, this adds
the ability to specify multiple commands in a
single mqtt messasge. Commands are space separated.
@mhdawson
Copy link
Author

bump

@mhdawson
Copy link
Author

@maddox just wondering if you've had a chance to take a look at this ?

@mhdawson
Copy link
Author

@maddox still around ?

@maddox
Copy link
Owner

maddox commented Mar 31, 2017

👋

In order to merge this, it'll need to have the same sort of support for the HTTP api as well. It'll need to be documented in the README too.

Thanks!

@mhdawson
Copy link
Author

mhdawson commented May 31, 2017

Sorry missed this comment. Now that I know it is possible to get it in, I'll update to address your suggestions.

@mhdawson
Copy link
Author

Still on my radar just pretty busy these days.

@mhdawson
Copy link
Author

I know its been a long time. Hope to get to this over the holidays

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants