Skip to content

Commit d1042b9

Browse files
stigusstigus
committed
- Fikset Steg2 som viste form komponenter selv om attributt ikke var huket av og førte til masse unødvendige get kall
- Fikset skjerming visning som ikke kunne hukes vekk igjen dersom den ble satt #deploy-test-frontend
1 parent 09c9afe commit d1042b9

File tree

2 files changed

+51
-51
lines changed

2 files changed

+51
-51
lines changed

‎apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/stegVelger/steg/steg1/paneler/Personinformasjon.tsx

+1-1
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,7 @@ PersoninformasjonPanel.initialValues = ({ set, opts, setMulti, del, has }) => {
328328
)
329329
},
330330
remove() {
331-
del('skjerming')
331+
del(['skjerming', paths.egenAnsattDatoFom.tpsM, paths.egenAnsattDatoFom.skjerming])
332332
},
333333
},
334334
norskBankkonto: {

‎apps/dolly-frontend/src/main/js/src/components/bestillingsveileder/stegVelger/steg/steg2/Steg2.tsx

+50-50
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import React, { lazy, Suspense, useContext, useEffect } from 'react'
2-
import { harAvhukedeAttributter } from '@/components/bestillingsveileder/utils'
3-
import { BestillingsveilederContext } from '@/components/bestillingsveileder/BestillingsveilederContext'
2+
import { useFormContext } from 'react-hook-form'
43
import { SigrunstubForm } from '@/components/fagsystem/sigrunstub/form/Form'
54
import { InntektstubForm } from '@/components/fagsystem/inntektstub/form/Form'
65
import { InntektsmeldingForm } from '@/components/fagsystem/inntektsmelding/form/Form'
@@ -14,13 +13,11 @@ import { MedlForm } from '@/components/fagsystem/medl/form/MedlForm'
1413
import { SykdomForm } from '@/components/fagsystem/sykdom/form/Form'
1514
import { OrganisasjonForm } from '@/components/fagsystem/organisasjoner/form/Form'
1615
import { TjenestepensjonForm } from '@/components/fagsystem/tjenestepensjon/form/Form'
17-
import { Alert } from '@navikt/ds-react'
1816
import { AlderspensjonForm } from '@/components/fagsystem/alderspensjon/form/Form'
1917
import { ArbeidsplassenForm } from '@/components/fagsystem/arbeidsplassen/form/Form'
2018
import { UforetrygdForm } from '@/components/fagsystem/uforetrygd/form/Form'
2119
import { SigrunstubPensjonsgivendeForm } from '@/components/fagsystem/sigrunstubPensjonsgivende/form/Form'
2220
import { KrrstubForm } from '@/components/fagsystem/krrstub/form/KrrForm'
23-
import { useFormContext } from 'react-hook-form'
2421
import { SkattekortForm } from '@/components/fagsystem/skattekort/form/Form'
2522
import { PensjonsavtaleForm } from '@/components/fagsystem/pensjonsavtale/form/Form'
2623
import { FullmaktForm } from '@/components/fagsystem/fullmakt/form/FullmaktForm'
@@ -29,76 +26,79 @@ import { YrkesskaderForm } from '@/components/fagsystem/yrkesskader/form/Form'
2926
import Loading from '@/components/ui/loading/Loading'
3027
import { PdlfForm } from '@/components/fagsystem/pdlf/form/Form'
3128
import { ArbeidssoekerregisteretForm } from '@/components/fagsystem/arbeidssoekerregisteret/form/Form'
29+
import { BestillingsveilederContext } from '@/components/bestillingsveileder/BestillingsveilederContext'
30+
import { harAvhukedeAttributter } from '@/components/bestillingsveileder/utils'
31+
import { Alert } from '@navikt/ds-react'
3232

3333
const HistarkForm = lazy(() => import('@/components/fagsystem/histark/form/HistarkForm'))
3434
const DokarkivForm = lazy(() => import('@/components/fagsystem/dokarkiv/form/DokarkivForm'))
3535

36-
const gruppeNavn = (gruppe) => <span style={{ fontWeight: 'bold' }}>{gruppe.navn}</span>
36+
const Steg2: React.FC = () => {
37+
const { getValues, setValue } = useFormContext()
38+
const opts: any = useContext(BestillingsveilederContext)
3739

38-
const getEmptyMessage = (leggTil, importTestnorge, gruppe = null) => {
39-
if (leggTil) {
40-
return 'Du har ikke lagt til flere egenskaper. Vennligst gå tilbake og velg nye egenskaper.'
41-
} else if (importTestnorge) {
42-
return (
43-
<span>
44-
Du har ikke lagt til egenskaper. Dolly vil importere valgt Test-Norge person(er) til
45-
{gruppe === null && <> gruppe du velger i neste steg.</>}
46-
{gruppe !== null && <> gruppen {gruppeNavn(gruppe)}.</>}
47-
</span>
48-
)
40+
const getEmptyMessage = (leggTil, importTestnorge, gruppe = null) => {
41+
if (leggTil) {
42+
return 'Du har ikke lagt til flere egenskaper. Vennligst gå tilbake og velg nye egenskaper.'
43+
} else if (importTestnorge) {
44+
return (
45+
<span>
46+
Du har ikke lagt til egenskaper. Dolly vil importere valgt Test-Norge person(er) til
47+
{gruppe === null && <> gruppe du velger i neste steg.</>}
48+
{gruppe !== null && <> gruppen {gruppeNavn(gruppe)}.</>}
49+
</span>
50+
)
51+
}
52+
return 'Du har ikke valgt noen egenskaper. Dolly oppretter personer med tilfeldige verdier.'
4953
}
50-
return 'Du har ikke valgt noen egenskaper. Dolly oppretter personer med tilfeldige verdier.'
51-
}
52-
53-
const Steg2 = () => {
54-
const opts: any = useContext(BestillingsveilederContext)
55-
const formMethods = useFormContext()
5654

5755
useEffect(() => {
5856
if (opts.gruppe?.id) {
59-
formMethods.setValue('gruppeId', opts.gruppe?.id)
57+
setValue('gruppeId', opts.gruppe?.id)
6058
}
6159
}, [])
6260

6361
const leggTil = opts.is.leggTil
6462
const importTestnorge = opts.is.importTestnorge
6563
const gruppe = opts.gruppe
6664

67-
if (!harAvhukedeAttributter(formMethods.getValues())) {
65+
if (!harAvhukedeAttributter(getValues())) {
6866
return <Alert variant={'info'}>{getEmptyMessage(leggTil, importTestnorge, gruppe)}</Alert>
6967
}
68+
const gruppeNavn = (gruppe: any) => <span style={{ fontWeight: 'bold' }}>{gruppe.navn}</span>
7069

7170
return (
7271
<div>
7372
<PdlfForm />
74-
<FullmaktForm />
75-
<AaregForm />
76-
<SigrunstubForm />
77-
<SigrunstubPensjonsgivendeForm />
78-
<InntektstubForm />
79-
<InntektsmeldingForm />
80-
<SkattekortForm />
81-
<ArbeidssoekerregisteretForm />
82-
<ArbeidsplassenForm />
83-
<PensjonForm />
84-
<PensjonsavtaleForm />
85-
<TjenestepensjonForm />
86-
<AlderspensjonForm />
87-
<UforetrygdForm />
88-
<AfpOffentligForm />
89-
<ArenaForm />
90-
<SykdomForm />
91-
<YrkesskaderForm />
92-
<BrregstubForm />
93-
<InstForm />
94-
<KrrstubForm />
95-
<MedlForm />
96-
<UdistubForm />
73+
{getValues('fullmakt') && <FullmaktForm />}
74+
{getValues('aareg') && <AaregForm />}
75+
{getValues('sigrunstub') && <SigrunstubForm />}
76+
{getValues('sigrunstubPensjonsgivende') && <SigrunstubPensjonsgivendeForm />}
77+
{getValues('inntektstub') && <InntektstubForm />}
78+
{getValues('inntektsmelding') && <InntektsmeldingForm />}
79+
{getValues('skattekort') && <SkattekortForm />}
80+
{getValues('arbeidssoekerregisteret') && <ArbeidssoekerregisteretForm />}
81+
{getValues('arbeidsplassenCV') && <ArbeidsplassenForm />}
82+
{getValues('pensjonforvalter') && <PensjonForm />}
83+
{getValues('pensjonsavtale') && <PensjonsavtaleForm />}
84+
{getValues('tjenestepensjon') && <TjenestepensjonForm />}
85+
{getValues('alderspensjon') && <AlderspensjonForm />}
86+
{getValues('uforetrygd') && <UforetrygdForm />}
87+
{getValues('afpOffentlig') && <AfpOffentligForm />}
88+
{getValues('arenaforvalter') && <ArenaForm />}
89+
{getValues('sykemelding') && <SykdomForm />}
90+
{getValues('yrkesskader') && <YrkesskaderForm />}
91+
{getValues('brregstub') && <BrregstubForm />}
92+
{getValues('instdata') && <InstForm />}
93+
{getValues('krrstub') && <KrrstubForm />}
94+
{getValues('medl') && <MedlForm />}
95+
{getValues('udistub') && <UdistubForm />}
96+
{getValues('organisasjon') && <OrganisasjonForm />}
97+
9798
<Suspense fallback={<Loading label="Laster komponenter..." />}>
98-
<DokarkivForm />
99-
<HistarkForm />
99+
{getValues('dokarkiv') && <DokarkivForm />}
100+
{getValues('histark') && <HistarkForm />}
100101
</Suspense>
101-
<OrganisasjonForm />
102102
</div>
103103
)
104104
}

0 commit comments

Comments
 (0)