[ios][messaging] onMessaggeReceived callback not triggered when showNotificationsWhenInForeground is true #1371

Author: EddyVerbruggen

demo-push/app/push-view-model.ts

@@ -48,6 +48,7 @@ export class PushViewModel extends Observable {
     messaging.getCurrentPushToken()
         .then(token => {
           // may be null/undefined if not known yet
+          console.log(token);
           alert({
             title: "Current Push Token",
             message: (!token ? "Not received yet (note that on iOS this does not work on a simulator)" : token + ("\n\nSee the console log if you want to copy-paste it.")),
@@ -192,7 +193,7 @@ export class PushViewModel extends Observable {
       // Whether you want this plugin to always handle the notifications when the app is in foreground.
       // Currently used on iOS only. Default false.
       // When false, you can still force showing it when the app is in the foreground by adding 'showWhenInForeground' to the notification as mentioned in the readme.
-      showNotificationsWhenInForeground: false
+      showNotificationsWhenInForeground: true
     })
         .then(() => console.log(">>>> Registered for push"))
         .catch(err => console.log(">>>> Failed to register for push"));

docs/MESSAGING.md

@@ -198,6 +198,8 @@ This results in a payload of:
 - App in the foreground: `{"foo":"bar", "gcm.message_id":"0:1522952720644653%3194ccac3194ccac", "foreground":true, "title":"My title", "body":"My text"}`
 - App in the background: `{"foo":"bar", "gcm.message_id":"0:1522952737879515%3194ccac3194ccac", "title":"My title", "body":"My text", "foreground":false}`
 
+Note that on iOS, when the notification was tapped, the payload will contain `notificationTapped: true`. That may help you with some logic in your app.
+
 ### (iOS) background notification example
 
 > Take note of `content_available:true` here

src/firebase.d.ts

@@ -548,6 +548,11 @@ export interface Message {
    * Any other data you may have added to the notification.
    */
   data: any;
+  /**
+   * Indicates whether or not the notification was tapped.
+   * iOS only.
+   */
+  notificationTapped?: boolean;
 }
 
 export function init(options?: InitOptions): Promise<any>;

src/messaging/messaging.ios.ts

@@ -407,9 +407,14 @@ function _registerForRemoteNotifications(resolve?, reject?) {
             userInfoJSON,
             inputText
           });
+
           if (_notificationActionTakenCallback) {
             _processPendingActionTakenNotifications();
           }
+
+          userInfoJSON.notificationTapped = actionIdentifier === UNNotificationDefaultActionIdentifier;
+        } else {
+          userInfoJSON.notificationTapped = false;
         }
 
         userInfoJSON.foreground = UIApplication.sharedApplication.applicationState === UIApplicationState.Active;