setNotificationOpenedHandler does not executes when app is terminated/not opened at all #540

apoorvpandey0 · 2022-03-05T16:13:56Z

setNotificationOpenedHandler works very well when app is in background or in focus, but when app is swiped away i.e terminated by the user, This listener is not being called so I'm not able to access notification additional data.
I'm testing this with Flutter 2.10.1 and Onesginal SDK 3.2.8
I hope I'm clear, please help!

andikanugraha · 2022-03-10T07:14:50Z

yes, I'm also having this issue. Is there any solution for this?

mjarpitanand · 2022-03-28T18:05:15Z

Same error i am getting.

ayechan-maung · 2022-03-30T05:15:26Z

We all get errors like that.

mjarpitanand · 2022-04-02T03:43:01Z

Any solution for this

AmmarHazem · 2022-04-12T12:41:10Z

Same issue here, any update on this ?

nan-li · 2022-04-12T20:33:33Z

Hi all,

Thank you for reporting this issue.

Can you share the device information for this? Is it on Android and/or iOS? Type of phone, for example Huawei, etc.

AmmarHazem · 2022-04-13T09:03:41Z

Thanks for replay @nan-li. I got this issue on iOS iPhone XR.

apoorvpandey0 · 2022-04-13T09:44:52Z

Hi all,

Thank you for reporting this issue.

Can you share the device information for this? Is it on Android and/or iOS?

For me it was on Android, although I don't think this is a device specific issue and believe that this feature has not even been implemented in the SDK for flutter.
I also saw a PR where @efraespada has implemented this feature #464

marcorubagotti · 2022-06-10T12:58:12Z

Any updates?

emawby · 2022-06-24T17:11:52Z

@marcorubagotti We are still investigating the issue and working on a fix. I apologize for the delay and thank you for your patience

mrqaidi · 2022-06-26T19:05:15Z

Any updates?

piotruela · 2022-07-20T09:05:43Z

Any updates on this?

daveshirman · 2022-07-22T14:49:11Z

To workaround:

You need to store the result of the notification when your app loads e.g. in local storage, inside this method: setNotificationOpenedHandler.
Then at some point after, check your storage for any notification data that came in.

After the app is loaded, the handler will run as expected.

alexobviously · 2022-07-30T13:08:58Z

I also have this issue and the workaround above does not work, as the notification handler (passed to setNotificationOpenedHandler) is never called if the app is opened from a notification.

Device: Poco X3 NFC, Android 11 (MIUI 12.5.8)

Did this ever work? Is this a bug or just missing functionality?

daveshirman · 2022-07-30T15:09:17Z

Not sure what to tell you, mine works doing exactly what I said. Whether it's a cold boot from a notification, where the app starts from zero, or whether it's already in memory, I get my data through in the handler and everything is working as expected. Can paste some code this week if that'd give an indication of how mine works?

…

On Sat, 30 Jul 2022, 09:09 Alex Baker, ***@***.***> wrote: I also have this issue and the workaround above does not work, as the notification handler (passed to setNotificationOpenedHandler is never called if the app is opened from a notification. Did this ever work? Surely this is core functionality? — Reply to this email directly, view it on GitHub <#540 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADM6KXTTN3ENX7IN3OL5KJTVWUSPLANCNFSM5P73X63A> . You are receiving this because you commented.Message ID: ***@***.***>

daveshirman · 2022-07-30T15:21:52Z

Just thought, you're not running AdBlock or anything similar on the device? I've had issues before when that was enabled. Also, are you setting up the handler in main.dart before MaterialApp gets executed?

…

On Sat, 30 Jul 2022, 11:09 Dave Shirman, ***@***.***> wrote: Not sure what to tell you, mine works doing exactly what I said. Whether it's a cold boot from a notification, where the app starts from zero, or whether it's already in memory, I get my data through in the handler and everything is working as expected. Can paste some code this week if that'd give an indication of how mine works? On Sat, 30 Jul 2022, 09:09 Alex Baker, ***@***.***> wrote: > I also have this issue and the workaround above does not work, as the > notification handler (passed to setNotificationOpenedHandler is never > called if the app is opened from a notification. > > Did this ever work? Surely this is core functionality? > > — > Reply to this email directly, view it on GitHub > <#540 (comment)>, > or unsubscribe > <https://github.com/notifications/unsubscribe-auth/ADM6KXTTN3ENX7IN3OL5KJTVWUSPLANCNFSM5P73X63A> > . > You are receiving this because you commented.Message ID: > ***@***.***> >

alexobviously · 2022-07-30T15:51:46Z

Yeah I have all the onesignal initialisation stuff before MaterialApp, even before WidgetsFlutterBinding.ensureInitialized();, and OneSignal.shared.setAppId is awaited. I've tried moving things around, adding delays in, etc, but it seems like nothing works. To confirm that the handler is never called, I put an exception in it, and it never occurs. The handler works as expected in all other cases - it's just when the app is opened from a notification that it doesn't.

Nothing like AdBlock running (though that is an interesting idea).

@daveshirman if you do have code you could share it would be very much appreciated (no rush!), maybe I've missed something.

daveshirman · 2022-08-04T16:51:10Z

@alexobviously To keep this ticket cleaner, I moved my workaround over to my site, in case you still wanted it:

https://www.codingandclimbing.co.uk/blog/flutter-onesignal-additionaldata-on-app-start

marcorubagotti · 2022-09-05T08:15:49Z

@daveshirman i have follow your workaround as explained in your site. Like @alexobviously the handler works in all cases except when the app is closed.
May can depend on how I have configured the routes and home settings in MaterialApp() ?

daveshirman · 2022-09-05T12:28:42Z

Not sure what to tell you. It works for me on both iOS and Android. Without being able to see your code compared to mine, I couldn't possibly help.

…

On Mon, 5 Sept 2022, 04:16 marcorubagotti, ***@***.***> wrote: @daveshirman <https://github.com/daveshirman> i have follow your workaround as explained in your site. Like @alexobviously <https://github.com/alexobviously> the handler works in all cases except when the app is closed. May can depend on how I have configured the routes and home settings in MaterialApp() ? — Reply to this email directly, view it on GitHub <#540 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADM6KXXX4G5QB6JTRXMTVETV4WT4DANCNFSM5P73X63A> . You are receiving this because you were mentioned.Message ID: ***@***.***>

alexobviously · 2022-09-05T12:51:41Z

Thanks for your code by the way @daveshirman. I tried exactly your example with my own onesignal config and it didn't work. So that confirms that there's some very mysterious, potentially device-specific bug (although we have tested on a fairly wide range of devices and none of them worked).
The only other possibility is that we have miscommunicated at some point so I'll just clarify for anyone who comes after this (hopefully someone from onesignal):
The issue is that when apps are cold booted from a notification, i.e. they are opened from a completely closed state by tapping a onesignal notification, the handler is not called when the app opens.

daveshirman · 2022-09-05T13:05:53Z

So strange. I've tested this on a range of physical devices, iOS and Android, from IPhone 6s through to 11 on the latest iOS and Android 13 even. No problems. Cold boot works as described on my post. I'm literally using in production and all is well with several apps. There must be something different in the setup between our code bases or package versions maybe? When you say you tried my example, did you literally just take my code including the App() wrapper widget class and add some test pages instead of PageCalendar etc?

…

On Mon, 5 Sept 2022, 08:51 Alex Baker, ***@***.***> wrote: Thanks for your code by the way @daveshirman <https://github.com/daveshirman>. I tried exactly your example with my own onesignal config and it didn't work. So that confirms that there's some very mysterious, potentially device-specific bug (although we have tested on a fairly wide range of devices and none of them worked). The only other possibility is that we have miscommunicated at some point so I'll just clarify for anyone who comes after this (hopefully someone from onesignal): The issue is that when apps are cold booted from a notification, i.e. they are opened from a completely closed state by tapping a onesignal notification, the handler is not called when the app opens. — Reply to this email directly, view it on GitHub <#540 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADM6KXWQTKE4FCBFWBBOOADV4XUGRANCNFSM5P73X63A> . You are receiving this because you were mentioned.Message ID: ***@***.***>

marcorubagotti · 2022-09-07T10:50:44Z

@daveshirman i have reviewed your workaround and the logic work but on app closed, when i tap the notification the context or navigationKey.currentState return null when try to navigate on a simple page.

I have implemented the navigationKey and onGenerateRoute logic but not work

daveshirman · 2022-09-07T13:15:24Z

Just for anyone who thinks my workaround doesn't work at all, here's a video of it literally working. This behaviour is identical on Android and this is a production app from TestFlight.

What's happening:

First I'm swiping up, showing you that there's no app in the recent apps list.
I tap the notification.
App loads
Saves a key-value store for that particular page from a payload data value in the OneSignal data
App checks that value and loads the appropriate page onto the stack - "Latest Content"

https://imgur.com/a/VE28Iur

glin94 · 2022-09-23T15:45:12Z

I noticed this problem on Xiaomi Note 10s (Android 11 MIUI 12.5.9), but on another device Xiaomi Note Pro 8 (Android 10 MIUI 12.0.5) everything works fine. I also checked on emulator (Android 11 ) and everything works

Oliver-Knight · 2022-11-18T10:41:21Z

@daveshirman I am also work pretty well in the same condition like you. But when I swipe and remove this notification message and didn't get this data anymore. Do u have any solution for this case.

daveshirman · 2022-11-18T10:43:47Z

I'm sorry, I don't understand your question. Could you rephrase or give an example please, so I can try to help?

…

On Fri, Nov 18, 2022 at 5:41 AM Thet Paing ***@***.***> wrote: @daveshirman <https://github.com/daveshirman> I am also work pretty well in the same condition like you. But when I swipe and remove this notification message and didn't get this data anymore. Do u have any solution for this case. — Reply to this email directly, view it on GitHub <#540 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADM6KXUP4GTAHFR7LYOL6LTWI5MNZANCNFSM5P73X63A> . You are receiving this because you were mentioned.Message ID: ***@***.***>

thiagotalma · 2022-12-10T01:50:22Z

Please, someone responsible for the project, pay attention to this issue.

It is very important that it is possible to execute commands when ANY notification is received, regardless of whether the application is in foreground or background or even closed.

On Android I have this option. But I'm migrating my app to Flutter and I'm missing that option.

Flutter is becoming too relevant for the OneSignal package to be ignored.

cc:
@nan-li
@emawby
@tanaynigam
@iAmWillShepherd
@Jeasmine
@brayden-onesignal
@gdeglin
@jkasten2
@leemunroe
@niels-s

KKimj · 2023-01-05T07:01:26Z

Hello!

Do I need to use another push service to route to a specific page while clicking on a push message in Terminated state?

Or has anyone else solved it?
Thank you.

emawby · 2024-01-30T17:46:08Z

Closing issue as stale. If this is still happening in 2024 feel free to tag me to reopen

nan-li added the Need More Information label Apr 12, 2022

emawby added Possible Bug and removed Need More Information labels Jun 24, 2022

emawby mentioned this issue Jun 24, 2022

when app is teminated , we can received the notification, but 【 OneSignal.shared .setNotificationOpenedHandler】 cannot be executed #337

Closed

emawby mentioned this issue Oct 10, 2022

App crashes on notification click #596

Closed

emawby closed this as completed Jan 30, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

setNotificationOpenedHandler does not executes when app is terminated/not opened at all #540

setNotificationOpenedHandler does not executes when app is terminated/not opened at all #540

apoorvpandey0 commented Mar 5, 2022 •

edited by github-actions bot

Loading

andikanugraha commented Mar 10, 2022

mjarpitanand commented Mar 28, 2022

ayechan-maung commented Mar 30, 2022

mjarpitanand commented Apr 2, 2022

AmmarHazem commented Apr 12, 2022

nan-li commented Apr 12, 2022 •

edited

Loading

AmmarHazem commented Apr 13, 2022 •

edited

Loading

apoorvpandey0 commented Apr 13, 2022 •

edited

Loading

marcorubagotti commented Jun 10, 2022

emawby commented Jun 24, 2022

mrqaidi commented Jun 26, 2022 •

edited

Loading

piotruela commented Jul 20, 2022

daveshirman commented Jul 22, 2022 •

edited

Loading

alexobviously commented Jul 30, 2022 •

edited

Loading

daveshirman commented Jul 30, 2022 via email

daveshirman commented Jul 30, 2022 via email

alexobviously commented Jul 30, 2022 •

edited

Loading

daveshirman commented Aug 4, 2022 •

edited

Loading

marcorubagotti commented Sep 5, 2022

daveshirman commented Sep 5, 2022 via email

alexobviously commented Sep 5, 2022

daveshirman commented Sep 5, 2022 via email

marcorubagotti commented Sep 7, 2022

daveshirman commented Sep 7, 2022

glin94 commented Sep 23, 2022

Oliver-Knight commented Nov 18, 2022

daveshirman commented Nov 18, 2022 via email

thiagotalma commented Dec 10, 2022

KKimj commented Jan 5, 2023

emawby commented Jan 30, 2024

setNotificationOpenedHandler does not executes when app is terminated/not opened at all #540

setNotificationOpenedHandler does not executes when app is terminated/not opened at all #540

Comments

apoorvpandey0 commented Mar 5, 2022 • edited by github-actions bot Loading

andikanugraha commented Mar 10, 2022

mjarpitanand commented Mar 28, 2022

ayechan-maung commented Mar 30, 2022

mjarpitanand commented Apr 2, 2022

AmmarHazem commented Apr 12, 2022

nan-li commented Apr 12, 2022 • edited Loading

AmmarHazem commented Apr 13, 2022 • edited Loading

apoorvpandey0 commented Apr 13, 2022 • edited Loading

marcorubagotti commented Jun 10, 2022

emawby commented Jun 24, 2022

mrqaidi commented Jun 26, 2022 • edited Loading

piotruela commented Jul 20, 2022

daveshirman commented Jul 22, 2022 • edited Loading

alexobviously commented Jul 30, 2022 • edited Loading

daveshirman commented Jul 30, 2022 via email

daveshirman commented Jul 30, 2022 via email

alexobviously commented Jul 30, 2022 • edited Loading

daveshirman commented Aug 4, 2022 • edited Loading

marcorubagotti commented Sep 5, 2022

daveshirman commented Sep 5, 2022 via email

alexobviously commented Sep 5, 2022

daveshirman commented Sep 5, 2022 via email

marcorubagotti commented Sep 7, 2022

daveshirman commented Sep 7, 2022

glin94 commented Sep 23, 2022

Oliver-Knight commented Nov 18, 2022

daveshirman commented Nov 18, 2022 via email

thiagotalma commented Dec 10, 2022

KKimj commented Jan 5, 2023

emawby commented Jan 30, 2024

apoorvpandey0 commented Mar 5, 2022 •

edited by github-actions bot

Loading

nan-li commented Apr 12, 2022 •

edited

Loading

AmmarHazem commented Apr 13, 2022 •

edited

Loading

apoorvpandey0 commented Apr 13, 2022 •

edited

Loading

mrqaidi commented Jun 26, 2022 •

edited

Loading

daveshirman commented Jul 22, 2022 •

edited

Loading

alexobviously commented Jul 30, 2022 •

edited

Loading

alexobviously commented Jul 30, 2022 •

edited

Loading

daveshirman commented Aug 4, 2022 •

edited

Loading