Skip to content

deCONZ - Option to load or not to load clip sensors on start#14480

Merged
balloob merged 5 commits intohome-assistant:devfrom
Kane610:deconz-allow-ignore-clip-sensors
May 29, 2018
Merged

deCONZ - Option to load or not to load clip sensors on start#14480
balloob merged 5 commits intohome-assistant:devfrom
Kane610:deconz-allow-ignore-clip-sensors

Conversation

@Kane610
Copy link
Copy Markdown
Member

@Kane610 Kane610 commented May 15, 2018

Description:

If using zigbee groups to control lights with sensors, deCONZ will create clip sensors, these sensors are of no relevance to home assistant, but users can also create and use their own sensors so there still needs to be an option to allow or ignore these sensors.

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass

If the code does not interact with devices:

  • Tests have been added to verify that the new code works.

@Kane610 Kane610 changed the title WIP: deCONZ - Option to load or not to load clip sensors on start deCONZ - Option to load or not to load clip sensors on start May 16, 2018
@Kane610
Copy link
Copy Markdown
Member Author

Kane610 commented May 18, 2018

@balloob are you ok with exposing options through config entries like this?

@balloob
Copy link
Copy Markdown
Member

balloob commented May 29, 2018

Yes, that's fine. In the future I also want to implement a way for users to just trigger an options config flow for components.

@balloob balloob merged commit 8c93b48 into home-assistant:dev May 29, 2018
@Kane610
Copy link
Copy Markdown
Member Author

Kane610 commented May 30, 2018

@balloob we talked about that a few weeks back, I'm all for that 👍

@Kane610 Kane610 deleted the deconz-allow-ignore-clip-sensors branch May 30, 2018 16:37
@balloob balloob mentioned this pull request Jun 8, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Sep 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants