@vouchfor/embeds 0.0.0-experiment.899f364 → 0.0.0-experiment.8a05fac

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vouchfor/embeds",
-  "version": "0.0.0-experiment.899f364",
+  "version": "0.0.0-experiment.8a05fac",
   "license": "MIT",
   "author": "Aaron Williams",
   "main": "dist/es/embeds.js",
@@ -36,7 +36,7 @@
   },
   "dependencies": {
     "@lit/task": "^1.0.0",
-    "@vouchfor/media-player": "0.0.0-experiment.899f364",
+    "@vouchfor/media-player": "0.0.0-experiment.8a05fac",
     "uuid": "^9.0.1"
   },
   "peerDependencies": {

package/src/components/Embed/controllers/tracking.ts

@@ -1,3 +1,4 @@
+/* eslint-disable max-lines */
 import { v4 as uuidv4 } from 'uuid';
 
 import type { Embed } from '..';
@@ -6,7 +7,8 @@ import type { ReactiveController, ReactiveControllerHost } from 'lit';
 
 import { getEnvUrls } from '~/utils/env';
 
-const STREAMED_THROTTLE = 10000;
+// In seconds due to checking against node.currentTime
+const STREAMED_THROTTLE = 10;
 
 type EmbedHost = ReactiveControllerHost & Embed;
 
@@ -37,7 +39,7 @@ class TrackingController implements ReactiveController {
   private _hasLoaded: BooleanMap = {};
   private _answersViewed: BooleanMap = {};
   private _streamedTime: TimeMap = {};
-  private _streamedPrevTimestamp: TimeMap = {};
+  private _streamLatestTime: TimeMap = {};
 
   constructor(host: EmbedHost) {
     this.host = host;
@@ -132,22 +134,24 @@ class TrackingController implements ReactiveController {
     const { publicApiUrl } = getEnvUrls(this.host.env);
     const { client, tab, request, visitor } = this._getUids();
 
-    if (this.host.enableTracking) {
-      navigator.sendBeacon(
-        `${publicApiUrl}/api/events`,
-        JSON.stringify({
-          event,
-          payload,
-          context: {
-            'x-uid-client': client,
-            'x-uid-tab': tab,
-            'x-uid-request': request,
-            'x-uid-visitor': visitor,
-            'x-reporting-metadata': this._getReportingMetadata()
-          }
-        })
-      );
+    if (this.host.disableTracking) {
+      return;
     }
+
+    navigator.sendBeacon(
+      `${publicApiUrl}/api/events`,
+      JSON.stringify({
+        event,
+        payload,
+        context: {
+          'x-uid-client': client,
+          'x-uid-tab': tab,
+          'x-uid-request': request,
+          'x-uid-visitor': visitor,
+          'x-reporting-metadata': this._getReportingMetadata()
+        }
+      })
+    );
   };
 
   private _handleVouchLoaded = ({ detail: vouchId }: CustomEvent<string>) => {
@@ -181,31 +185,33 @@ class TrackingController implements ReactiveController {
     }
   };
 
-  private _handleVideoPlay = ({ detail: { id, node } }: CustomEvent<VideoEventDetail>) => {
+  private _handleVideoPlay = ({ detail: { id, key, node } }: CustomEvent<VideoEventDetail>) => {
     const vouchId = this._findVouchId();
 
     if (!vouchId) {
       return;
     }
+
     // Only increment play count once per session
-    if (!this._answersViewed[id]) {
+    if (!this._answersViewed[key]) {
       this._sendTrackingEvent('VOUCH_RESPONSE_VIEWED', {
         vouchId,
         answerId: id
       });
-      this._answersViewed[id] = true;
+      this._answersViewed[key] = true;
     }
-    this._streamedTime[id] = node.currentTime;
-    this._streamedPrevTimestamp[id] = Date.now();
+
+    this._streamedTime[key] = node.currentTime;
+    this._streamLatestTime[key] = node.currentTime;
   };
 
-  private _handleVideoTimeUpdate = ({ detail: { id, node } }: CustomEvent<VideoEventDetail>) => {
+  private _handleVideoTimeUpdate = ({ detail: { id, key, node } }: CustomEvent<VideoEventDetail>) => {
     const vouchId = this._findVouchId();
 
     if (!vouchId) {
       return;
     }
-    const currentTimestamp = Date.now();
+
     if (
       node.currentTime &&
       !node.paused &&
@@ -213,20 +219,23 @@ class TrackingController implements ReactiveController {
       // Only fire the video seeked event when this video is the active one
       id === this.host.scene?.video?.id &&
       // Throttle the frequency that we send streamed events while playing
-      currentTimestamp - this._streamedPrevTimestamp[id] > STREAMED_THROTTLE
+      node.currentTime - this._streamedTime[key] > STREAMED_THROTTLE
     ) {
       this._sendTrackingEvent('VIDEO_STREAMED', {
         vouchId,
         answerId: id,
-        streamStart: this._streamedTime[id],
+        streamStart: this._streamedTime[key],
         streamEnd: node.currentTime
       });
-      this._streamedTime[id] = node.currentTime;
-      this._streamedPrevTimestamp[id] = currentTimestamp;
+      this._streamedTime[key] = node.currentTime;
+    }
+
+    if (!this.host.paused) {
+      this._streamLatestTime[key] = node.currentTime;
     }
   };
 
-  private _handleVideoPause = ({ detail: { id, node } }: CustomEvent<VideoEventDetail>) => {
+  private _handleVideoPause = ({ detail: { id, key } }: CustomEvent<VideoEventDetail>) => {
     const vouchId = this._findVouchId();
 
     if (!vouchId) {
@@ -234,19 +243,20 @@ class TrackingController implements ReactiveController {
     }
 
     // Don't send a tracking event if the video pauses when seeking backwards
-    if (node.currentTime > this._streamedTime[id]) {
+    if (this._streamLatestTime[key] > this._streamedTime[key] + 0.5) {
       // Send a video streamed event any time the video pauses then reset the streamed state
       // We do this to capture the last bit of time that the video was played between the previous
       // stream event and the video being paused manually or stopping because it ended
       this._sendTrackingEvent('VIDEO_STREAMED', {
         vouchId,
         answerId: id,
-        streamStart: this._streamedTime[id],
-        streamEnd: node.currentTime
+        streamStart: this._streamedTime[key],
+        streamEnd: this._streamLatestTime[key]
       });
     }
-    delete this._streamedTime[id];
-    delete this._streamedPrevTimestamp[id];
+
+    delete this._streamedTime[key];
+    delete this._streamLatestTime[key];
   };
 
   hostConnected() {

package/src/components/Embed/index.ts

@@ -17,7 +17,7 @@ import '@vouchfor/media-player';
 type EmbedProps = Pick<MediaPlayerProps, 'data' | 'aspectRatio' | 'preload' | 'autoplay' | 'controls'> & {
   env: Environment;
   apiKey: string;
-  enableTracking?: boolean;
+  disableTracking?: boolean;
   trackingSource?: string;
   vouchId?: string;
   templateId?: string;
@@ -33,7 +33,7 @@ class Embed extends LitElement {
 
   @property({ type: String }) env: EmbedProps['env'] = 'prod';
   @property({ type: String }) apiKey: EmbedProps['apiKey'] = '';
-  @property({ type: Boolean }) enableTracking: EmbedProps['enableTracking'] = true;
+  @property({ type: Boolean }) disableTracking: EmbedProps['disableTracking'] = false;
   @property({ type: String }) trackingSource: EmbedProps['trackingSource'] = 'embed';
 
   @property({ type: Array }) controls: EmbedProps['controls'];
@@ -59,6 +59,7 @@ class Embed extends LitElement {
     'waiting',
 
     'video:loadeddata',
+    'video:seeking',
     'video:seeked',
     'video:play',
     'video:playing',