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 MQTT state field requirements to wiki for HA #474

Closed
ronschaeffer opened this issue Jun 13, 2019 · 4 comments
Closed

Add MQTT state field requirements to wiki for HA #474

ronschaeffer opened this issue Jun 13, 2019 · 4 comments

Comments

@ronschaeffer
Copy link

Please add to the wiki which MQTT state fields are required for optimum integration with Home Assistant.

I would create a PR, but I'm unsure what it correct. I'm currently using the following, which appears to work properly.

"group_state_fields":[  
      "state",
      "brightness",
      "saturation",
      "color",
      "mode",
      "color_temp",
      "bulb_mode",
      "computed_color",
      "effect",
      "device_id",
      "group_id",
      "device_type"
   ]
@ronschaeffer ronschaeffer changed the title Add state field requirements to wiki for HA Add MQTT state field requirements to wiki for HA Jun 13, 2019
@sidoh
Copy link
Owner

sidoh commented Jun 13, 2019

There should be defaults for this setting, which is why it's not documented on that wiki article (note that they are documented on the README).

However... just tried clearing my config, and there were no defaults set. This was a regression introduced by this commit, which cleaned up some internals.

Will push a fix.

@sidoh
Copy link
Owner

sidoh commented Jun 13, 2019

Will be fixed in 1.9.1.

@sidoh
Copy link
Owner

sidoh commented Jun 13, 2019

Fixed the bug and added a line about making sure that config is set on the wiki page.

Thank you for reporting!

@sidoh sidoh closed this as completed Jun 13, 2019
@ronschaeffer
Copy link
Author

That was quick. Thanks!

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

No branches or pull requests

2 participants