From eecfa0161d9f7abb23af02ba946d62d27fceb18e Mon Sep 17 00:00:00 2001 From: David Russell Date: Mon, 7 Apr 2025 11:00:30 +0100 Subject: [PATCH 1/2] adds opt in service for localhost --- src/browser/index.tsx | 6 ++++-- src/browser/services/preview-optin-service.ts | 12 ++++++++++++ 2 files changed, 16 insertions(+), 2 deletions(-) create mode 100644 src/browser/services/preview-optin-service.ts diff --git a/src/browser/index.tsx b/src/browser/index.tsx index 89aa5244229..c4d3c7ac8c4 100644 --- a/src/browser/index.tsx +++ b/src/browser/index.tsx @@ -23,14 +23,16 @@ import ReactDOM from 'react-dom' import AppInit, { setupSentry } from './AppInit' import './init' import { navigateToPreview } from './modules/Stream/StartPreviewFrame' +import { optedInByLocalhost } from 'browser-services/preview-optin-service' setupSentry() + ;(async () => { - const doesPreferQuery = localStorage.getItem('prefersOldBrowser') === 'false' + const optedInToPreview = optedInByLocalhost() try { const response = await fetch('./preview/manifest.json') if (response.status === 200) { - if (doesPreferQuery) { + if (optedInToPreview) { navigateToPreview() } else { localStorage.setItem('previewAvailable', 'true') diff --git a/src/browser/services/preview-optin-service.ts b/src/browser/services/preview-optin-service.ts new file mode 100644 index 00000000000..227692e8b5a --- /dev/null +++ b/src/browser/services/preview-optin-service.ts @@ -0,0 +1,12 @@ +const notOptedOutOfPreview = (): boolean => { + const prefersOldBrowser = localStorage.getItem('prefersOldBrowser') + const doesPreferQuery = prefersOldBrowser === 'false' + + return doesPreferQuery || prefersOldBrowser === null +} + +export const optedInByLocalhost = (): boolean => { + console.log('notOptedOutOfPreview', notOptedOutOfPreview()) + console.log('window.location.hostname', window.location.hostname) + return notOptedOutOfPreview() && window.location.hostname === 'localhost' +} From 55e54c0715c94bb9e552260c2de4dd9f9a76c89e Mon Sep 17 00:00:00 2001 From: David Russell Date: Mon, 7 Apr 2025 11:01:50 +0100 Subject: [PATCH 2/2] removes console logging --- src/browser/services/preview-optin-service.ts | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/browser/services/preview-optin-service.ts b/src/browser/services/preview-optin-service.ts index 227692e8b5a..df8becf76c2 100644 --- a/src/browser/services/preview-optin-service.ts +++ b/src/browser/services/preview-optin-service.ts @@ -6,7 +6,5 @@ const notOptedOutOfPreview = (): boolean => { } export const optedInByLocalhost = (): boolean => { - console.log('notOptedOutOfPreview', notOptedOutOfPreview()) - console.log('window.location.hostname', window.location.hostname) return notOptedOutOfPreview() && window.location.hostname === 'localhost' }