@codecademy/tracking 0.17.1-alpha.e28253.0 → 0.17.1

package/CHANGELOG.md

@@ -3,11 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
-### [0.17.1-alpha.e28253.0](https://github.com/Codecademy/client-modules/compare/@codecademy/tracking@0.17.0...@codecademy/tracking@0.17.1-alpha.e28253.0) (2021-12-13)
+### [0.17.1](https://github.com/Codecademy/client-modules/compare/@codecademy/tracking@0.17.0...@codecademy/tracking@0.17.1) (2021-12-17)
 
-**Note:** Version bump only for package @codecademy/tracking
 
+### Bug Fixes
 
+* **tracking:** remove navigator.sendBeacon .bind and add try/catch ([b924d7e](https://github.com/Codecademy/client-modules/commit/b924d7e1ffd22d75adcaf15c9726d10f19b2924e))
 
 
 

package/dist/events/__tests__/track.test.js

@@ -114,6 +114,19 @@ describe('createTracker', function () {
           method: 'POST'
         });
       });
+      it('calls to fetch when beacon throws an error', function () {
+        Object.defineProperty(navigator, 'sendBeacon', {
+          writable: true,
+          value: function value() {
+            throw new Error('Oh no!');
+          }
+        });
+        tracker[event](expectedProps);
+        expect(fetch).toHaveBeenCalledWith('https://www.codecademy.com/analytics/user?utm_source=twitter', {
+          body: expect.any(FormData),
+          method: 'POST'
+        });
+      });
     });
   };
 

package/dist/events/track.js

@@ -39,16 +39,17 @@ export var createTracker = function createTracker(_ref) {
       var k = _ref3[0];
       var v = _ref3[1];
       form.append(k, v.toString());
-    } // Firefox allows users to disable navigator.sendBeacon, and very old Safari versions don't have it.
-
-
-    var sendBeacon = // [WEB-1700]: Additionally, Chrome 79-80 gives "Illegal invocation" with ?., so through 2022 we should support them.
-    // It seems similar to this: https://github.com/vercel/next.js/issues/23856
-    // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
-    navigator.sendBeacon && navigator.sendBeacon.bind(navigator);
+    }
 
-    if (sendBeacon !== null && sendBeacon !== void 0 && sendBeacon(uri, form)) {
-      return;
+    try {
+      // Firefox allows users to disable navigator.sendBeacon, and very old Safari versions don't have it.
+      // [WEB-1700]: Additionally, Chrome 79-80 gives "Illegal invocation" with ?., so through 2022 we should support them.
+      // It seems similar to this: https://github.com/vercel/next.js/issues/23856
+      // eslint-disable-next-line @typescript-eslint/prefer-optional-chain
+      if (navigator.sendBeacon && navigator.sendBeacon(uri, form)) {
+        return;
+      }
+    } catch (_unused) {// Even with the proper scoping, Chrome 79-80 still gives "Illegal invocation" crashes. Sigh.
     } // Either way, we fall back to standard fetch if sendBeacon fails.
     // We don't mind this rejecting with an error because it's tracking, and we'll know if that starts to fail.
     // eslint-disable-next-line @typescript-eslint/no-floating-promises

package/dist/integrations/types.d.ts

@@ -4,10 +4,10 @@ export interface UserIntegrationSummary {
     id: string;
 }
 export interface SegmentAnalytics {
-    identify: (id: string, details: Record<string, string>, options: SegmentAnalyticsOptions) => void;
+    identify(id: string, details: Record<string, string>, options: SegmentAnalyticsOptions): void;
     initialize?: boolean;
-    load: (writeKey: string, options: SegmentAnalyticsOptions) => void;
-    page: () => void;
+    load(writeKey: string, options: SegmentAnalyticsOptions): void;
+    page(): void;
 }
 export interface SegmentDestination {
     category: string;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@codecademy/tracking",
   "description": "Tracking library for Codecademy apps.",
-  "version": "0.17.1-alpha.e28253.0",
+  "version": "0.17.1",
   "author": "Codecademy Engineering <dev@codecademy.com>",
   "module": "./dist/index.js",
   "main": "./dist/index.js",
@@ -36,5 +36,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "1643cfff92132c0ef6f3861e10f0c36fd673fcf3"
+  "gitHead": "f07787036cf95208eb5d78a7a357674c8129fb0f"
 }