Merge pull request #20 from aolsenjazz/dev

fix(renderer state): Only reset loopback messages when setProjectName…

Author: aolsenjazz

src/renderer/features/recent-loopback-messages/recent-loopback-messages-slice.ts

@@ -37,15 +37,13 @@ export const recentLoopbackMessagesSlice = createAppSlice({
         if (state[qid].length > 100) state[qid].shift();
       },
     ),
-  }),
-  extraReducers: (builder) => {
-    // on project change, clear recent input events
-    builder.addCase('projectName/setName', (state) => {
+
+    clearAllMessages: create.reducer((state) => {
       Array.from(Object.keys(state)).forEach((k) => {
         delete state[k];
       });
-    });
-  },
+    }),
+  }),
 });
 
 export const selectRecentLoopbackMessages = (state: RootState, id: string) =>

src/renderer/middleware/project-name-listener.ts

@@ -0,0 +1,25 @@
+import { startAppListening } from '../store/listener-middleware';
+import { projectNameSlice } from '../features/project-name/project-name-slice';
+import { recentLoopbackMessagesSlice } from '../features/recent-loopback-messages/recent-loopback-messages-slice';
+
+/**
+ * Listens to changes to the current superset dashboard layout. This callback is invoked whenever
+ * a gridComponent is added or removed from the layout.
+ *
+ * It's currently updating in a not-smart way; whenever an update happens, every IkiDynamicMarkdown
+ * gridComponent notifies the ContextService to alert us that the layout has changed. Therefore,
+ * we end up receiving far too many changed events
+ */
+startAppListening({
+  actionCreator: projectNameSlice.actions.setName,
+  effect: async (action, listenerApi) => {
+    // ignore if the action is dispatched with the same payload
+    if (action.payload === listenerApi.getState().projectName.name) {
+      return;
+    }
+
+    listenerApi.dispatch(
+      recentLoopbackMessagesSlice.actions.clearAllMessages(),
+    );
+  },
+});

src/renderer/store/listener-middleware.ts

@@ -0,0 +1,10 @@
+import { createListenerMiddleware } from '@reduxjs/toolkit';
+
+import type { AppDispatch, RootState } from './store';
+
+export const listenerMiddleware = createListenerMiddleware();
+
+export const startAppListening = listenerMiddleware.startListening.withTypes<
+  RootState,
+  AppDispatch
+>();

src/renderer/store/store.ts

@@ -14,6 +14,8 @@ import { recentLoopbackMessagesSlice } from '../features/recent-loopback-message
 import { recentRemoteMessagesSlice } from '../features/recent-remote-messages/recent-remote-messages-slice';
 import { projectNameSlice } from '../features/project-name/project-name-slice';
 
+import '../middleware/project-name-listener';
+
 const rootReducer = combineSlices(
   connectedDevicesSlice,
   configuredDevicesSlice,