Fix bug 1583527 - Show redux logs only in development. (#1390)

mozilla · Sep 24, 2019 · 1a127d8 · 1a127d8
1 parent 4dcc96d
commit 1a127d8
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 6 deletions.
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -48,6 +48,7 @@ module.exports = {
         DIFF_EQUAL: false,
         DIFF_DELETE: false,
         ga: false,
+        process: false,
         generalShortcutsHandler: true,
         traversalShortcutsHandler: true,
         editorShortcutsHandler: true,

diff --git a/frontend/src/store.js b/frontend/src/store.js
@@ -7,16 +7,21 @@ import history from './history';
 import rootReducer from './rootReducer';
 
 
-const loggerMiddleware = createLogger();
+const middlewares = [
+    routerMiddleware(history),
+    thunkMiddleware,
+];
+
+// Log actions only in development.
+if (process.env.NODE_ENV === 'development') {
+    const loggerMiddleware = createLogger();
+    middlewares.push(loggerMiddleware);
+}
 
 export default createStore(
     connectRouter(history)(rootReducer),
     {}, // initial state
     compose(
-        applyMiddleware(
-            routerMiddleware(history),
-            thunkMiddleware,
-            loggerMiddleware,
-        )
+        applyMiddleware(...middlewares)
     )
 );