Skip to content

Commit 47fa732

Browse files
navelgentrinelinderudhavstein
committedMar 10, 2025
🐛 Revert bruk av provider for person
Det førte til brudd i kommunikasjon mellom komponenter som brukte samme atom på innsiden og utsiden av provider. Gjaldt f. eks. toast og opptegnelse-pollingen. Co-authored-by: Trine Linderud <trine.linderud@nav.no> Co-authored-by: Jakob Havstein Eriksen <jakob.havstein.eriksen@nav.no>
1 parent 91991b2 commit 47fa732

File tree

3 files changed

+22
-19
lines changed

3 files changed

+22
-19
lines changed
 

‎src/app/person/[aktorId]/layout.tsx

+13-19
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
'use client';
22

3-
import { Provider } from 'jotai';
43
import dynamic from 'next/dynamic';
5-
import { useParams } from 'next/navigation';
64
import React, { PropsWithChildren, ReactElement } from 'react';
75

86
import { useKeyboardShortcuts } from '@hooks/useKeyboardShortcuts';
@@ -29,8 +27,6 @@ const Historikk = dynamic(() => import('@saksbilde/historikk').then((mod) => mod
2927
});
3028

3129
export default function Layout({ children }: PropsWithChildren): ReactElement {
32-
const { aktorId } = useParams<{ aktorId?: string }>();
33-
3430
useRefreshPersonVedOpptegnelse();
3531
usePollEtterOpptegnelser();
3632
useVarselOmSakErTildeltAnnenSaksbehandler();
@@ -39,20 +35,18 @@ export default function Layout({ children }: PropsWithChildren): ReactElement {
3935
useRefetchDriftsmeldinger();
4036

4137
return (
42-
<Provider key={aktorId}>
43-
<div className={styles.Saksbilde}>
44-
<InfovarselOmStans />
45-
<PersonHeader />
46-
<Timeline />
47-
<AmplitudeProvider>
48-
<VenterPåEndringProvider>
49-
<Venstremeny />
50-
<Saksbilde>{children}</Saksbilde>
51-
<Historikk />
52-
<EmojiTilbakemeldingMedPeriode />
53-
</VenterPåEndringProvider>
54-
</AmplitudeProvider>
55-
</div>
56-
</Provider>
38+
<div className={styles.Saksbilde}>
39+
<InfovarselOmStans />
40+
<PersonHeader />
41+
<Timeline />
42+
<AmplitudeProvider>
43+
<VenterPåEndringProvider>
44+
<Venstremeny />
45+
<Saksbilde>{children}</Saksbilde>
46+
<Historikk />
47+
<EmojiTilbakemeldingMedPeriode />
48+
</VenterPåEndringProvider>
49+
</AmplitudeProvider>
50+
</div>
5751
);
5852
}

‎src/routes/oversikt/Oversikt.tsx

+2
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import { BehandlingsstatistikkView } from '@oversikt/behandlingsstatistikk/Behan
1515
import { FiltermenySkeleton } from '@oversikt/filtermeny/Filtermeny';
1616
import { BehandletIdagTable } from '@oversikt/table/BehandletIdagTable';
1717
import { OppgaverTable } from '@oversikt/table/oppgaverTable/OppgaverTable';
18+
import { useResetSkjønnsfastsettelseFormState } from '@saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/skjønnsfastsettingForm/SkjønnsfastsettingForm';
1819
import { useOppgaveFeed } from '@state/oppgaver';
1920

2021
import { IngenOppgaver } from './IngenOppgaver';
@@ -48,6 +49,7 @@ export const Oversikt = (): ReactElement => {
4849
useKeyboardShortcuts();
4950
useFjernPersonFraApolloCache();
5051
useRefetchDriftsmeldinger();
52+
useResetSkjønnsfastsettelseFormState();
5153

5254
return (
5355
<main className={styles.Oversikt}>

‎src/routes/saksbilde/sykepengegrunnlag/skjønnsfastsetting/form/skjønnsfastsettingForm/SkjønnsfastsettingForm.tsx

+7
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,13 @@ import styles from './SkjønnsfastsettingForm.module.css';
3535

3636
const skjemaFamily = atomFamily((_skjæringstidspunkt: string) => atom<Maybe<SkjønnsfastsettingFormFields>>(null));
3737

38+
export const useResetSkjønnsfastsettelseFormState = () => {
39+
useEffect(() => {
40+
skjemaFamily.setShouldRemove(() => true);
41+
skjemaFamily.setShouldRemove(null);
42+
}, []);
43+
};
44+
3845
export const useAktiveArbeidsgivere = (
3946
person: PersonFragment,
4047
period: BeregnetPeriodeFragment | GhostPeriodeFragment,

0 commit comments

Comments
 (0)
Please sign in to comment.