Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dshokouhi
Copy link
Member

Breaking Change:

Nothing breaks

Description:

Not all accounts were supported by mistake, this library fix corrects the issue for user accounts.

Related issue (if applicable): fixes #26732

Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io#N/A

Example entry for configuration.yaml (if applicable):

sensor:
  - platform: obihai
    host: 192.168.1.x
    username: user
    password: user

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.
  • I have followed the development checklist

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

  • The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
  • New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
  • Untested files have been added to .coveragerc.

@dshokouhi
Copy link
Member Author

if possible please include in the next hot fix

@MartinHjelmare MartinHjelmare added this to the 0.99.1 milestone Sep 19, 2019
@MartinHjelmare MartinHjelmare merged commit 94192ec into home-assistant:dev Sep 19, 2019
@dshokouhi dshokouhi deleted the bump_pyobihai branch September 19, 2019 20:28
@balloob balloob mentioned this pull request Sep 19, 2019
@lock lock bot locked and limited conversation to collaborators Sep 20, 2019
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.

Error while setting up platform obihai

4 participants