Skip to content

Update gps_controller to not require location accuracy in android.#155

Merged
stopflock merged 3 commits into
FoggedLens:mainfrom
reb1995:fix-android-gps-location-accuracy-requirement
May 5, 2026
Merged

Update gps_controller to not require location accuracy in android.#155
stopflock merged 3 commits into
FoggedLens:mainfrom
reb1995:fix-android-gps-location-accuracy-requirement

Conversation

@reb1995

@reb1995 reb1995 commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

I have only tested this for Android. I do not know how to test for iOS, however it seems like it should work.

I believe this would fix: #149.

This updates the GpsController to explicitly use Android's native hardware Location Manager instead of the Fused Location Provider (FLP). This will prevent Google Services from attempting to handle the request and asking for location accuracy. This will require a full GPS lock to function and will take longer to get a location fix vs using Location Accuracy. I believe this to be a worthy trade off with this app, though.

Adds basic target matching. If Android Do... Else use old code.
@reb1995

reb1995 commented Apr 6, 2026

Copy link
Copy Markdown
Contributor Author

Tested/confirmed working without location accuracy.

Screenshot_20260406-135548.png

Screenshot_20260406-135551.png

reb1995 added 2 commits April 12, 2026 19:39
Remove extra import that isn't required. Now passes flutter analyze.
@reb1995

reb1995 commented Apr 13, 2026

Copy link
Copy Markdown
Contributor Author

Updated the commit to remove an extra import that wasn't needed. Passing all tests now.

@reb1995

reb1995 commented Apr 26, 2026

Copy link
Copy Markdown
Contributor Author

Just pinging to bring it up again. Is this something we can merge?

@stopflock

Copy link
Copy Markdown
Collaborator

Sure

@stopflock stopflock merged commit 5b55171 into FoggedLens:main May 5, 2026
4 checks passed
@reb1995 reb1995 deleted the fix-android-gps-location-accuracy-requirement branch May 9, 2026 16:17
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.

Why does the app require location accuracy?

2 participants