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

Kill on destroy #25

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

Conversation

mattveraldi
Copy link

Hi,
I needed to add this behaviour to my react-native app, so I implemented this notificationConfig.
Feel free to merge it if you think it's useful.
Best regards.

@hazelf42
Copy link

hazelf42 commented Feb 2, 2021

+1 would like this added

@esinanturan
Copy link

esinanturan commented Feb 16, 2021

@mattveraldi 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.
I had to add

android.os.Process.killProcess(android.os.Process.myPid());
System.exit(0);

to the onHostDestroy method of the module

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