@segment/analytics.js-integration-clevertap - npm Package Compare versions

Comparing version 1.0.1 to 1.0.2

HISTORY.md

@@ -0,1 +1,6 @@
+1.0.2 / 2017-08-30
+==================
+
+  * Fix region bug fix 
+
 1.0.1 / 2017-08-24
@@ -2,0 +7,0 @@ ==================

lib/index.js

@@ -35,4 +35,7 @@ 'use strict';
   window.clevertap.account.push({ id: this.options.clevertap_account_id });
-  if (this.options.region) {
-    window.clevertap.region = this.options.region;
+  var region = this.options.region;
+  if (region && is.string(region)) {
+    // the hardcoded value actually returns 'in.' intentionally w the period because it is used for the direct integration
+    // and since dealing with mongo is much more painful, we will strip here
+    window.clevertap.region = region.replace('.', '');
   }
@@ -39,0 +42,0 @@ var protocol = useHttps() ? 'https': 'http';

package.json

 {
   "name": "@segment/analytics.js-integration-clevertap",
   "description": "The CleverTap analytics.js integration.",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "keywords": [
@@ -6,0 +6,0 @@ "analytics.js",

test/index.test.js

@@ -63,2 +63,9 @@ 'use strict';
       });
+
+      it('should strip periods in region', function() {
+        var region = 'in.';
+        clevertap.options.region = region;
+        analytics.initialize();
+        analytics.assert(window.clevertap.region === 'in');
+      });
     });
@@ -65,0 +72,0 @@ });

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

20633

increased by2.7%

Lines of code

412

increased by2.23%

No dependency changes