onOpened and onReceive not firing #606

burdell · 2018-08-17T19:02:36Z

Description:
I've upgraded from 3.2.5 to 3.2.6 which resolved #583, but still can't get the onReceived and onOpened functions to fire when app is minimized. The functions do fire if app is completely shut down. Confirmed its working on iOS.

I've tried yarn remove/yarn add and react-native unlink/react-native link but still not seeing it working

Environment

Steps to Reproduce Issue:

Start app on Android
Minimize app
Trigger push notification
Open app by clicking push notificaition. No event fired
Quit app completely
Trigger push notification
Open app by clicking push notificaition. Event fired

The text was updated successfully, but these errors were encountered:

pacozaa · 2018-08-20T07:31:00Z

+1

pacozaa · 2018-08-20T08:39:31Z

I have to go back to 3.2.4

burdell · 2018-08-20T14:56:18Z

@pacozaa Did going back to 3.2.4 work for you? I still am seeing the same issue.

After doing more research I've found that if the initial notification happens from when the app is totally shut down, any notifications received when the app is minimized cause the handlers to fire.

But if I start the app, minimize without receiving a notification, the handlers are not fired when a notification comes in

Nightsd01 · 2018-08-20T18:37:51Z

@burdell I followed your steps to reproduce, but it works for me every time. Are you certain you tested on 3.2.6...? How do you know these handlers are not firing?

Can you test out our demo project with 3.2.6 and tell me if the issue occurs?

pacozaa · 2018-08-22T03:31:57Z

@burdell yes it is.

steverob · 2018-08-30T19:53:29Z

@Nightsd01 how do you run the demo project?

Nightsd01 · 2018-09-14T01:23:50Z

@pacozaa It's in the examples/RNOneSignal folder. You can run yarn, then run react-native link react-native-onesignal.

Once that's done, you should be able to just replace with your App ID in index.js and should be able to run the project. You can try modifying the SDK versions and so on to match what you are using in your real app.

Please let me know if you can reproduce this issue in our demo project or not - as of now I have not been able to reproduce this and 99% of the time it is user error - but it's possible it's a bug that only effects certain versions/devices as well.

burdell · 2018-09-18T13:41:00Z

Sorry, been away from this for a few weeks but I was able to verify that it was working in the demo app

Couldn't ever get it to work in our app though even after going through and trying to copy how the demo app was setup 🤷‍♂️

mcmar · 2018-09-19T18:29:16Z

I'm seeing different, but related behavior to what's described here on Android (3.2.7)
1, Foreground: onOpened/onReceived fire correctly
2, Background: onOpened/onReceived fire correctly
3, Dismissed (swiped-away from multitask switcher): onOpened/onReceived do not fire

On iOS, all of 1, 2, and 3 work correctly.

I am able to reproduce this behavior in the demo app on react-native-onesignal@3.2.7, which leads me to believe that this may be a recent regression on Android. Possibly related to the recent fixes for background?

akos-bako · 2018-09-28T10:18:38Z

I was experiencing the same issue as @burdell, but I managed to solve it.

On iOS, everything worked flawlessly. But OnOpened wouldn’t fire on Android if the app was in inactive (background or killed). I’ve experienced that it usually fires once every session, but once it’s put in the background again, and tap on a new notification, it just shows the app. What I found weird is that there are two instances of my application after I tapped on the notification.

react-native run-android
app cold starts (main instance)
press home button to put app in background
trigger & tap on new notification
onOpened IS called, but the app cold starts, despite the fact it was running 10 seconds before
NOTE: if i check my recents I have two instances of the application
press home to put app in background (new instance disappears from recents)
trigger & tap on new notification
onOpened is NOT called, app cold starts again
NOTE: in the recents there is only a single instance

I suspected this behaviour could be the root of the issue, and all I had to do to fix this is to modify the AndroidManifest.xml from:

<activity
        android:name=".MainActivity”
        android:launchMode=“singleTop”
        >

to: 

<activity
        android:name=".MainActivity”
        android:launchMode="singleInstance"
        >

After this, everything works perfectly on Android. I’m not sure why the example uses singleTop.

What I did:

