Skip to content
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.

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

AirVisual doesn't support scan_interval any longer #33182

Closed
twsl opened this issue Mar 23, 2020 · 5 comments
Closed

AirVisual doesn't support scan_interval any longer #33182

twsl opened this issue Mar 23, 2020 · 5 comments

Comments

@twsl
Copy link

twsl commented Mar 23, 2020

Environment

  • Home Assistant release with the issue: 0.107.5
  • Last working Home Assistant release (if known): -
  • Operating environment (Hass.io/Docker/Windows/etc.): Docker
  • Integration causing this issue: airvisual
  • Link to integration documentation on our website: https://www.home-assistant.io/integrations/airvisual/

Problem-relevant configuration.yaml

airvisual:
    api_key: YOUR_AIRVISUAL_API_KEY
    geographies:
        latitude: 42.81212
        longitude: 108.12422
    scan_interval: 300

Traceback/Error logs

Invalid config for [airvisual]: [scan_interval] is an invalid option for [airvisual]. Check: airvisual->airvisual->scan_interval. (See ?, line ?).

Additional information

The scan_interval option is removed. This was probably a design decission. But the documentation needs to be updated.

@probot-home-assistant
Copy link

Hey there @bachya, mind taking a look at this issue as its been labeled with a integration (airvisual) you are listed as a codeowner for? Thanks!

@bachya
Copy link
Contributor

bachya commented Mar 23, 2020

This was noted in the breaking changes for 0.107.0 (original PR: #32072).

@bachya bachya closed this as completed Mar 23, 2020
@twsl
Copy link
Author

twsl commented Mar 23, 2020

The documentation still needs an update.

@bachya
Copy link
Contributor

bachya commented Mar 23, 2020

Ah, I didn’t see that part of your original comment. Thanks!

@frenck
Copy link
Member

frenck commented Mar 23, 2020

Opened up home-assistant/home-assistant.io#12496 to address this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants