search

sgbasaraner / yeelight-cli

a low level, cross-platform command line client for Yeelight
MIT License
55 stars 10 forks source link

Methods without parameters require a space character to run correctly #2

Closed mtardal closed 6 years ago

mtardal commented 6 years ago

Methods like "toggle", "set_default" which don't have any parameters, require a space character at the end of the command to work, otherwise the bulb returns "invalid command" message.

Thanks for this sick lil wrapper by the way. I had some ideas about Yeelight but was too lazy to start as the existing sources were mostly in Chinese, this will be an useful base for me to build on top of.

mtardal commented 6 years ago

I looked at the source code and I think the source stems from parsing of the parameters when there isn't any parameters. I can fix it if you want.

sgbasaraner commented 6 years ago

Thank you for the notification. Yes, I think your assessment is correct. I'm waiting for a pull request.