8000 Sytadin default value must be a list · Issue #10233 · home-assistant/core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Sytadin default value must be a list #10233

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.

Sign up for GitHub

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

Jump to bottom
Closed
gautric opened this issue Oct 30, 2017 · 0 comments · Fixed by #10234
Closed

Sytadin default value must be a list #10233

gautric opened this issue Oct 30, 2017 · 0 comments · Fixed by #10234

Comments

@gautric
Copy link
Contributor
gautric commented Oct 30, 2017

Home Assistant release (hass --version):
pre 0.57

Python release (python3 --version):
Python 3.6.3

Component/platform:
Sytadin

Description of problem:
Sytadin CONF_MONITORED_CONDITIONS default value must be a list, and actual default value is string.
Only appends with the default value (minimal config).

Expected:
default value must be inside a list

Traceback (if applicable):

2017-10-30 21:56:02 ERROR (MainThread) [homeassistant.components.sensor] Error while setting up platform sytadin
Traceback (most recent call last):
  File "/Users/gautric/Source/git/hass/home-assistant/homeassistant/helpers/entity_component.py", line 170, in _async_setup_platform
    SLOW_SETUP_MAX_WAIT, loop=self.hass.loop)
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/tasks.py", line 358, in wait_for
    return fut.result()
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/asyncio/futures.py", line 245, in result
    raise self._exception
  File "/opt/local/Library/Frameworks/Python.framework/Versions/3.6/lib/python3.6/concurrent/futures/thread.py", line 56, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/Users/gautric/Source/git/hass/home-assistant/homeassistant/components/sensor/sytadin.py", line 63, in setup_platform
    sytadin, name, option, SENSOR_TYPES[option][0],
KeyError: 't'

't' is the first letter of OPTION_TRAFFIC_JAM value

Additional info:
gautric pushed a commit to gautric/home-assistant that referenced this issue Oct 30, 2017
Sytadin default value must be a list home-assistant#10233
cde7ae9
gautric pushed a commit to gautric/home-assistant that referenced this issue Oct 30, 2017
Sytadin default value must be a list home-assistant#10233 v2
8ea165a
@gautric gautric mentioned this issue Oct 30, 2017
Merged
fabaff pushed a commit that referenced this issue Oct 31, 2017
@gautric @fabaff
Sytadin default value must be a list #10233 (#10234)
8832de8 
* Sytadin default value must be a list #10233

* Sytadin default value must be a list #10233 v2
@balloob balloob mentioned this issue Nov 2, 2017
Merged
@home-assistant home-assistant locked and limited conversation to collaborators Mar 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Sytadin default value must be a list #10233 gautric/home-assistant
1 participant
@gautric
0