-
-
Notifications
You must be signed in to change notification settings - Fork 858
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
[LocationPermission] Fix warning UI unresponsive in mainthread when c… #1329
[LocationPermission] Fix warning UI unresponsive in mainthread when c… #1329
Conversation
…alling locationServicesEnabled
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.
Hi @honghai2196,
The change looks good, however could you please also update the CHANGELOG.md and pubspec.yaml as requesting in the PR template?
…read when checking location state
I have pushed changes follow your comment. Please re-review. Thanks |
@mvanbeusekom please re-review it. thanks |
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.
LGTM
@honghai2196 thank you very much for this contribution. I have just published these changes to pub.dev. |
…alling locationServicesEnabled
Fix Xcode warning when checking Location Services Enabled/Disabled
This PR fix this issue: 1002
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.