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

GeoJSON platform #6316

Merged

Conversation

exxamalte
Copy link
Contributor

@exxamalte exxamalte commented Sep 19, 2018

Description:
This describes a new platform to integrate GeoJSON feeds.

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

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 19, 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 a other repo and removed to-do labels Sep 21, 2018
required: false
type: string
default: 20km
scan_interval:
Copy link
Member

Choose a reason for hiding this comment

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

We usually don't mention this, since it's a standard option for platforms.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK, I'll mention the default value of 5 minute in the description instead.

logo: geo_location.png
ha_category: Geo Location
ha_release: "0.79"
---
Copy link
Member

Choose a reason for hiding this comment

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

Add a ha_iot_class.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added "Cloud Polling"

@MartinHjelmare MartinHjelmare merged commit a114c7f into home-assistant:next Sep 22, 2018
@ghost ghost removed the ready-for-review This PR needs to be reviewed label Sep 22, 2018
@exxamalte exxamalte deleted the geo_location_geojson_platform branch October 24, 2018 09:09
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 a other 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.

3 participants