Skip to content

Bump pywemo version. Improves notifications for insight devices.#6159

Merged
pvizeli merged 1 commit into
devfrom
bump-pywemo-version
Feb 22, 2017
Merged

Bump pywemo version. Improves notifications for insight devices.#6159
pvizeli merged 1 commit into
devfrom
bump-pywemo-version

Conversation

@pavoni
Copy link
Copy Markdown
Contributor

@pavoni pavoni commented Feb 22, 2017

Description:
That updates the library version to improve notifications for wemo insight switches. It turns on subscriptions for power useage changes. Thanks to @jumpkick for the detective work and contribution

#6001 adds additional support for more attributes, but since the version upgrade itself was simple and useful I thought it was worth adding this as an preliminary PR.

Related issue (if applicable):
fixes #3506

Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>

Example entry for configuration.yaml (if applicable):

Checklist:

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

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

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • New dependencies have been added to the REQUIREMENTS variable (example).
  • New dependencies are only imported inside functions that use them (example).
  • New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
  • New files were added to .coveragerc.

If the code does not interact with devices:

  • Local tests with tox run successfully. Your PR cannot be merged unless tests pass
  • Tests have been added to verify that the new code works.

@mention-bot
Copy link
Copy Markdown

@pavoni, thanks for your PR! By analyzing the history of the files in this pull request, we identified @jaharkes, @balloob and @fabaff to be potential reviewers.

@pvizeli pvizeli merged commit 896e047 into dev Feb 22, 2017
@pavoni pavoni deleted the bump-pywemo-version branch February 22, 2017 16:17
@home-assistant home-assistant locked and limited conversation to collaborators Jun 2, 2017
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.

Wemo insight (+ perhaps maker) state attributes don't update

4 participants