Skip to content

Fixing buienradar sensor config key error#21351

Merged
fabaff merged 1 commit into
home-assistant:devfrom
hmmbob:patch-2
Feb 23, 2019
Merged

Fixing buienradar sensor config key error#21351
fabaff merged 1 commit into
home-assistant:devfrom
hmmbob:patch-2

Conversation

@hmmbob
Copy link
Copy Markdown
Contributor

@hmmbob hmmbob commented Feb 23, 2019

Description:

Fixing warning about unsupported config key 'name' in Buienradar sensor, whilst it was actually supported (but just not validated).

Related issue (if applicable): n/a

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

Example entry for sensors.yaml:

### This will set the sensor name to sensor.volkel_[monitored_condition], 
### i.e. sensor.volkel_temperature
- platform: buienradar
  name: 'volkel'
  monitored_conditions:
    - stationname
    - condition
    - temperature

### This will set the sensor name to the default 'br': 
### sensor.br_[monitored_condition], i.e. sensor.br_temperature
- platform: buienradar
  monitored_conditions:
    - stationname
    - condition
    - temperature

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.

Fixing warning about unsupported config key in Buienradar sensor, whilst it was actually supported (but not validated)
Copy link
Copy Markdown
Member

@fabaff fabaff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks 🐦

@fabaff fabaff merged commit 954bd4e into home-assistant:dev Feb 23, 2019
@ghost ghost removed the in progress label Feb 23, 2019
@hmmbob hmmbob deleted the patch-2 branch February 23, 2019 15:22
@balloob balloob mentioned this pull request Mar 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants