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

Improve documentation around android background location posting #29

Closed
RaddishIoW opened this issue May 5, 2021 · 4 comments
Closed
Labels
android Related to the Android platform documentation Improvements or additions to documentation stale

Comments

@RaddishIoW
Copy link

As it currently stands on Android, the plug-in does not execute the callback function in JS when the location changes while the app is in the background/the phone is locked.
The only way that the plug-in can communicate location changes to an API/server is through the http posting functionality.

This is different to the behaviour on iOS, where the callback function is called, allowing the app to post, even when in the background.

This needs to be communicated better in the documentation, as it is not clear.

@RaddishIoW RaddishIoW added documentation Improvements or additions to documentation android Related to the Android platform labels May 5, 2021
@HarelM
Copy link
Collaborator

HarelM commented May 5, 2021

I fully agree, also I think it's important to emphasize the foreground service mode or maybe make it default as I believe if someone is using this plugin he will want the service to run as much as possible without getting killed by the os.

@RaddishIoW
Copy link
Author

I have made it a bit clearer that you need to use the HTTP posting for background posts of location, but I'd like to make it a bit more prominent in the docs...

@stale
Copy link

stale bot commented Dec 19, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 19, 2021
@stale
Copy link

stale bot commented Jan 2, 2022

This issue has been automatically closed, because it has not had recent activity. If you believe this issue shouldn't be closed, please reopen or write down a comment requesting issue reopening with explanation why you think it's important. Thank you for your contributions.

@stale stale bot closed this as completed Jan 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android Related to the Android platform documentation Improvements or additions to documentation stale
Projects
None yet
Development

No branches or pull requests

2 participants