Fix b/232072111

Author: zwu52

packages/messaging/src/helpers/externalizePayload.test.ts

@@ -26,7 +26,8 @@ describe('externalizePayload', () => {
       notification: {
         title: 'title',
         body: 'body',
-        image: 'image'
+        image: 'image',
+        click_action: "https://www.self_orgin.com",
       },
       from: 'from',
       // eslint-disable-next-line camelcase
@@ -39,7 +40,10 @@ describe('externalizePayload', () => {
       notification: { title: 'title', body: 'body', image: 'image' },
       from: 'from',
       collapseKey: 'collapse',
-      messageId: 'mid'
+      messageId: 'mid',
+      fcmOptions: {
+        link: "https://www.self_orgin.com"
+      },
     };
     expect(externalizePayload(internalPayload)).to.deep.equal(payload);
   });
@@ -62,7 +66,8 @@ describe('externalizePayload', () => {
       data: { foo: 'foo', bar: 'bar', baz: 'baz' },
       from: 'from',
       collapseKey: 'collapse',
-      messageId: 'mid'
+      messageId: 'mid',
+      
     };
     expect(externalizePayload(internalPayload)).to.deep.equal(payload);
   });

packages/messaging/src/helpers/externalizePayload.ts

@@ -83,7 +83,7 @@ function propagateFcmOptions(
 
   payload.fcmOptions = {};
 
-  const link = messagePayloadInternal.fcmOptions!.link;
+  const link = messagePayloadInternal.fcmOptions?.link ?? messagePayloadInternal.notification?.click_action;
   if (!!link) {
     payload.fcmOptions!.link = link;
   }