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

[Android] Moved geolocation plugin to Fused Location Provider #964

Closed
wants to merge 2,096 commits into from
Closed

[Android] Moved geolocation plugin to Fused Location Provider #964

wants to merge 2,096 commits into from

Conversation

exelban
Copy link

@exelban exelban commented Nov 9, 2018

Related to #379
Moved from standard android geolocation API to Fused Location Provider.

With old standard API, I have many problems with position updates.
Sometimes API returns position from many hours ago. And I can't make a force update.

This version of plugin work for a few months on our app. And during this time don't have any problems with GPS.

I tried to make API the same as with standard Android API. So, I think the plugin will be backward compatible.

mlynch and others added 30 commits June 25, 2018 17:00
Allow to configure Swift version for Cordova Plugins
Make LocalNotification id an Integer
Handle no extConfig on app init
* switch rootPage to ModalsPage for testing

* add buttonTitle to showAlert example

* add electron modals plugin

* Add alert implementation

* Add showActions implementation

* Add okButtonTitle and cancelButtonTitle to showConfirm example

* Add confirm implementation

* Implement prompt

* Change example back

* Update plugins.ts

* Update modals.ts

* Change back prompt to web implementation for now
baasgeo and others added 18 commits October 23, 2018 17:51
Using Capacitor on Android, no accuracy is recieved on the Geolocation plugin. This fix adds the property to the GeolocationPosition object.
* Documenting Push Notifications icon on Android.

Closes #926.
#956)

* Ability to configure splash autohiding on app launch in `capacitor.config.json`

* Update "Configuration" section in docs
…thod only (#962)

* Android: Intercept proxyied requests in handleProxyRequest for GET method only
@aheiz
Copy link

aheiz commented Jan 1, 2020

Why is this closed? Do we have to use the community plugin "@johannesnormannjensen/capacitor-fused-location" to use fused location?

@exelban
Copy link
Author

exelban commented Jan 2, 2020

@aheiz I rewrite geolocation plugin to use in my project. And I create a PR for this. But there was no feedback from maintainers for a long time. And I closed this PR.

@shuhikari
Copy link

shuhikari commented Mar 25, 2020

@exelban Did you manage to get the geolocation working ?

@exelban
Copy link
Author

exelban commented Mar 26, 2020

@exelban Did you manage to get the geolocation working ?

Yes

@shc261392
Copy link

@exelban Would you like to share the rewritten plugin? I've been experiencing some problems with the original geolocation plugin too and it would be great if I could use your solution.

@exelban
Copy link
Author

exelban commented Mar 27, 2020

@shc261392 yeap. I will share the code in the next few days

@exelban
Copy link
Author

exelban commented Mar 27, 2020

@shc261392 @shuhikari link to code and a short description of how to wap it with existing Geolocation plugin.

@jcesarmobile
Copy link
Member

capacitor 2.0.0-beta.1 uses fused location provider, you can install it like this: npm intall @capacitor/android@next

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.

None yet