update klaviyo integration to support exchange ID (#618)

Author: chrisconlon-klaviyo

integrations/klaviyo/lib/index.js

@@ -21,7 +21,7 @@ var Klaviyo = (module.exports = integration('Klaviyo')
   .global('_learnq')
   .option('apiKey', '')
   .option('enforceEmail', false)
-  .tag('<script src="//a.klaviyo.com/media/js/analytics/analytics.js">'));
+  .tag('<script src="//static.klaviyo.com/onsite/js/klaviyo.js">'));
 
 /**
  * Initialize.
@@ -58,23 +58,31 @@ Klaviyo.prototype.loaded = function() {
  */
 
 Klaviyo.prototype.identify = function(identify) {
-  // if you don't send either userId or email, noop
-  if (!identify.userId() && !identify.email()) return;
   // TODO: should map/alias the rest of the reserved props
   var traitAliases = {
     email: '$email',
     id: '$id',
     firstName: '$first_name',
     lastName: '$last_name',
     phone: '$phone_number',
-    title: '$title'
+    title: '$title',
+    exchangeId: '$exchange_id'
   };
-  // don't add $id or id if email only option is enforced to prevent some edge case dupe profile issues with Klaviyo API
+
   var traits = identify.traits(traitAliases);
+
+  // If there is not a user ID, email address, or an exchange ID, noop
+  //
+  // NOTE: An exchange ID is an encrypted identifier that Klaviyo
+  //       consumes server-side. It is considered a valid identifier.
+  if (!identify.userId() && !identify.email() && !traits.$exchange_id) return;
+
+  // don't add $id or id if email only option is enforced to prevent some edge case dupe profile issues with Klaviyo API
   if (this.options.enforceEmail) {
     remove(traits, 'id');
     remove(traits, '$id');
   }
+
   // if you enforce email but you don't send email, noop
   if (this.options.enforceEmail && !traits.$email) return;
   push('identify', traits);

integrations/klaviyo/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@segment/analytics.js-integration-klaviyo",
   "description": "The Klaviyo analytics.js integration.",
-  "version": "2.2.3",
+  "version": "2.2.4",
   "keywords": [
     "analytics.js",
     "analytics.js-integration",

integrations/klaviyo/test/index.test.js

@@ -114,6 +114,17 @@ describe('Klaviyo', function() {
         ]);
       });
 
+      it('should send an $exchange_id and traits', function() {
+        analytics.identify(undefined, {
+          $exchange_id: 'exchange-id',
+          foo: true
+        });
+        analytics.called(window._learnq.push, [
+          'identify',
+          { $exchange_id: 'exchange-id', foo: true }
+        ]);
+      });
+
       it('should alias traits', function() {
         analytics.identify('id', {
           email: 'name@example.com',