Skip to content

Add color_status sensor for Nest Protect #14868

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

Merged

Conversation

awarecan
Copy link
Contributor

@awarecan awarecan commented Jun 8, 2018

Description:

Add color status sensor for Nest Protect. Address feature request from https://community.home-assistant.io/t/nest-protect-ui-light-color-motion-sensor/8781
Nest API ref: https://developers.nest.com/documentation/cloud/api-smoke-co-alarm#ui_color_state

Related issue (if applicable): fixes #

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.io#5506

Example entry for configuration.yaml (if applicable):

Checklist:

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

If user exposed functionality or configuration variables are added/changed:

@ghost ghost added in progress and removed small-pr PRs with less than 30 lines. labels Jun 8, 2018
@awarecan awarecan closed this Jun 8, 2018
@ghost ghost removed the in progress label Jun 8, 2018
@awarecan awarecan reopened this Jun 8, 2018
@ghost ghost added the in progress label Jun 8, 2018
@syssi syssi merged commit d3d9d9e into home-assistant:dev Jun 9, 2018
@ghost ghost removed the in progress label Jun 9, 2018
@awarecan awarecan deleted the add-nest-smoke-co-alarm-color-sensor branch June 19, 2018 01:54
@balloob balloob mentioned this pull request Jun 22, 2018
girlpunk pushed a commit to girlpunk/home-assistant that referenced this pull request Sep 4, 2018
@home-assistant home-assistant locked and limited conversation to collaborators Dec 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants