iOS OnNotificationOpened not firing from terminated app? - here's why

[Lydecker]

Just a heads up - I was trying to figure out why OnNotificationOpened wasn't being fired when the app was being launched from a push notification (i.e the app is terminated, and you launch it by tapping on a notification). It turns out it was because there is core code commented out in this library's source:

As soon as I uncommented this - everything works on iOS as expected!

Would love an explanation as to why this is commented out....

[mimunoz]

@rdelrosario can you explain please?

[almirvuk]

Same here :( OnNotificationOpened is not called when the app is terminated and push notification is clicked.

Is there any additional step which we need to add in order to make this work, in replacement for that commented part of code? @rdelrosario Thanks!!!

[rdelrosario]

Actually I will update the code over the weekend uncommenting this and working with other needed fixes.

The reason this was commented was because at some point didn't seem to be needed.

[Lydecker]

Hi @rdelrosario ,

Thanks for spending time working on this this weekend.

Please can you also test autoRegister = false on BOTH Android and iOS. Currently it works on neither platform as expected (although the incorrect behaviour is different on each platform) #303 #307

[rdelrosario]

Can you let me know on which version is happening? On latest beta version?

…

On Fri, May 29, 2020, 6:35 PM Lydecker ***@***.***> wrote: Hi @rdelrosario <https://github.com/rdelrosario> , Thanks for spending time working on this this weekend. Please can you also test autoRegister = false on BOTH Android and iOS. Currently it works on neither platform as expected (although the incorrect behaviour is different on each platform) #303 <#303> #307 <#307> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#315 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AATOAJ6NYKPVIT7LPLH5ZULRUA2EBANCNFSM4MZ6WN4A> .

[ivandharo]

@rdelrosario
Hi rendy, i want to report this bug, when i use the code below to change default channelid and name, firebase still sending notifications through miscellaneous channel, why it's happening?

FirebasePushNotificationManager.DefaultNotificationChannelId = "General";

btw, great work with this plugin.

[Lydecker]

Can you let me know on which version is happening? On latest beta version?

Hi @rdelrosario - people are experiencing the autoRegister = false issues on the latest version 3.1.6.

[almirvuk]

Hi @rdelrosario thank you for your weekend work on this one!

I have updated the project to 3.3.10, and I am experiencing the same behaviour, works fine when app is running and not terminated.

My setup code:

 FirebasePushNotificationManager.Initialize(options);

   FirebasePushNotificationManager.CurrentNotificationPresentationOption =
            UserNotifications.UNNotificationPresentationOptions.Alert |
            UserNotifications.UNNotificationPresentationOptions.Badge;

     return base.FinishedLaunching(app, options);
 }

 public override void RegisteredForRemoteNotifications(UIApplication application, NSData deviceToken)
 {
      FirebasePushNotificationManager.DidRegisterRemoteNotifications(deviceToken);
 }

 public override void FailedToRegisterForRemoteNotifications(UIApplication application, NSError error)
 {
        FirebasePushNotificationManager.RemoteNotificationRegistrationFailed(error);
 }
 
 public override void DidReceiveRemoteNotification(UIApplication application, NSDictionary userInfo, Action<UIBackgroundFetchResult> completionHandler)
 {
        FirebasePushNotificationManager.DidReceiveMessage(userInfo);
        
        completionHandler(UIBackgroundFetchResult.NewData);
  }

Notification payload contains, priority: high, for iOS I am using the notification part of payload, and notification is received but OnNotificationOpened, is not called when the app is terminated completely. Do I need to add something else, is it something wrong on my side of setup.

Thanks!

[almirvuk]

I was testing this via multiple notifications, and one of the times this worked. Not sure why I have also added some bool value to Xamarin.Essentials Preferences just to set it before any other actions inside of OnNotificationOpened, just to check if my sample project (MasterDetail template from VS New Project) has race condition between constructor and OnNotificationOpened, but the value stays false.

But it is weird that it worked once in this testing, once again the problem is when the app is closed, terminated completely. I will try more to see if it will trigger the OnNotificationOpened again.

[almirvuk]

@Lydecker have you tested new version?

[almirvuk]

Just tested again, it has some kind of issue when the app is terminated. I received the notification, opened the app manually by clicking on the icon, and then tapped on the received notification and it worked, it triggered the OnNotificationOpened. But only if app was running already.

On Android same XF shared project code, works fine, user is navigated to second page when OnNotificationOpened is triggered both in running and terminated mode.

[almirvuk]

I just moved the CrossFirebasePushNotification.Current.OnNotificationOpened to the constructor and it works fine like that, previously it was inside of OnStart method just like in sample project. It seems to work just fine when that part of the code is inside of constructor. I will test this more and let you know.

[Lydecker]

Hi haven't had a chance to test yet... will do soon.

What do you mean 'ctor' ?

[almirvuk]

Constructor... I just edited the replies.