@codecademy/tracking 0.17.0 → 0.17.1-alpha.cb4d1f.0

package/CHANGELOG.md

@@ -3,6 +3,15 @@
 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.cb4d1f.0](https://github.com/Codecademy/client-modules/compare/@codecademy/tracking@0.17.0...@codecademy/tracking@0.17.1-alpha.cb4d1f.0) (2021-12-17)
+
+
+### Bug Fixes
+
+* **tracking:** directly call navigator.sendBeacon without .bind ([3e4d58b](https://github.com/Codecademy/client-modules/commit/3e4d58bc2aa8e0832cd68401319837efa3d0ad90))
+
+
+
 ## [0.17.0](https://github.com/Codecademy/client-modules/compare/@codecademy/tracking@0.16.1...@codecademy/tracking@0.17.0) (2021-11-30)
 
 

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/package.json

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