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 SmartThings Climate platform #20963

Merged

Conversation

@andrewsayre
Copy link
Member

andrewsayre commented Feb 11, 2019

Description:

Adds SmartThings Climate platform that represents SmartThings devices that have thermostat-related capabilities.

Highlights:

  • Supports thermostats implementing both legacy and current SmartThings capabilities with climate features: operation mode, operating state (state attribute), fan mode, target temp high, target temp low and humidity (state attribute)
  • Climate entities are automatically if supported by the device.
  • Users will need to re-authorize the Home Assistant Automation in the SmartThings Classic mobile app if they have previously used this component to begin receiving push updates for this platform. This is completed by navigating to Automations -> Home Assistant -> Done -> Allow

Pull request in home-assistant.io with documentation: home-assistant/home-assistant.io#8473

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 user exposed functionality or configuration variables are added/changed:

If the code communicates with devices, web services, or third-party tools:

  • New dependencies are only imported inside functions that use them (example).
  • New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

If the code does not interact with devices:

  • Tests have been added to verify that the new code works.
@MartinHjelmare MartinHjelmare merged commit e8ed56c into home-assistant:dev Feb 12, 2019
5 checks passed
5 checks passed
Hound No violations found. Woof!
WIP ready for review
Details
cla-bot Everyone involved has signed the CLA
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.02%) to 93.333%
Details
@ghost ghost removed the in progress label Feb 12, 2019
@andrewsayre andrewsayre deleted the andrewsayre:feature-smartthings-climate branch Feb 12, 2019
@MartinHjelmare

This comment has been minimized.

Copy link
Member

MartinHjelmare commented Feb 12, 2019

See home-assistant/home-assistant-polymer#2647 for target temp null issue.

@balloob balloob mentioned this pull request Feb 20, 2019
@GuGu927

This comment has been minimized.

Copy link

GuGu927 commented Feb 22, 2019

Hi
I’m using Samsung WiFi Kit(cassette air conditioner)
(my wifi kit's model name is AIM-H03N. It's similar with MIM-H03U)
and i can control it via ST APP.
but HA-ST components cant load it.
can you help me?

@andrewsayre

This comment has been minimized.

Copy link
Member Author

andrewsayre commented Feb 22, 2019

@GuGu927 This is a merged pull request, please open an issue if you are looking for assistance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.