-
-
Notifications
You must be signed in to change notification settings - Fork 312
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
base: master
Are you sure you want to change the base?
Conversation
hmm not sure why my link is not generating properly 🤔 |
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."
(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" should be followed by a comma.
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. |
Docs update for: home-assistant/android#4818