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.

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

@capacitor/geolocation does NOT throw an error even though GPS is unavailable #1696

Closed
ZweiEuro opened this issue Jul 23, 2023 · 4 comments
Closed

Comments

@ZweiEuro
Copy link

Hi!
Given this issue: #822

I'd assume that devices where the google play service is not available, and thereby GPS is not available. checkPermissions should throw an error, it doesn't though.

This took me forever to debug since e.g. on /e/os operating system (one without playstore, and with which i am developing) it just doesn't return anything upon requesting a location but returns just fine when checking permissions.

Please consider printing at least a warning or something so other people don't waste 3h searching for something that's not even a bug. Or consider adding "This requires playstore location service to work properly" to the docs.

@Ionitron
Copy link
Collaborator

This issue needs more information before it can be addressed.
In particular, the reporter needs to provide a minimal sample app that demonstrates the issue.
If no sample app is provided within 15 days, the issue will be closed.

Please see the Contributing Guide for how to create a Sample App.

Thanks!
Ionitron 💙

@ZweiEuro
Copy link
Author

ZweiEuro commented Jul 24, 2023

run your own example on any non playstore device: https://github.com/ionic-team/capacitor-plugins/tree/main/geolocation#example

There is nothing more to it.

@Ionitron
Copy link
Collaborator

It looks like this issue didn't get the information it needed, so I'll close it for now. If I made a mistake, sorry! I am just a bot.

Have a great day!
Ionitron 💙

Copy link

ionitron-bot bot commented Nov 26, 2023

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of the plugin, please create a new issue and ensure the template is fully filled out.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Nov 26, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants