Memoize props passed to useAppLogic

Fixes a strange bug where the enhet would reset if some combination of
props was passed, due to how the props are passed into the decorator and
the spreading mutating the object, the logic somehow causes part of the
state to reset. Wrapping the props in useMemo seems to fix this.

Author: LudvigHz

packages/internarbeidsflate-decorator-v3/src/Decorator.tsx

@@ -1,4 +1,4 @@
-import React, { PropsWithChildren } from 'react';
+import React, { PropsWithChildren, useMemo } from 'react';
 import './index.css';
 import useAppLogic from './hooks/useAppLogic';
 import Banner from './components/Banner';
@@ -11,16 +11,20 @@ import { useAppState } from './states/AppState';
 import { DecoratorProps } from './types/AppProps';
 
 const Decorator: React.FC<PropsWithChildren<DecoratorProps>> = (props) => {
-  useAppLogic({
-    ...props,
-    ignoreExternalFnr: props.fnrSyncMode === 'writeOnly',
-    fetchActiveUserOnMount:
-      props.fnrSyncMode !== 'writeOnly' && props.fetchActiveUserOnMount,
+  const memoizedProps = useMemo(
+    () => ({
+      ...props,
+      ignoreExternalFnr: props.fnrSyncMode === 'writeOnly',
+      fetchActiveUserOnMount:
+        props.fnrSyncMode !== 'writeOnly' && props.fetchActiveUserOnMount,
 
-    ignoreExternalEnhet: props.enhetSyncMode === 'writeOnly',
-    fetchActiveEnhetOnMount:
-      props.enhetSyncMode !== 'writeOnly' && props.fetchActiveEnhetOnMount,
-  });
+      ignoreExternalEnhet: props.enhetSyncMode === 'writeOnly',
+      fetchActiveEnhetOnMount:
+        props.enhetSyncMode !== 'writeOnly' && props.fetchActiveEnhetOnMount,
+    }),
+    [props],
+  );
+  useAppLogic(memoizedProps);
 
   const ref = useOnOutsideClick<HTMLElement>(() =>
     useAppState.setState({ open: false }),