Merge pull request #209 from wcalebgray/fcm-data-upgrade

Add passthrough for settings to enable data notification
appfeel · Jun 21, 2024 · b438728 · b438728
2 parents 4b3c3f3 + 5c2d937
commit b438728
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 6 deletions.
diff --git a/src/sendFCM.js b/src/sendFCM.js
@@ -87,7 +87,7 @@ const sendFCM = (regIds, data, settings) => {
 
   const promises = [];
 
-  const fcmMessage = FcmMessage.build(data, regIds);
+  const fcmMessage = FcmMessage.build(data, settings.fcm);
 
   let chunk = 0;
 

diff --git a/src/utils/fcmMessage.js b/src/utils/fcmMessage.js
@@ -33,19 +33,19 @@ class FcmMessage {
     }, {});
   }
 
-  static buildAndroidMessage(params) {
-    const message = buildGcmMessage(params, {});
+  static buildAndroidMessage(params, options) {
+    const message = buildGcmMessage(params, options);
 
     const androidMessage = message.toJson();
 
     androidMessage.ttl = androidMessage.time_to_live * 1000;
+    androidMessage.data = this.normalizeDataParams(androidMessage.data);
 
     delete androidMessage.content_available;
     delete androidMessage.mutable_content;
     delete androidMessage.delay_while_idle;
     delete androidMessage.time_to_live;
     delete androidMessage.dry_run;
-    delete androidMessage.data;
 
     return androidMessage;
   }
@@ -60,7 +60,7 @@ class FcmMessage {
     return { headers: this.normalizeDataParams(headers), payload };
   }
 
-  static build(params) {
+  static build(params, options) {
     const { providersExclude = [], ...fcmMessageParams } = params;
 
     const data = this.normalizeDataParams(fcmMessageParams.custom);
@@ -72,7 +72,10 @@ class FcmMessage {
     }
 
     if (!providersExclude.includes('android')) {
-      createParams.android = this.buildAndroidMessage(fcmMessageParams);
+      createParams.android = this.buildAndroidMessage(
+        fcmMessageParams,
+        options
+      );
     }
 
     return new this(createParams);