Add voluptuous to wemo. #2895

pavoni · 2016-08-19T22:12:53Z

Description:

Related issue (if applicable): fixes #

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

Example entry for configuration.yaml (if applicable):

Checklist:

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

If code communicates with devices, web services, or a:

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.

fabaff · 2016-08-20T12:03:46Z

Looks good to me.

This was reported in https://community.home-assistant.io/t/issue-after-update/3339/4 home-assistant#2895 added voluptuous to wemo, but in the process it removed defaulting static to an empty array.

Add voluptuous to wemo.

60c7fde

fabaff mentioned this pull request Aug 19, 2016

Use voluptuous #2800

Closed

fabaff merged commit 482f32b into dev Aug 20, 2016

pavoni deleted the wemo_voluptuous branch August 20, 2016 14:33

technicalpickles mentioned this pull request Aug 28, 2016

Fix wemo component without static configuration #3032

Closed

8 tasks

home-assistant locked and limited conversation to collaborators Mar 17, 2017

Provide feedback