Skip to content

NSW Rural Fire Service platform#6353

Merged
frenck merged 4 commits intohome-assistant:nextfrom
exxamalte:geo_location_nsw_rural_fire_service_feed_platform
Oct 14, 2018
Merged

NSW Rural Fire Service platform#6353
frenck merged 4 commits intohome-assistant:nextfrom
exxamalte:geo_location_nsw_rural_fire_service_feed_platform

Conversation

@exxamalte
Copy link
Copy Markdown
Contributor

@exxamalte exxamalte commented Sep 25, 2018

Description:
Description of how to integrate the new NSW Rural Fire Service platform.

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

Checklist:

  • Branch: next is for changes and new documentation that will go public with the next home-assistant release. Fixes, changes and adjustments for the current release should be created against current.
  • The documentation follow the standards.

@ghost ghost added the to-do label Sep 25, 2018
@frenck frenck added new-integration This PR adds documentation for a new Home Assistant integration ready-for-review This PR needs to be reviewed next This PR goes into the next branch has-parent This PR has a parent PR in another repo and removed to-do labels Sep 30, 2018
DubhAd
DubhAd previously approved these changes Oct 1, 2018
Copy link
Copy Markdown
Contributor

@DubhAd DubhAd left a comment

Choose a reason for hiding this comment

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

👍

@DubhAd DubhAd added awaits-parent Awaits the merge of an parent PR and removed ready-for-review This PR needs to be reviewed labels Oct 1, 2018
logo: nsw-rural-fire-service.png
ha_category: Geo Location
ha_iot_class: "Cloud Polling"
ha_release: "0.80"
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.

0.81

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.

OK

radius:
description: The distance in kilometers around Home Assistant's coordinates in which incidents are included.
required: false
type: string
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.

Do we have float as option?

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.

Yes, it is actually a float

Copy link
Copy Markdown
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Nice! Can be merged when build passes.

@frenck frenck merged commit f381ba7 into home-assistant:next Oct 14, 2018
@ghost ghost removed the awaits-parent Awaits the merge of an parent PR label Oct 14, 2018
@exxamalte exxamalte deleted the geo_location_nsw_rural_fire_service_feed_platform branch October 14, 2018 21:26
@Captain-Glen
Copy link
Copy Markdown

Hi,

This plugin is great!!!

But I wonder if you could please update the documentation on how to use it in an automation. Specifically how to create a trigger if any fire is within a certain distance and then send an email using the notify: -platform smtp plugin.

I suspect your code allows this. But I haven't been able to figure out how to use it in an automation.

@exxamalte
Copy link
Copy Markdown
Contributor Author

Thanks @Captain-Glen - the automation part is covered in #7041 where I describe how to use the new geo_location trigger. Not sure when that will be online.

@Captain-Glen
Copy link
Copy Markdown

Captain-Glen commented Nov 6, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

has-parent This PR has a parent PR in another repo new-integration This PR adds documentation for a new Home Assistant integration next This PR goes into the next branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants