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

Conversation

Projects
None yet
4 participants
@mjj4791
Copy link
Contributor

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 balloob merged commit 7751858 into home-assistant:dev Aug 1, 2017

4 checks passed

cla-bot Everyone involved has signed the CLA
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage decreased (-0.004%) to 93.637%
Details
hound No violations found. Woof!
@balloob

This comment has been minimized.

Copy link
Member

commented Aug 1, 2017

Thanks 🐬

@fabaff fabaff referenced this pull request Aug 12, 2017

Merged

0.51 #8919

dethpickle added 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.
You can’t perform that action at this time.