diff --git a/ui/analytics.js b/ui/analytics.js index ff92f9cf1..79911b54d 100644 --- a/ui/analytics.js +++ b/ui/analytics.js @@ -87,19 +87,20 @@ if (window.localStorage.getItem(SHARE_INTERNAL) === 'true') internalAnalyticsEna * @returns {String} */ function getDeviceType() { - if (!IS_WEB) { - return 'elt'; - } - const userAgent = navigator.userAgent || navigator.vendor || window.opera; - - if (/android/i.test(userAgent)) { - return 'adr'; - } - - // iOS detection from: http://stackoverflow.com/a/9039885/177710 - if (/iPad|iPhone|iPod/.test(userAgent) && !window.MSStream) { - return 'ios'; - } + // We may not care what the device is if it's in a web browser. Commenting out for now. + // if (!IS_WEB) { + // return 'elt'; + // } + // const userAgent = navigator.userAgent || navigator.vendor || window.opera; + // + // if (/android/i.test(userAgent)) { + // return 'adr'; + // } + // + // // iOS detection from: http://stackoverflow.com/a/9039885/177710 + // if (/iPad|iPhone|iPod/.test(userAgent) && !window.MSStream) { + // return 'ios'; + // } // default as web, this can be optimized return 'web'; @@ -168,7 +169,7 @@ function startWatchmanIntervalIfNotRunning() { lastSentTime = new Date(); // only set an interval if analytics are enabled and is prod - if (internalAnalyticsEnabled && isProduction) { + if (isProduction && IS_WEB) { watchmanInterval = setInterval(sendAndResetWatchmanData, 1000 * SEND_DATA_TO_WATCHMAN_INTERVAL); } }