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

Adding stopWithTask tag and onTaskRemoved to the service #32

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mathBordin
Copy link

@mathBordin mathBordin commented Nov 14, 2020

I noticed that my application wasn't killed because the foreground service continued to run after closing the app, even calling stopService on componentWillUnmount. So, I made this change to solve this problem by adding the stopWithTask tag on the AndroidManifest.xml and calling stopSelf inside onTaskRemoved.

You can check for this problem by verifying that the application's PID is the same after reopening it. In fact, it continues to run even when it is closed.

Before changes:

Captura de Tela (110)

After changes:

Captura de Tela (111)

This change was based on this topic on Stack Overflow: https://stackoverflow.com/questions/53334235/how-to-properly-stop-a-foreground-service

matheus-bg added 2 commits November 13, 2020 22:54
I noticed that my application was killed  because the foreground service continued to run after closing the app. So, I made this change to solve this problem by adding the stopWithTask tag on the AndroidManifest.xml and calling stopSelf inside  onTaskRemoved.
@esinanturan
Copy link

@mathBordin It's killing the notification but when I checked the app status on the app info it seems app still working in the background and the option 'force stop' is available. And I checked it starting the service causing this without foreground service when I kill the app there is no 'force stop' option available. I think it's not exactly killing the service

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.

3 participants