-
-
Notifications
You must be signed in to change notification settings - Fork 662
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
Fixes a bug where iOS location settings, e.g. accuracy
and distanceFilter
are overridden by different calls
#1600
Conversation
…eFilter` are overridden by different calls
@mvanbeusekom Can someone take a look at the PR? Or what's the flow to be merged into master? |
Hi @ksenia-lyagusha, Thank you for submitting this PR. Would you mind reverting the changes to the Also please revert the changes for the |
…r those. Revert the changes for the requestLocation API. Now use separate oneTimeLocationManager with startUpdatingLocation logic. Reverted and updated tests.
a1c2ebc
to
70f9d44
Compare
@mvanbeusekom I updated PR, can you please review it then I create a second PR with updating example to the main geolocator package? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ksenia-lyagusha thank you for applying the feedback. I think this looks great!
I have just published the update to pub.dev, version 2.3.8 of the geolocator_apple package should now be available. |
@mvanbeusekom Thank you. Created a new PR (#1605) with updated version. |
Hi @SebastianKeller, Thank you for reporting this. The |
Hi @mvanbeusekom, that was fast :) I can confirm, 2.3.8+1 is no longer crashing. Thank you! |
Hi @mvanbeusekom |
Fixes a bug where iOS location settings, e.g.
accuracy
anddistanceFilter
are overridden by different callsFixes this comment in the thread #1116 (comment) and probably the whole ticket #1116
CLLocationManager
instances, for one time request location and persistent request location.[CLLocationManager requestLocation]
method for one time location.accuracy
anddistanceFilter
are overridden by different calls.iOS 11
as lower is not supported anymore by Xcode.Pre-launch Checklist
pubspec.yaml
with an appropriate new version according to the pub versioning philosophy, or this PR is does not need version changes.CHANGELOG.md
to add a description of the change.///
).main
.dart format .
and committed any changes.flutter analyze
and fixed any errors.