I used the example app as a template in my project
I set up the redux rehydration to make sure everything happens after the store is set up (as seen here How to get initial notification? #332 (comment) )
Change to singleInstance

I'm using:
"react-native": "0.55.4",
"react-native-onesignal": “3.2.7” 

Nightsd01 · 2018-10-03T23:34:02Z

@mcmar Some android device makers will not allow your app to execute received callback if the app has been intentionally dismissed/terminated by the user. That would explain why the received callback didn't work for you - but you are also saying opened didn't work as well?

9/10 times when a user reports this issue it is because they have a bug in their code inside of the callback, or it's because of their launchmode/view hierarchy has an issue. We have not seen a user that has been able to reproduce in our example project.

When you reproduce in our example project, can you please zip up the project exactly as it was when you reproduced and post a link here?

Can you also post more information such as a logcat when you open the notification for the first 15 seconds? Can you also tell us what version/s of android you are able to reproduce this issue on?

aldoetobex · 2018-10-06T07:48:16Z

I've found how to solve this problem.
I use latest version of react-native-onesignal.

Just delete 2 lines below :

@OverRide
public void onHostDestroy() {
//OneSignal.removeNotificationOpenedHandler();
//OneSignal.removeNotificationReceivedHandler();
}

I have compared the previous version with the current version and viola! that's where the main problem is.

p-ugulino · 2018-10-27T18:47:12Z

I also have the same problem.
I've tested with samsung devices and android 7 or lower and everything is fine.
Though when I test on a Motorola with android 8 it doesn't work.

@Nightsd01 would you have any idea why this might happen?

Thanks

meetyourdev · 2018-11-26T12:35:18Z

I was experiencing the same issue as @burdell, but I managed to solve it.

On iOS, everything worked flawlessly. But OnOpened wouldn’t fire on Android if the app was in inactive (background or killed). I’ve experienced that it usually fires once every session, but once it’s put in the background again, and tap on a new notification, it just shows the app. What I found weird is that there are two instances of my application after I tapped on the notification.

react-native run-android

app cold starts (main instance)

press home button to put app in background

trigger & tap on new notification

onOpened IS called, but the app cold starts, despite the fact it was running 10 seconds before
NOTE: if i check my recents I have two instances of the application

press home to put app in background (new instance disappears from recents)

trigger & tap on new notification

onOpened is NOT called, app cold starts again
NOTE: in the recents there is only a single instance

I suspected this behaviour could be the root of the issue, and all I had to do to fix this is to modify the AndroidManifest.xml from:
<activity
        android:name=".MainActivity”
        android:launchMode=“singleTop”
        >
to: 
<activity
        android:name=".MainActivity”
        android:launchMode="singleInstance"
        >
After this, everything works perfectly on Android. I’m not sure why the example uses singleTop.

What I did:

I used the example app as a template in my project

I set up the redux rehydration to make sure everything happens after the store is set up (as seen here #332 (comment) )

Change to singleInstance

I'm using:
"react-native": "0.55.4",
"react-native-onesignal": “3.2.7”

This did not work for me.
When app is in background i.e app is minimized and in memory, OpOpened does not fire. This happens only in Oreo and above. The notification when clicked on it, the app takes about 10-20 seconds to launch.

udarts · 2018-11-27T08:24:25Z

Hi all,

I am having a similar issue. On both iOS and Android, when I receive a push notification, I click on the notification and it opens the app. I created a code that when the app opens and it checks if there is a push notification, then opens a different screen to display the notification.

Now the issue is, that the first time when I click on the notification the app opens, but nothing happens after that. Then closing the app, reopen it and suddenly the screen appears with the notification text shown.

So is the onOpened function not working the first time? This happens on iOS and Android.

rgomezp · 2019-04-16T22:37:28Z

@meetyourdev @udarts ,
Are you still having this issue?

mauricioalexandres · 2020-02-13T17:44:25Z

Any answers to the problem?

samitha9125 · 2020-04-21T15:49:50Z

This still happens. Any solution?

samitha9125 · 2020-04-21T15:52:39Z

OnReceived is still not firing when app is background. Any solution?

rgomezp · 2020-04-21T22:27:47Z

Howdy,
We haven't been able to reproduce this issue. Does anyone have an example project we could test with?

samitha9125 · 2020-04-22T11:36:45Z

@rgomezp I’m sorry. This does not happen always but when several notifications come to the mobile or when app was in background for few hours this happens. On both iOS and Android. This doesn’t happen always but regularly.

rgomezp · 2020-04-22T20:54:35Z

@samitha9125 ,
The received event only fires if the app is in foreground. Take a look at our "events" section from our documentation .

@mauricioalexandres ,
Can you provide some details regarding your specific implementation?

samitha9125 · 2020-04-22T21:00:13Z

@rgomezp Oh thank you for that. I didn’t see that earlier. So how can I get incoming notifications when my app is in the background?

rgomezp · 2020-04-27T18:49:15Z

Howdy,
We have a section describing how to implement Background Notifications. I'll close the issue since this isn't really an ongoing issue. Also, consider reaching out through our official OneSignal.com support channel for quicker support.

Enjoy!

eskiesirius · 2020-07-21T07:10:37Z

Hello.. i know this is an old issue but can you guide me? I already created an incoming call screen but the problem is that the incoming call screen wont display when the app is inactive.. so is there other way to display it even it is inactive?

arrygoo mentioned this issue Nov 27, 2018

Commented out the content of onHostDestroy in RNOneSignal.java SEAMTechnic/react-native-onesignal#1

Merged

rgomezp added Possible Bug Help Wanted and removed Possible Bug labels Mar 13, 2019

rgomezp closed this as completed Apr 24, 2019

rgomezp reopened this Apr 21, 2020

rgomezp closed this as completed Apr 27, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

onOpened and onReceive not firing #606

onOpened and onReceive not firing #606

burdell commented Aug 17, 2018

pacozaa commented Aug 20, 2018

pacozaa commented Aug 20, 2018

burdell commented Aug 20, 2018 •

edited

Nightsd01 commented Aug 20, 2018 •

edited

pacozaa commented Aug 22, 2018

steverob commented Aug 30, 2018

Nightsd01 commented Sep 14, 2018

burdell commented Sep 18, 2018

mcmar commented Sep 19, 2018 •

edited

akos-bako commented Sep 28, 2018

Nightsd01 commented Oct 3, 2018

aldoetobex commented Oct 6, 2018

p-ugulino commented Oct 27, 2018 •

edited

meetyourdev commented Nov 26, 2018

udarts commented Nov 27, 2018 •

edited

rgomezp commented Apr 16, 2019

mauricioalexandres commented Feb 13, 2020

samitha9125 commented Apr 21, 2020

samitha9125 commented Apr 21, 2020

rgomezp commented Apr 21, 2020

samitha9125 commented Apr 22, 2020

rgomezp commented Apr 22, 2020

samitha9125 commented Apr 22, 2020

rgomezp commented Apr 27, 2020

eskiesirius commented Jul 21, 2020

onOpened and onReceive not firing #606

onOpened and onReceive not firing #606

Comments

burdell commented Aug 17, 2018

pacozaa commented Aug 20, 2018

pacozaa commented Aug 20, 2018

burdell commented Aug 20, 2018 • edited

Nightsd01 commented Aug 20, 2018 • edited

pacozaa commented Aug 22, 2018

steverob commented Aug 30, 2018

Nightsd01 commented Sep 14, 2018

burdell commented Sep 18, 2018

mcmar commented Sep 19, 2018 • edited

akos-bako commented Sep 28, 2018

Nightsd01 commented Oct 3, 2018

aldoetobex commented Oct 6, 2018

p-ugulino commented Oct 27, 2018 • edited

meetyourdev commented Nov 26, 2018

udarts commented Nov 27, 2018 • edited

rgomezp commented Apr 16, 2019

mauricioalexandres commented Feb 13, 2020

samitha9125 commented Apr 21, 2020

samitha9125 commented Apr 21, 2020

rgomezp commented Apr 21, 2020

samitha9125 commented Apr 22, 2020

rgomezp commented Apr 22, 2020

samitha9125 commented Apr 22, 2020

rgomezp commented Apr 27, 2020

eskiesirius commented Jul 21, 2020

burdell commented Aug 20, 2018 •

edited

Nightsd01 commented Aug 20, 2018 •

edited

mcmar commented Sep 19, 2018 •

edited

p-ugulino commented Oct 27, 2018 •

edited

udarts commented Nov 27, 2018 •

edited