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

buienradar dates tz-aware #8767

Merged
merged 1 commit into from
Aug 1, 2017
Merged

buienradar dates tz-aware #8767

merged 1 commit into from
Aug 1, 2017

Conversation

mjj4791
Copy link
Contributor

@mjj4791 mjj4791 commented Jul 31, 2017

Description:

Sensor:
Attribution of a Buienradar sensor will now show the actual datetime of measurement at the weatherstation. (this might be up to 10-20 minutes before the update was detected over the buienradar API.)

Weather:
Forecasted data are now timezone aware and is attributed to noon dutch time (Europe/Amsterdam) and not at midnight. This will allow someone to correctly show buienradar weather forecast when not running in Europe/Amsterdam timezone.

No updated documentation
No config changes

Checklist:

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][ex-requir]).
  • New dependencies are only imported inside functions that use them ([example][ex-import]).
  • New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.

@balloob
Copy link
Member

balloob commented Aug 1, 2017

Thanks 🐬

@fabaff fabaff mentioned this pull request Aug 12, 2017
dethpickle pushed a commit to dethpickle/home-assistant that referenced this pull request Aug 18, 2017
@home-assistant home-assistant locked and limited conversation to collaborators Dec 11, 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.

4 participants