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

Add note about geofence limit #1141

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Add note about geofence limit #1141

wants to merge 10 commits into from

Conversation

dshokouhi
Copy link
Member

Docs update for: home-assistant/android#4818

@dshokouhi
Copy link
Member Author

hmm not sure why my link is not generating properly 🤔

@jpelgrom
Copy link
Member

Google has a hard limit of 100 geofences per app.

Do we really need to mention Google here? That creates the question for me if it is different if HA were to use location service x. Or people could be thinking you are talking about phone brands here? Suggestion because there is no alternative implementation without GMS at the moment: "Android apps can create up to 100 geofences."

I'm also missing the answer to the question what this means for me if I have more than 100 zones. Suggestion to add immediately after the previous suggested sentence: "If you have more than 100 zones, Home Assistant will only create geofences and receive events for the first 100 zones."

When combined with High Accuracy mode zone with trigger range constraints you will need to double the count per zone included in the constraint.

(insert new line before this sentence, because not everyone is using high accuracy mode)

Maybe add a comma after the link? When reading it out loud there is a pause there.

Suggestion for the second part that feels easier to understand to me: "each zone included in the constraint will create 2 geofences".

For example if you have 2 zones total

"For example" should be followed by a comma.

and only want the trigger range for 1 zone that means there will be 3 geofences created.

Is want right here? The reasons for (not) using that option are explained on the other page. Suggestion: "with trigger range configured for 1 zone, the app will create 3 geofences.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants