From ff92f575b5c78ed7f76c571585ba024056d20e79 Mon Sep 17 00:00:00 2001 From: Aleksei Poroshin Date: Mon, 21 Aug 2023 10:54:41 +0200 Subject: [PATCH 1/2] =?UTF-8?q?Endret=20n=C3=A6ringstyper=20(Array)=20til?= =?UTF-8?q?=20n=C3=A6ringstype=20(ikke=20array)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../src/forms/virksomhet/VirksomhetInfo.tsx | 13 ++++++------- .../src/forms/virksomhet/VirksomhetSummary.tsx | 13 +++++-------- .../__tests__/mapVirksomhetToApiData.test.ts | 2 +- .../src/forms/virksomhet/mapVirksomhetToApiData.ts | 2 +- .../src/forms/virksomhet/types.ts | 2 +- .../src/forms/virksomhet/virksomhetUtils.ts | 3 +-- .../stories/virksomhet/VirksomhetExample.tsx | 4 ++-- 7 files changed, 17 insertions(+), 22 deletions(-) diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetInfo.tsx b/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetInfo.tsx index 6eaecd7efe..07eac625ba 100644 --- a/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetInfo.tsx +++ b/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetInfo.tsx @@ -9,8 +9,8 @@ import JaNeiSvar from '../../components/summary/JaNeiSvar'; import Sitat from '../../components/summary/Sitat'; import SummaryBlock from '../../components/summary/SummaryBlock'; import TallSvar from '../../components/summary/TallSvar'; -import { VirksomhetApiData } from './types'; -import { erVirksomhetRegnetSomNyoppstartet, harFiskerNæringstype } from './virksomhetUtils'; +import { Næringstype, VirksomhetApiData } from './types'; +import { erVirksomhetRegnetSomNyoppstartet } from './virksomhetUtils'; import ExpandableInfo from '@navikt/sif-common-core-ds/lib/components/expandable-info/ExpandableInfo'; import Block from '@navikt/sif-common-core-ds/lib/atoms/block/Block'; @@ -20,11 +20,10 @@ interface Props { const renderVirksomhetSummary = (virksomhet: VirksomhetApiData, intl: IntlShape) => { const land = virksomhet.registrertIUtlandet ? virksomhet.registrertIUtlandet.landnavn : 'Norge'; - const næringstyper = virksomhet.næringstyper - .map((næring) => intlHelper(intl, `sifForms.virksomhet.næringstype_${næring}`)) - .join(', '); + const næringstype = intlHelper(intl, `sifForms.virksomhet.næringstype_${virksomhet.næringstype}`); + const fiskerinfo = - harFiskerNæringstype(virksomhet.næringstyper) && virksomhet.fiskerErPåBladB !== undefined + virksomhet.næringstype === Næringstype.FISKE && virksomhet.fiskerErPåBladB !== undefined ? { erPåBladB: virksomhet.fiskerErPåBladB !== undefined && virksomhet.fiskerErPåBladB === true, } @@ -41,7 +40,7 @@ const renderVirksomhetSummary = (virksomhet: VirksomhetApiData, intl: IntlShape) return ( - {næringstyper}. + {næringstype}. {fiskerinfo && (
{fiskerinfo.erPåBladB === false ? ( diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetSummary.tsx b/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetSummary.tsx index fbaae30636..0fc5df1345 100644 --- a/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetSummary.tsx +++ b/packages/sif-common-forms-ds/src/forms/virksomhet/VirksomhetSummary.tsx @@ -28,13 +28,10 @@ const getFiskerNæringTekst = (intl: IntlShape, erPåBladB: boolean) => { export const renderVirksomhetSummary = (virksomhet: VirksomhetApiData, intl: IntlShape) => { const land = virksomhet.registrertIUtlandet ? virksomhet.registrertIUtlandet.landnavn : 'Norge'; - const næringstyper = virksomhet.næringstyper - .map((næring) => - næring === Næringstype.FISKE && virksomhet.fiskerErPåBladB !== undefined - ? getFiskerNæringTekst(intl, virksomhet.fiskerErPåBladB) - : intlHelper(intl, `sifForms.virksomhet.næringstype_${næring}`) - ) - .join(', '); + const næringstype = + virksomhet.næringstype === Næringstype.FISKE && virksomhet.fiskerErPåBladB !== undefined + ? getFiskerNæringTekst(intl, virksomhet.fiskerErPåBladB) + : intlHelper(intl, `sifForms.virksomhet.næringstype_${virksomhet.næringstype}`); const tidsinfo = virksomhet.tilOgMed ? intlHelper(intl, 'sifForms.virksomhet.summary.tidsinfo.avsluttet', { @@ -50,7 +47,7 @@ export const renderVirksomhetSummary = (virksomhet: VirksomhetApiData, intl: Int {virksomhet.navnPåVirksomheten}. - {næringstyper}. + {næringstype}.
{virksomhet.registrertINorge && ( diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/__tests__/mapVirksomhetToApiData.test.ts b/packages/sif-common-forms-ds/src/forms/virksomhet/__tests__/mapVirksomhetToApiData.test.ts index 1fc6a77abc..298db7edcb 100644 --- a/packages/sif-common-forms-ds/src/forms/virksomhet/__tests__/mapVirksomhetToApiData.test.ts +++ b/packages/sif-common-forms-ds/src/forms/virksomhet/__tests__/mapVirksomhetToApiData.test.ts @@ -26,7 +26,7 @@ const virksomhetFormData: Virksomhet = { }; const virksomhetApiData: VirksomhetApiData = { - næringstyper: [Næringstype.ANNEN], + næringstype: Næringstype.ANNEN, navnPåVirksomheten: 'ABC', fraOgMed: dateToISODate(fom), tilOgMed: null, diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/mapVirksomhetToApiData.ts b/packages/sif-common-forms-ds/src/forms/virksomhet/mapVirksomhetToApiData.ts index f164f203b2..390e3c10b8 100644 --- a/packages/sif-common-forms-ds/src/forms/virksomhet/mapVirksomhetToApiData.ts +++ b/packages/sif-common-forms-ds/src/forms/virksomhet/mapVirksomhetToApiData.ts @@ -14,7 +14,7 @@ export const mapVirksomhetToVirksomhetApiData = ( const erNyoppstartet = erVirksomhetRegnetSomNyoppstartet(virksomhet.fom); const data: VirksomhetApiData = { - næringstyper: [virksomhet.næringstype], + næringstype: virksomhet.næringstype, navnPåVirksomheten: virksomhet.navnPåVirksomheten, registrertINorge, ...(registrertINorge diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/types.ts b/packages/sif-common-forms-ds/src/forms/virksomhet/types.ts index 7aa6f9c643..62afb5f866 100644 --- a/packages/sif-common-forms-ds/src/forms/virksomhet/types.ts +++ b/packages/sif-common-forms-ds/src/forms/virksomhet/types.ts @@ -78,7 +78,7 @@ export const isVirksomhet = (virksomhet: Partial): virksomhet is Vir }; export interface VirksomhetApiData { - næringstyper: Næringstype[]; + næringstype: Næringstype; fiskerErPåBladB?: boolean; fraOgMed: ISODate; tilOgMed?: ISODate | null; diff --git a/packages/sif-common-forms-ds/src/forms/virksomhet/virksomhetUtils.ts b/packages/sif-common-forms-ds/src/forms/virksomhet/virksomhetUtils.ts index 72fc177d4a..63cd2e4fe2 100644 --- a/packages/sif-common-forms-ds/src/forms/virksomhet/virksomhetUtils.ts +++ b/packages/sif-common-forms-ds/src/forms/virksomhet/virksomhetUtils.ts @@ -6,8 +6,7 @@ import dayjs from 'dayjs'; import { guid } from '@navikt/sif-common-utils'; import { Næringstype, Virksomhet, VirksomhetFormValues } from './types'; -export const harFiskerNæringstype = (næringstyper: Næringstype[]): boolean => - næringstyper.find((n) => n === Næringstype.FISKE) !== undefined; +export const harFiskerNæringstype = (næringstype: Næringstype): boolean => næringstype === Næringstype.FISKE; export const erFiskerNæringstype = (næringstype?: Næringstype): boolean => næringstype ? næringstype === Næringstype.FISKE : false; diff --git a/packages/sif-common-forms-ds/storybook/stories/virksomhet/VirksomhetExample.tsx b/packages/sif-common-forms-ds/storybook/stories/virksomhet/VirksomhetExample.tsx index 3984b9dac0..296c803fb9 100644 --- a/packages/sif-common-forms-ds/storybook/stories/virksomhet/VirksomhetExample.tsx +++ b/packages/sif-common-forms-ds/storybook/stories/virksomhet/VirksomhetExample.tsx @@ -4,7 +4,7 @@ import { useState } from 'react'; import { useIntl } from 'react-intl'; import Block from '@navikt/sif-common-core-ds/lib/atoms/block/Block'; import { TypedFormikForm, TypedFormikWrapper, YesOrNo } from '@navikt/sif-common-formik-ds/lib'; -import { getListValidator } from '@navikt/sif-common-formik-ds/lib/validation'; +import { getRequiredFieldValidator } from '@navikt/sif-common-formik-ds/lib/validation'; import getFormErrorHandler from '@navikt/sif-common-formik-ds/lib/validation/intlFormErrorHandler'; import { ValidationError } from '@navikt/sif-common-formik-ds/lib/validation/types'; import flat from 'flat'; @@ -77,7 +77,7 @@ const VirksomhetExample = () => { name={FormField.virksomhet} harFlereVirksomheter={harFlereVirksomheter} - validate={getListValidator({ required: true })} + validate={getRequiredFieldValidator()} labels={{ addLabel: harFlereVirksomheter ? 'Registrer virksomhet' : 'Legg til', deleteLabel: 'Fjern', From 68bf31310bf3b55b5d43e56df4668cec525be063 Mon Sep 17 00:00:00 2001 From: Aleksei Poroshin Date: Mon, 21 Aug 2023 11:32:17 +0200 Subject: [PATCH 2/2] changeset --- .changeset/fifty-rabbits-juggle.md | 5 +++++ 1 file changed, 5 insertions(+) create mode 100644 .changeset/fifty-rabbits-juggle.md diff --git a/.changeset/fifty-rabbits-juggle.md b/.changeset/fifty-rabbits-juggle.md new file mode 100644 index 0000000000..5e978f2bc8 --- /dev/null +++ b/.changeset/fifty-rabbits-juggle.md @@ -0,0 +1,5 @@ +--- +'@navikt/sif-common-forms-ds': minor +--- + +Endret næringstyper (Array) til næringstype (ikke array) i Virksomhet