CrossGeeks / FirebasePushNotificationPlugin

Firebase Push Notification Plugin for Xamarin iOS and Android
MIT License
394 stars 178 forks source link

OnNotificationReceived not called on iOS 13.2 & 13.3 notification from FCM server #321

Open pranavtiw opened 4 years ago

pranavtiw commented 4 years ago

If you are creating an issue for a BUG please fill out this information. If you are asking a question or requesting a feature you can delete the sections below.

Failure to fill out this information will result in this issue being closed. If you post a full stack trace in a bug it will be closed, please post it to http://gist.github.com and then post the link here.

Bug Information

Version Number of Plugin:3.1.6 Device Tested On:iOS 13.2 & iOS 13.3 iPhone 6 Simulator Tested On:None Version of VS: 8.6 Version of Xamarin: Xamarin.iOS Version: 13.18.1.31 Versions of other things you are using:

Steps to reproduce the Behavior

From FCM fire a Push notification . Notification works and OnNotificationReceived called on all 12 versions but 13 onwards its not hitting on notification received.

Expected Behavior

On notification Received should be called on iOS 13 versions .Specifically observed on 13.2 & 13.3.

Actual Behavior

On notification Received is not called on iOS 13 versions .Specifically observed on 13.2 & 13.3. Its observed 30% of the time on 13 versions .70% of the time its working fine.

Code snippet

Screenshots

almirvuk commented 4 years ago

Same here :( ... Works fine on iOS 12, but on iOS 13.5 there is no notification.

@rdelrosario is there any news about this one? :) Thanks!

rdelrosario commented 4 years ago

Please try on the latest version

almirvuk commented 4 years ago

Same again, it does not work on iPhone X, 13.5. The same app, same code is working just fine on iOS 12. Notifications were received as expected.

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, and content available, I am using the notification part of the payload and Remote notification is checked in Info.plist file. Do I need to add something else, is it something wrong on my side of the setup.

I just tried to send the test notification from Firebase console: /notification/compose, and notification is not received as well.

Again, this is working just fine for iOS 12. 🙂

Thanks!

pranavtiw commented 4 years ago

Even I tried Updating the package ...Issue still exists

almirvuk commented 4 years ago

Updated my iPhone X, to 13.5.1 yesterday and the same sample project works fine now. :)

@rdelrosario Do you have any clue about what could be changed for other versions (lower) iOS 13 versions.

almirvuk commented 4 years ago

@pranavtiw is there any chance for you to update the test device and try it again?