Merge pull request #118 from usermaven/next

Next

packages/javascript-sdk/src/core/client.ts

@@ -351,7 +351,7 @@ export class UsermavenClient {
         let processedProps = eventProps || {};
 
         const payload: any = {
-            event_id: generateId(),
+            event_id: '',
             user: {
                 anonymous_id: this.anonymousId,
                 id: userId,

packages/javascript-sdk/src/tracking/autocapture.ts

@@ -159,14 +159,21 @@ class AutoCapture {
 
     private getElementsJson(targetElementList: Element[], e: Event): any[] {
         const elementsJson: any[] = [];
-        let href;
+        let href: false | string | null = null;
+        let explicitNoCapture = false;
 
         _each(targetElementList, (el) => {
             if (isTag(el, 'a')) {
                 href = el.getAttribute('href');
                 href = shouldCaptureElement(el) && shouldCaptureValue(href) && href;
             }
 
+            // Check for 'ph-no-capture' class
+            const classes = getClassName(el).split(' ');
+            if (_includes(classes, 'ph-no-capture')) {
+                explicitNoCapture = true;
+            }
+
             elementsJson.push(this.getPropertiesFromElement(el));
         });
 
@@ -178,7 +185,7 @@ class AutoCapture {
             elementsJson[0]['attr__href'] = this.sanitizeUrl(href);
         }
 
-        return elementsJson;
+        return explicitNoCapture ? [] : elementsJson;
     }
 
     private getPropertiesFromElement(element: Element): any {