Only send beacons if the feature is not disabled (github#20282)

JamesMGreene · web-flow · commit 0e90603d945a · 2021-07-02T19:42:42.000Z
Fixes https://github.com/github/docs-internal/issues/20263
diff --git a/javascripts/events.ts b/javascripts/events.ts
@@ -103,8 +103,13 @@ export function sendEvent({ type, version = '1.0.0', ...props }: SendEventProps)
 
     ...props,
   }
-  const blob = new Blob([JSON.stringify(body)], { type: 'application/json' })
-  navigator.sendBeacon('/events', blob)
+
+  // Only send the beacon if the feature is not disabled in the user's browser
+  if (navigator?.sendBeacon) {
+    const blob = new Blob([JSON.stringify(body)], { type: 'application/json' })
+    navigator.sendBeacon('/events', blob)
+  }
+
   return body
 }
 

Original file line number	Diff line number	Diff line change
`@@ -103,8 +103,13 @@ export function sendEvent({ type, version = '1.0.0', ...props }: SendEventProps)`
`103`	`103`
`104`	`104`	`...props,`
`105`	`105`	`}`
`106`		`- const blob = new Blob([JSON.stringify(body)], { type: 'application/json' })`
`107`		`- navigator.sendBeacon('/events', blob)`
	`106`	`+`
	`107`	`+ // Only send the beacon if the feature is not disabled in the user's browser`
	`108`	`+ if (navigator?.sendBeacon) {`
	`109`	`+ const blob = new Blob([JSON.stringify(body)], { type: 'application/json' })`
	`110`	`+ navigator.sendBeacon('/events', blob)`
	`111`	`+ }`
	`112`	`+`
`108`	`113`	`return body`
`109`	`114`	`}`
`110`	`115`