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.

Sign up for GitHub

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

Jump to bottom

Request permission on android (7+) #1

Closed
netdur opened this issue Jul 14, 2017 · 4 comments
Closed

Request permission on android (7+) #1

netdur opened this issue Jul 14, 2017 · 4 comments

Comments

@netdur
Copy link

netdur commented Jul 14, 2017

LocationPlugin.java.txt
@Lyokone
Copy link
Owner

Lyokone commented Jul 17, 2017

Thanks for noticing it, added your code with some modifications. I wasn't able to invoke OnRequestPermissionsResultCallback so you still have to relaunch the app to use it. Aiming SDK 21 seems safer ...

Closed
@long1eu
Copy link
Contributor

long1eu commented Jan 23, 2018

It's not safer, as you know the user can still deny permission. 61 % of devices running android are Marshmallow and above so why not progressing along?

@garyloud garyloud mentioned this issue Mar 6, 2018
Closed
@g123k
Copy link
Contributor

g123k commented Mar 19, 2018 •
edited
Loading

My PR #25 fixes this.
Feel free to send me your feedbacks.

@g123k g123k mentioned this issue Mar 19, 2018
Merged
@Lyokone
Copy link
Owner

Lyokone commented Apr 5, 2018

Thanks for your contrib !

@Lyokone Lyokone closed this as completed Apr 5, 2018
@JohnAherne JohnAherne mentioned this issue May 29, 2018
Closed
@alexjuniodev alexjuniodev mentioned this issue May 30, 2018
Closed
@kruseborn kruseborn mentioned this issue Jun 7, 2018
Closed
@dadmad817 dadmad817 mentioned this issue Oct 5, 2018
Closed
Somnibyte pushed a commit to Somnibyte/flutterlocation that referenced this issue Dec 19, 2018
@prefanatic
Flutter Location Updates (Lyokone#1)
d2dd7d9 
* Modify FlutterLocation plugin to include new  command so that developers can display the permission modal at specific points within their codebase. Added LocationPermissionStreamHandler to allow for listening for the users permission modal response.

* Remove permission stream and replace with method channel. Refactor.

* Remove permission stream.

* Fix variable error.
@preetjdp preetjdp mentioned this issue Feb 10, 2019
Closed
@cronos88 cronos88 mentioned this issue Mar 12, 2019
Closed
@prabhuk56 prabhuk56 mentioned this issue May 3, 2019
Closed
@sarimk80 sarimk80 mentioned this issue May 4, 2019
Closed
@Mohammadhamza43 Mohammadhamza43 mentioned this issue Sep 3, 2019
Closed
@Velek Velek mentioned this issue Feb 8, 2020
Closed
@EightMinuteYouth EightMinuteYouth mentioned this issue Feb 19, 2020
Closed
mehmetf pushed a commit that referenced this issue Oct 23, 2020
@creativepsyco
Merge pull request #1 from Lyokone/master
1078eb2 
Update with master fork
@Sarthakkore1109 Sarthakkore1109 mentioned this issue Jan 20, 2021
Closed
Lyokone pushed a commit that referenced this issue Mar 8, 2021
@kevlatus
Merge pull request #1 from Lyokone/master
9aeecbc 
merge null safety
@saaivignesh20-zz saaivignesh20-zz mentioned this issue Jun 16, 2021
Closed
@1712916 1712916 mentioned this issue Nov 10, 2022
Closed
@kostadin24 kostadin24 mentioned this issue Mar 8, 2023
Closed
@marciotisouza marciotisouza mentioned this issue Jul 29, 2023
Closed
@keskink keskink mentioned this issue Jul 30, 2023
Closed
@burakglc burakglc mentioned this issue Aug 3, 2023
Closed
@dixiak dixiak mentioned this issue Aug 12, 2023
Open
@fozlerabbi321 fozlerabbi321 mentioned this issue Jul 28, 2024
Open
@JEThilenius JEThilenius mentioned this issue Oct 3, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@g123k @netdur @Lyokone @long1eu