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 Google Assistant garage type #23115

Merged
merged 5 commits into from Apr 15, 2019
Merged

Add Google Assistant garage type #23115

merged 5 commits into from Apr 15, 2019

Conversation

giefca
Copy link
Contributor

@giefca giefca commented Apr 15, 2019

Description:

Add new undocumented device types in google assistant. Test with garage type. It only change the blind icon in the app for a generic icon and the type of the device. This doesn't add or substrat any functionality.

Related issue (if applicable): fixes #23076

Checklist:

  • The code change is tested and works locally.
  • Local tests pass with tox. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.

If the code does not interact with devices:

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

@codecov
Copy link

codecov bot commented Apr 15, 2019

Codecov Report

Merging #23115 into dev will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev   #23115      +/-   ##
==========================================
+ Coverage   94.09%   94.09%   +<.01%     
==========================================
  Files         452      452              
  Lines       36795    36802       +7     
==========================================
+ Hits        34623    34630       +7     
  Misses       2172     2172
Impacted Files Coverage Δ
...ssistant/components/google_assistant/smart_home.py 93.33% <100%> (+0.22%) ⬆️
homeassistant/components/google_assistant/const.py 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f89f88...4a3477f. Read the comment docs.

@ghost ghost assigned balloob Apr 15, 2019
@balloob balloob merged commit 497038b into home-assistant:dev Apr 15, 2019
@ghost ghost removed the in progress label Apr 15, 2019
elupus pushed a commit to elupus/home-assistant that referenced this pull request Apr 16, 2019
* Tests

* Add Google Assistant GARAGE type

* Update test_trait.py

* Key device class by domain

* Update smart_home.py
@joshtbernstein
Copy link

Will this change prevent Google from opening my garage door when I say "Hey Google, open all of the shades"? I was about to file an issue on that, but looks like this may resolve that?

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

Successfully merging this pull request may close these issues.

Add more Google Assistant device types
5 participants