ui: Ensure Redux logger is last in middleware chain

This change updates the Redux store initialization to ensure that the
Redux logger is last in the middleware change. If we do not do this,
Redux Logger might log thunds and promises, but not actual actions.

See https://github.com/LogRocket/redux-logger/issues/20

2 files changed, 3 insertions, 14 deletions

diff --git a/opendc-web/opendc-web-ui/src/store/configure-store.js b/opendc-web/opendc-web-ui/src/store/configure-store.js
index d8f343ed..13bcd69e 100644
--- a/opendc-web/opendc-web-ui/src/store/configure-store.js
+++ b/opendc-web/opendc-web-ui/src/store/configure-store.js
@@ -6,24 +6,16 @@ import thunk from 'redux-thunk'
 import { authRedirectMiddleware } from '../auth/index'
 import rootReducer from '../reducers/index'
 import rootSaga from '../sagas/index'
-import { dummyMiddleware } from './middlewares/dummy-middleware'
 import { viewportAdjustmentMiddleware } from './middlewares/viewport-adjustment'
 
 const sagaMiddleware = createSagaMiddleware()
 
-let logger
+const middlewares = [thunk, sagaMiddleware, authRedirectMiddleware, viewportAdjustmentMiddleware]
+
 if (process.env.NODE_ENV !== 'production') {
-    logger = createLogger()
+    middlewares.push(createLogger())
 }
 
-const middlewares = [
-    process.env.NODE_ENV === 'production' ? dummyMiddleware : logger,
-    thunk,
-    sagaMiddleware,
-    authRedirectMiddleware,
-    viewportAdjustmentMiddleware,
-]
-
 export let store = undefined
 
 export default function configureStore() {
diff --git a/opendc-web/opendc-web-ui/src/store/middlewares/dummy-middleware.js b/opendc-web/opendc-web-ui/src/store/middlewares/dummy-middleware.js
deleted file mode 100644
index 5ba35691..00000000
--- a/opendc-web/opendc-web-ui/src/store/middlewares/dummy-middleware.js
+++ /dev/null
@@ -1,3 +0,0 @@
-export const dummyMiddleware = (store) => (next) => (action) => {
-    next(action)
-}