Merge pull request #2656 from mehrzad-fs/bot-error-disable-track

Mehrzad / track error if isProduction

Author: Aaron

src/botPage/view/blockly/index.js

@@ -26,7 +26,7 @@ import { getLanguage } from '../../../common/lang';
 import { observer as globalObserver } from '../../../common/utils/observer';
 import { showDialog } from '../../bot/tools';
 import GTM from '../../../common/gtm';
-import { parseQueryString } from '../../../common/utils/tools';
+import { parseQueryString, isProduction } from '../../../common/utils/tools';
 import { TrackJSError } from '../logger';
 import config from '../../common/const';
 
@@ -56,7 +56,7 @@ const disposeBlocksWithLoaders = () => {
 
 const marketsWereRemoved = xml => {
     if (!Array.from(xml.children).every(block => !removeUnavailableMarkets(block))) {
-        if (window.trackJs) {
+        if (window.trackJs && isProduction()) {
             trackJs.track('Invalid financial market');
         }
         showDialog({

src/botPage/view/logger.js

@@ -1,5 +1,6 @@
 import { observer as globalObserver } from '../../common/utils/observer';
 import { getToken } from '../../common/utils/storageManager';
+import { isProduction } from '../../common/utils/tools';
 
 const log = (type, ...args) => {
     if (type === 'warn') {
@@ -66,7 +67,7 @@ const notifyError = error => {
 
     notify({ className: 'error', message, position: 'right' });
 
-    if (trackJs) {
+    if (trackJs && isProduction()) {
         trackJs.console.log(error);
         trackJs.track(code || error.name);
     }

src/common/integrations/GoogleDrive.js

@@ -1,7 +1,7 @@
 /* global google,gapi */
 import { getLanguage } from '../lang';
 import { observer as globalObserver } from '../utils/observer';
-import { translate } from '../utils/tools';
+import { translate, isProduction } from '../utils/tools';
 import config from '../../botPage/common/const';
 import { load } from '../../botPage/view/blockly';
 import { TrackJSError } from '../../botPage/view/logger';
@@ -42,7 +42,7 @@ class GoogleDrive {
                                 .removeClass('invisible');
                         },
                         error => {
-                            if (window.trackJs) {
+                            if (window.trackJs && isProduction()) {
                                 trackJs.track(
                                     `${translate(
                                         'There was an error initialising Google Drive'
@@ -53,7 +53,7 @@ class GoogleDrive {
                     );
             },
             onerror: error => {
-                if (window.trackJs) {
+                if (window.trackJs && isProduction()) {
                     trackJs.track(
                         `${translate('There was an error loading Google Drive libraries')} - Error: ${JSON.stringify(
                             error