Skip to content

Added docs for DWD WarnApp Sensor #3062

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

Merged
merged 2 commits into from
Sep 9, 2017
Merged

Added docs for DWD WarnApp Sensor #3062

merged 2 commits into from
Sep 9, 2017

Conversation

runningman84
Copy link
Contributor

Description:
Added docs fornew sensor

Pull request in home-assistant (if applicable): home-assistant/core#8657

@runningman84 runningman84 changed the title Added docs for dwdwarnapp sensor Added docs for DWD WarnApp Sensor Jul 26, 2017
Copy link
Contributor

@Landrash Landrash left a comment

Choose a reason for hiding this comment

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

Looks good and can be merged when parent PR is merged.

@@ -0,0 +1,43 @@
---
layout: page
title: "DWD WarnApp"
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is it called WarnApp?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Because it uses the same data as the Android / IOS App "DWD WarnWetter". I could rename it to "DWD Warn Weather" if this would fit better?

Copy link
Contributor

Choose a reason for hiding this comment

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

I believe that would be preferable.

- Find your region here: `https://www.dwd.de/DE/wetter/warnungen_landkreise/warnWetter_node.html?ort=Hamburg`
- Verify if you find any warning for your region here: `https://www.dwd.de/DWD/warnungen/warnapp_landkreise/json/warnings.json?jsonp=loadWarnings`

The warning level is between 0 (no danger) and 4:
Copy link
Contributor

Choose a reason for hiding this comment

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

4 is what?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The warning level is between 0 (no danger) and 4:

Warnungen vor extremem Unwetter (Stufe 4)
Unwetterwarnungen (Stufe 3)
Warnungen vor markantem Wetter (Stufe 2)
Wetterwarnungen (Stufe 1)

Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry I was unclear. Shouldn't 4 be specified in the same way as 0 to be clear.

Like this for example(Excuse the bad translation)

The warning level is between 0 (no danger) and 4(warnings of extreme weather):

  • Warnungen vor extremem Unwetter (Stufe 4)
  • Unwetterwarnungen (Stufe 3)
  • Warnungen vor markantem Wetter (Stufe 2)
  • Wetterwarnungen (Stufe 1)

@Landrash Landrash added the new-integration This PR adds documentation for a new Home Assistant integration label Jul 28, 2017
@frenck
Copy link
Member

frenck commented Sep 6, 2017

@runningman84 I've noticed that the parent PR has been merged.
Could you please take care of this PR as well? 😃

As suggested by Landrash
@frenck frenck merged commit 386bd3d into home-assistant:next Sep 9, 2017
@balloob balloob added this to the 0.53 milestone Sep 9, 2017
balloob pushed a commit that referenced this pull request Sep 9, 2017
* Added docs for dwdwarnapp

* Updated documentation

As suggested by Landrash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new-integration This PR adds documentation for a new Home Assistant integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants