From 3ab9ef0fa8ce5a8b81cb5224d067a8e7f1c39a75 Mon Sep 17 00:00:00 2001
From: Kamran Ahmed <kamranahmed.se@gmail.com>
Date: Sat, 16 Nov 2019 13:50:30 +0400
Subject: [PATCH] Do not send analytics for dev mode

---
 components/helmet/index.js | 27 ++++++++++++++++-----------
 lib/gtag.js                | 10 ++++++++++
 2 files changed, 26 insertions(+), 11 deletions(-)

diff --git a/components/helmet/index.js b/components/helmet/index.js
index c4e141bc7..5c9072c9a 100644
--- a/components/helmet/index.js
+++ b/components/helmet/index.js
@@ -1,3 +1,4 @@
+const { PHASE_DEVELOPMENT_SERVER } = require('next/constants');
 import NextHead from 'next/head';
 import siteConfig from 'storage/site';
 import { GA_TRACKING_ID } from 'lib/gtag';
@@ -22,13 +23,13 @@ const Helmet = (props) => (
     <meta name="keywords" content={ siteConfig.keywords.join(',') } />
 
     <meta name="viewport" content="width=device-width, user-scalable=yes, initial-scale=1.0, maximum-scale=3.0, minimum-scale=1.0" />
-    <link rel="canonical" href={ siteConfig.url.web } />
+    <link rel="canonical" href={ siteConfig.url } />
     <meta httpEquiv="Content-Language" content="en" />
 
     <meta property="og:title" content={ prepareTitle(props.title) } />
     <meta property="og:description" content={ prepareDescription(props.description) } />
     <meta property="og:image" content={ siteConfig.logo } />
-    <meta property="og:url" content={ siteConfig.url.web } />
+    <meta property="og:url" content={ siteConfig.url } />
     <meta property="og:type" content="website" />
     <meta property="article:publisher" content={ `https://facebook.com/${siteConfig.facebook}` } />
     <meta property="og:site_name" content={ siteConfig.name } />
@@ -55,15 +56,19 @@ const Helmet = (props) => (
     <link rel="icon" href="/static/manifest/favicon.ico" type="image/x-icon" />
 
     { /* Global Site Tag (gtag.js) - Google Analytics */ }
-    <script async src={ `https://www.googletagmanager.com/gtag/js?id=${GA_TRACKING_ID}` } />
-    <script dangerouslySetInnerHTML={{
-      __html: `
-        window.dataLayer = window.dataLayer || [];
-        function gtag(){dataLayer.push(arguments);}
-        gtag('js', new Date());
-        gtag('config', '${GA_TRACKING_ID}');
-      `,
-    }} />
+    { !PHASE_DEVELOPMENT_SERVER && (
+      <>
+        <script async src={ `https://www.googletagmanager.com/gtag/js?id=${GA_TRACKING_ID}` } />
+        <script dangerouslySetInnerHTML={{
+          __html: `
+            window.dataLayer = window.dataLayer || [];
+            function gtag(){dataLayer.push(arguments);}
+            gtag('js', new Date());
+            gtag('config', '${GA_TRACKING_ID}');
+          `,
+        }} />
+      </>
+    )}
   </NextHead>
 );
 
diff --git a/lib/gtag.js b/lib/gtag.js
index ee264269f..f05a9dbd6 100644
--- a/lib/gtag.js
+++ b/lib/gtag.js
@@ -2,6 +2,11 @@ export const GA_TRACKING_ID = 'UA-139582634-1';
 
 // https://developers.google.com/analytics/devguides/collection/gtagjs/pages
 export const firePageView = url => {
+  if (!window.gtag) {
+    console.warn('Missing GTAG – Analytics disabled');
+    return;
+  }
+
   window.gtag('config', GA_TRACKING_ID, {
     page_path: url,
   })
@@ -9,6 +14,11 @@ export const firePageView = url => {
 
 // https://developers.google.com/analytics/devguides/collection/gtagjs/events
 export const event = ({ action, category, label, value }) => {
+  if (!window.gtag) {
+    console.warn('Missing GTAG – Analytics disabled');
+    return;
+  }
+
   window.gtag('event', action, {
     event_category: category,
     event_label: label,