Legg til 'ignore' til syncmode

Skal ignorere context begge veier for den respektive verdien. Mest
brukbar or apper som bruker dekoratøren men som ikke bryr seg om aktiv
context.

Author: LudvigHz

README.md

@@ -70,8 +70,8 @@ export interface DecoratorProps {
   enableHotkeys?: boolean | undefined; // Aktivere hurtigtaster
   fetchActiveEnhetOnMount?: boolean | undefined; // Om enhet er undefined fra container appen, og denne er satt til true, henter den sist aktiv enhet og bruker denne.
   fetchActiveUserOnMount?: boolean | undefined; // Om fnr er undefined fra container appen, og denne er satt til true for at den skal hente siste aktiv fnr.
-  fnrSyncMode?: 'sync' | 'writeOnly'; // Modus til fnr state management. "sync" er default. "writeOnly" gjør at endringer aldri hentes men vil settes dersom det oppdateres lokalt i appen
-  enhetSyncMode?: 'sync' | 'writeOnly'; // Samme som fnrSyncMode, men for enhet state.
+  fnrSyncMode?: 'sync' | 'writeOnly' | 'ignore'; // Modus til fnr state management. "sync" er default. "writeOnly" gjør at endringer aldri hentes men vil settes dersom det oppdateres lokalt i appen. "ignore" verken henter fra context eller skriver til context ved oppdatert state lokalt.
+  enhetSyncMode?: 'sync' | 'writeOnly' | 'ignore'; // Samme som fnrSyncMode, men for enhet state.
   onEnhetChanged: (enhetId?: string | null, enhet?: Enhet) => void; // Kalles når enheten endres
   onFnrChanged: (fnr?: string | null) => void; // Kalles når fnr enheten endres
   onLinkClick?: (link: { text: string; url: string }) => void; // Kan brukes for å legge til callbacks ved klikk på lenker i menyen. Merk at callbacken ikke kan awaites og man må selv håndtere at siden lukkes. Nyttig for å f.eks tracke navigasjon events i amplitude

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

@@ -8,20 +8,31 @@ import NewEnhetModal from './components/modals/NewEnhetModal';
 import ErrorMessage from './components/ErrorMessageDisplay';
 import { useOnOutsideClick } from './hooks/useOnOutsideClick';
 import { useAppState } from './states/AppState';
-import { DecoratorProps } from './types/AppProps';
+import { AppProps, DecoratorProps } from './types/AppProps';
 
 const Decorator: React.FC<PropsWithChildren<DecoratorProps>> = (props) => {
   const memoizedProps = useMemo(
-    () => ({
-      ...props,
-      ignoreExternalFnr: props.fnrSyncMode === 'writeOnly',
-      fetchActiveUserOnMount:
-        props.fnrSyncMode !== 'writeOnly' && props.fetchActiveUserOnMount,
+    () =>
+      ({
+        ...props,
+        ignoreExternalFnr:
+          props.fnrSyncMode === 'writeOnly' || props.fnrSyncMode === 'ignore',
+        fetchActiveUserOnMount:
+          props.fnrSyncMode !== 'writeOnly' &&
+          props.fnrSyncMode !== 'ignore' &&
+          props.fetchActiveUserOnMount,
 
-      ignoreExternalEnhet: props.enhetSyncMode === 'writeOnly',
-      fetchActiveEnhetOnMount:
-        props.enhetSyncMode !== 'writeOnly' && props.fetchActiveEnhetOnMount,
-    }),
+        ignoreExternalEnhet:
+          props.enhetSyncMode === 'writeOnly' ||
+          props.enhetSyncMode === 'ignore',
+        fetchActiveEnhetOnMount:
+          props.enhetSyncMode !== 'writeOnly' &&
+          props.enhetSyncMode !== 'ignore' &&
+          props.fetchActiveEnhetOnMount,
+
+        fnrWriteDisabled: props.fnrSyncMode === 'ignore',
+        enhetWriteDisabled: props.enhetSyncMode === 'ignore',
+      }) satisfies AppProps,
     [props],
   );
   useAppLogic(memoizedProps);

packages/internarbeidsflate-decorator-v3/src/__mocks__/mock-handlers.ts

@@ -37,6 +37,8 @@ const getSuccessResponse = (
 type Context = { aktivEnhet: string | null; aktivBruker: string | null };
 let context: Context = { aktivEnhet: '0118', aktivBruker: '10108000398' };
 
+export const getMockContext = () => context;
+
 export const updateMockContext = (newContext: Partial<Context>) => {
   context = {
     ...context,

packages/internarbeidsflate-decorator-v3/src/store/EnhetValueManager.ts

@@ -15,6 +15,8 @@ export class EnhetValueManager extends ContextValueManager {
   #fnrValueManager: FnrValueManager;
   #propsUpdateHandler: PropsUpdateHandler;
   #onEnhetChanged?: (newEnhet?: string | null, enhetObject?: Enhet) => void;
+
+  #writeDisabled?: boolean;
   constructor(
     substateProps: SubstateHandlerProps,
     errorMessageManager: ErrorMessageManager,
@@ -31,9 +33,11 @@ export class EnhetValueManager extends ContextValueManager {
     enhet,
     onEnhetChanged,
     veileder,
+    enhetWriteDisabled,
   }: StoreProps) => {
     this.#registerPropsHandler();
     this.#onEnhetChanged = onEnhetChanged;
+    this.#writeDisabled = !!enhetWriteDisabled;
     if (enhet && enhet === this.state.enhet.value) {
       return;
     }
@@ -156,6 +160,11 @@ export class EnhetValueManager extends ContextValueManager {
     newEnhetId?: string,
   ) => {
     this.#changeEnhetLocally(enheter, newEnhetId);
+
+    if (this.#writeDisabled) {
+      return;
+    }
+
     return this.contextHolderApi.changeEnhet(newEnhetId);
   };
 

packages/internarbeidsflate-decorator-v3/src/store/FnrValueManager.ts

@@ -14,6 +14,8 @@ export class FnrValueManager extends ContextValueManager {
   #propsUpdateHandler: PropsUpdateHandler;
   #onFnrUpdated?: (fnr?: string | null) => void;
 
+  #writeDisabled?: boolean;
+
   constructor(
     substateProps: SubstateHandlerProps,
     errorMessageMaanger: ErrorMessageManager,
@@ -30,8 +32,10 @@ export class FnrValueManager extends ContextValueManager {
     fnr,
     userKey,
     onFnrChanged,
+    fnrWriteDisabled,
   }: StoreProps) => {
     this.#onFnrUpdated = onFnrChanged;
+    this.#writeDisabled = !!fnrWriteDisabled;
     if (!veileder.ident) {
       this.changeFnrLocally();
       return;
@@ -90,6 +94,10 @@ export class FnrValueManager extends ContextValueManager {
   readonly changeFnrLocallyAndExternally = async (newFnr?: string) => {
     this.optimisticUpdate('fnr');
     this.changeFnrLocally(newFnr);
+    if (this.#writeDisabled) {
+      return;
+    }
+
     const res = await this.contextHolderApi.changeFnr(newFnr);
     if (res.error) {
       this.#errorMessageManager.addErrorMessage(

packages/internarbeidsflate-decorator-v3/src/store/__test__/StoreHandler.test.ts

@@ -6,6 +6,7 @@ import { StoreHandler } from '../StoreHandler';
 import { AppProps } from '../../types/AppProps';
 import {
   getHandlers,
+  getMockContext,
   mockMe,
   updateMockContext,
 } from '../../__mocks__/mock-handlers';
@@ -225,10 +226,58 @@ describe('StoreHandler test', () => {
     await awaitTimeout(10, 'For å la staten bli propagert');
 
     expect(spy).toHaveBeenCalledTimes(2);
-    expect(Object.values(storeHandler.state.errorMessages).length).toBe(0);
     expect(storeHandler.state.enhet.value).toBe('0219');
   });
 
+  it('skal ikke sende ny enhet til contxt-apiet hvis den får ny enhet og syncMode = ignore', async () => {
+    const storeHandler = new StoreHandler();
+    const spy = vi.spyOn(
+      storeHandler.enhetValueManager,
+      'changeEnhetLocallyAndExternally',
+    );
+    storeHandler.propsUpdateHandler.onPropsUpdated({
+      enhet: '0118',
+      enhetWriteDisabled: true,
+      ...defaultProps,
+    });
+
+    await ws.connected;
+
+    expect(spy).toHaveBeenCalledOnce();
+
+    storeHandler.propsUpdateHandler.onPropsUpdated({
+      enhet: '0219',
+      enhetWriteDisabled: true,
+      ...defaultProps,
+    });
+
+    await awaitTimeout(10, 'For å la staten bli propagert');
+
+    expect(spy).toHaveBeenCalledTimes(2);
+    expect(storeHandler.state.enhet.value).toBe('0219');
+    expect(getMockContext().aktivEnhet).toEqual('0118');
+  });
+
+  it('skal ikke sende ny fnr til contxt-apiet hvis den får ny fnr og syncMode = ignore', async () => {
+    const storeHandler = new StoreHandler();
+    const spy = vi.spyOn(
+      storeHandler.fnrValueManager,
+      'changeFnrLocallyAndExternally',
+    );
+
+    await storeHandler.propsUpdateHandler.onPropsUpdated({
+      fnr: '07063000250',
+      enhetWriteDisabled: true,
+      ...defaultProps,
+    });
+
+    await awaitTimeout(10, 'For å la staten bli propagert');
+
+    expect(spy).toHaveBeenCalledOnce();
+    expect(storeHandler.state.fnr.value).toBe('07063000250');
+    expect(getMockContext().aktivBruker).toEqual('10108000398');
+  });
+
   it('skal hente fnr fra context apiet om `fetchActiveUserOnMount` er satt til true', async () => {
     const storeHandler = new StoreHandler();
 

packages/internarbeidsflate-decorator-v3/src/types/AppProps.ts

@@ -18,12 +18,20 @@ interface DomainProps {
 
   ignoreExternalFnr?: boolean;
   ignoreExternalEnhet?: boolean;
+  enhetWriteDisabled?: boolean;
+  fnrWriteDisabled?: boolean;
 }
 
 export interface AppProps extends DomainProps, Omit<AppState, 'open'> {}
 
 export interface DecoratorProps
-  extends Omit<AppProps, 'ignoreExternalFnr' | 'ignoreExternalEnhet'> {
-  fnrSyncMode?: 'writeOnly' | 'sync';
-  enhetSyncMode?: 'writeOnly' | 'sync';
+  extends Omit<
+    AppProps,
+    | 'ignoreExternalFnr'
+    | 'ignoreExternalEnhet'
+    | 'enhetWriteDisabled'
+    | 'fnrWriteDisabled'
+  > {
+  fnrSyncMode?: 'writeOnly' | 'sync' | 'ignore';
+  enhetSyncMode?: 'writeOnly' | 'sync' | 'ignore';
 }