Skip to content

Add Ukraine Alert: Air-raid sirens in Ukraine integration#22672

Merged
balloob merged 3 commits intohome-assistant:nextfrom
PaulAnnekov:ua_air_raid_siren_component
May 8, 2022
Merged

Add Ukraine Alert: Air-raid sirens in Ukraine integration#22672
balloob merged 3 commits intohome-assistant:nextfrom
PaulAnnekov:ua_air_raid_siren_component

Conversation

@PaulAnnekov
Copy link
Copy Markdown
Contributor

@PaulAnnekov PaulAnnekov commented May 7, 2022

Proposed change

As you know, more than 2 months ago Russian troops started invasion of Ukraine. They bombing our cities with rockets and artillery. I'm introducing a new component that exposes air-raid sirens in Ukraine into HA. Uses Ukraine Alarm API.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@probot-home-assistant probot-home-assistant Bot added in-progress This PR/Issue is currently being worked on needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels May 7, 2022
@probot-home-assistant probot-home-assistant Bot added the next This PR goes into the next branch label May 7, 2022
@probot-home-assistant
Copy link
Copy Markdown

It seems that this PR is targeted against an incorrect branch. Documentation updates which apply to our current stable release should target the current branch. Please change the target branch of this PR to current and rebase if needed. If this is documentation for a new feature, please add a link to that PR in your description.
(message by DocsTargetBranch)

Comment on lines +17 to +24
The Ukraine Alarm integration uses the [Ukraine Alarm](https://www.ukrainealarm.com/) web service as a source for air-raid sirens in Ukraine for your region. It's became very important since [Russian troops invasion](https://war.ukraine.ua/).

For each region integration exposes 4 binary sensors:

- Air
- Unknown
- Artillery
- Urban Fights
Copy link
Copy Markdown
Member

@balloob balloob May 8, 2022

Choose a reason for hiding this comment

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

Suggested change
The Ukraine Alarm integration uses the [Ukraine Alarm](https://www.ukrainealarm.com/) web service as a source for air-raid sirens in Ukraine for your region. It's became very important since [Russian troops invasion](https://war.ukraine.ua/).
For each region integration exposes 4 binary sensors:
- Air
- Unknown
- Artillery
- Urban Fights
The Ukraine Alarm integration uses the [Ukraine Alarm](https://www.ukrainealarm.com/) web service to offer alerts. The integration will create 4 binary sensors for your selected region in Ukraine:
- Air
- Artillery
- Urban Fights
- Unknown

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

to offer alerts

Actually, the official term is "air-raid siren" or "civil defense siren" (https://en.wikipedia.org/wiki/Civil_defense_siren). "alerts" is too generic. I've merged your suggestion, but left the term. Is it ok?

- Air
- Artillery
- Urban Fights
- Unknown
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is it explained on the official site what the unknown alarm represents?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No. I will ask support. I think it means "something is flying but we don't know what".

@klaasnicolaas klaasnicolaas added parent-merged The parent PR has been merged already and removed needs-rebase The PR has been branched of the wrong base branch or targets an incorrect target branch labels May 8, 2022
Comment thread source/_integrations/ukraine_alarm.markdown Outdated
Co-authored-by: Shay Levy <levyshay1@gmail.com>
@balloob balloob merged commit d6169ee into home-assistant:next May 8, 2022
@probot-home-assistant probot-home-assistant Bot removed in-progress This PR/Issue is currently being worked on parent-merged The parent PR has been merged already labels May 8, 2022
@balloob
Copy link
Copy Markdown
Member

balloob commented May 8, 2022

Cherry-picked into current.

balloob pushed a commit that referenced this pull request May 8, 2022
Co-authored-by: Shay Levy <levyshay1@gmail.com>
@github-actions github-actions Bot locked and limited conversation to collaborators May 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed has-parent This PR has a parent PR in another repo next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants