fix(iterable): update addition of default value for some fields (#2310)

rudderlabs · Jun 30, 2023 · 0f1ebba · 0f1ebba
1 parent c91dadc
commit 0f1ebba
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 12 deletions.
diff --git a/src/v0/destinations/iterable/data/IterableIdentifyConfig.json b/src/v0/destinations/iterable/data/IterableIdentifyConfig.json
@@ -4,10 +4,25 @@
     "sourceKeys": ["traits.email", "context.traits.email"],
     "required": false
   },
+  {
+    "destKey": "preferUserId",
+    "sourceKeys": ["traits.preferUserId", "context.traits.preferUserId"],
+    "metadata": { "defaultValue": true },
+    "required": false
+  },
+  {
+    "destKey": "mergeNestedObjects",
+    "sourceKeys": ["traits.mergeNestedObjects", "context.traits.mergeNestedObjects"],
+    "metadata": { "defaultValue": true },
+    "required": false
+  },
   {
     "destKey": "dataFields",
     "sourceKeys": ["traits", "context.traits"],
-    "required": false
+    "required": false,
+    "metadata": {
+      "excludes": ["preferUserId", "mergeNestedObjects"]
+    }
   },
   {
     "destKey": "userId",

diff --git a/src/v0/destinations/iterable/data/IterableRegisterDeviceTokenConfig.json b/src/v0/destinations/iterable/data/IterableRegisterDeviceTokenConfig.json
@@ -15,5 +15,11 @@
       "anonymousId"
     ],
     "required": false
+  },
+  {
+    "destKey": "preferUserId",
+    "sourceKeys": ["traits.preferUserId", "context.traits.preferUserId"],
+    "metadata": { "defaultValue": true },
+    "required": false
   }
 ]
diff --git a/src/v0/destinations/iterable/util.js b/src/v0/destinations/iterable/util.js
@@ -25,7 +25,6 @@ const identifyDeviceAction = (message) => {
   let rawPayload = {};
   rawPayload = constructPayload(message, mappingConfig[ConfigCategory.IDENTIFY_DEVICE.name]);
   rawPayload.device = constructPayload(message, mappingConfig[ConfigCategory.DEVICE.name]);
-  rawPayload.preferUserId = true;
   if (isAppleFamily(message.context.device.type)) {
     rawPayload.device.platform = 'APNS';
   } else {
@@ -46,8 +45,6 @@ const identifyAction = (message, category) => {
     addExternalIdToTraits(message);
   }
   const rawPayload = constructPayload(message, mappingConfig[category.name]);
-  rawPayload.preferUserId = true;
-  rawPayload.mergeNestedObjects = true;
   validateMandatoryField(rawPayload);
   return rawPayload;
 };
@@ -135,8 +132,6 @@ const trackPurchaseAction = (message, category) => {
   rawPayload = constructPayload(message, mappingConfig[category.name]);
   rawPayload.user = constructPayload(message, mappingConfig[ConfigCategory.IDENTIFY.name]);
   validateMandatoryField(rawPayload.user);
-  rawPayload.user.preferUserId = true;
-  rawPayload.user.mergeNestedObjects = true;
   rawPayload.items = message.properties.products;
   if (rawPayload.items && Array.isArray(rawPayload.items)) {
     rawPayload.items.forEach((el) => {
@@ -186,8 +181,6 @@ const updateCartAction = (message) => {
   const rawPayloadItemArr = [];
   rawPayload.user = constructPayload(message, mappingConfig[ConfigCategory.IDENTIFY.name]);
   validateMandatoryField(rawPayload.user);
-  rawPayload.user.preferUserId = true;
-  rawPayload.user.mergeNestedObjects = true;
   if (rawPayload.items && Array.isArray(rawPayload.items)) {
     rawPayload.items.forEach((el) => {
       const element = constructPayload(el, mappingConfig[ConfigCategory.PRODUCT.name]);

diff --git a/test/__tests__/data/iterable_input.json b/test/__tests__/data/iterable_input.json
@@ -47,7 +47,8 @@
           "city": "Bangalore",
           "name": "manashi",
           "email": "[email protected]",
-          "country": "India"
+          "country": "India",
+          "preferUserId": false
         },
         "library": {
           "name": "analytics-node",
@@ -636,7 +637,8 @@
           "version": "1.0.0"
         },
         "traits": {
-          "email": "[email protected]"
+          "email": "[email protected]",
+          "mergeNestedObjects": false
         },
         "library": {
           "name": "RudderLabs JavaScript SDK",

diff --git a/test/__tests__/data/iterable_output.json b/test/__tests__/data/iterable_output.json
@@ -17,7 +17,7 @@
           "email": "[email protected]",
           "country": "India"
         },
-        "preferUserId": true,
+        "preferUserId": false,
         "mergeNestedObjects": true
       }
     },
@@ -336,7 +336,7 @@
           },
           "userId": "123456",
           "preferUserId": true,
-          "mergeNestedObjects": true
+          "mergeNestedObjects": false
         },
         "XML": {},
         "JSON_ARRAY": {},