Skip to content

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