Merge pull request #1363 from mzedel/fix/livechat-errors-in-dev

made the inclusion of the livechat depend on the environment the bundle was created for

Author: mzedel

src/js/components/app.js

@@ -94,7 +94,10 @@ export const AppRoot = ({
           {onboardingComponent ? onboardingComponent : null}
           {showDismissHelptipsDialog && <ConfirmDismissHelptips />}
           {showDeviceConnectionDialog && <DeviceConnectionDialog onCancel={() => setShowConnectingDialog(false)} />}
-          <LiveChatBox />
+          {
+            // eslint-disable-next-line no-undef
+            ENV === 'production' && <LiveChatBox />
+          }
         </>
       ) : (
         publicRoutes

tests/setupTests.js

@@ -55,6 +55,7 @@ beforeAll(async () => {
     setItem: jest.fn(),
     removeItem: jest.fn()
   };
+  window.ENV = 'test';
   server = setupServer(...handlers);
   await server.listen();
   Object.defineProperty(navigator, 'appVersion', { value: 'Test', writable: true });

webpack.config.js

@@ -97,6 +97,7 @@ module.exports = (env, argv) => ({
       Buffer: ['buffer', 'Buffer']
     }),
     new webpack.DefinePlugin({
+      ENV: JSON.stringify(argv.mode),
       XTERM_VERSION: JSON.stringify(require('./package.json').dependencies.xterm),
       XTERM_FIT_VERSION: JSON.stringify(require('./package.json').dependencies['xterm-addon-fit']),
       XTERM_SEARCH_VERSION: JSON.stringify(require('./package.json').dependencies['xterm-addon-search'])