Skip to content

Commit

Permalink
liten oppryding av duplisert kode og bedre responsivitet i prosess-be…
Browse files Browse the repository at this point in the history
…regningsgrunnlag (#3330)

* liten oppryding av duplisert kode og bedre responsivitet i prosess-beregningsgrunnlag

* trenger ikke disable når form ikke er dirty

* trenger ikke disable når form ikke er dirty

* Revert "trenger ikke disable når form ikke er dirty"

This reverts commit 7a43fe8.

* Revert "trenger ikke disable når form ikke er dirty"

This reverts commit 488f663.

* går tilbake til submit mode

* renamer intern komponentpakke

* rename mappe

* forenkler isDisabled

* yarn.lock

* fikse ts-error

* flytter interne mapper i egen folder

* tsconfig

* fjerner vite-config for interne folders

* legge til internal i lerna

* forenkle interne pakker
  • Loading branch information
vebnor authored Oct 7, 2024
1 parent 9ef2601 commit bfadcf2
Show file tree
Hide file tree
Showing 64 changed files with 238 additions and 577 deletions.
2 changes: 1 addition & 1 deletion lerna.json
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,6 @@
"npmClientArgs": ["--no-package-lock", "--legacy-peer-deps"]
}
},
"packages": ["packages/*"],
"packages": ["packages/*", "packages/internal/*"],
"$schema": "node_modules/lerna/schemas/lerna-schema.json"
}
3 changes: 2 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,8 @@
},
"workspaces": {
"packages": [
"packages/*"
"packages/*",
"packages/internal/*"
]
},
"lint-staged": {
Expand Down
5 changes: 3 additions & 2 deletions packages/fakta-beregning-redesign/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
"storybook": "storybook dev --quiet -p 7002"
},
"dependencies": {
"@ft-frontend-saksbehandling/internal-components": "workspace:^",
"@navikt/aksel-icons": "7.1.0",
"@navikt/ds-css": "7.1.0",
"@navikt/ds-react": "7.1.0",
Expand Down Expand Up @@ -62,8 +63,8 @@
"react-intl": "6.6.8"
},
"devDependencies": {
"@navikt/ft-storybook-utils": "^2.4.1",
"@navikt/ft-utils-test": "^2.4.1",
"@ft-frontend-saksbehandling/ft-utils-test": "workspace:^",
"@ft-frontend-saksbehandling/storybook-utils": "workspace:^",
"@storybook/addon-actions": "8.3.3",
"@storybook/react": "8.3.3",
"@storybook/react-vite": "8.3.3",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import {
Inntektskategori,
OpptjeningAktivitetType,
} from '@navikt/ft-kodeverk';
import { alleKodeverk as alleKodeverkMock } from '@navikt/ft-storybook-utils';
import { alleKodeverk as alleKodeverkMock } from '@ft-frontend-saksbehandling/storybook-utils';
import {
AndelForFaktaOmBeregning,
BeregningAvklaringsbehov,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import {
BeregningsgrunnlagTilBekreftelse,
Vilkarperiode,
} from '@navikt/ft-types';
import { SubmitButton } from '@ft-frontend-saksbehandling/internal-components';
import { OverstyringKnapp, VerticalSpacer } from '@navikt/ft-ui-komponenter';
import { UseFormGetValues, useFormContext } from 'react-hook-form';
import { FormattedMessage, IntlShape, useIntl } from 'react-intl';
Expand All @@ -19,7 +20,6 @@ import { BeregningAktiviteterTransformedValues } from '../../typer/interface/Ber
import FaktaBeregningAvklaringsbehovCode from '../../typer/interface/FaktaBeregningAvklaringsbehovCode';
import KodeverkForPanel from '../../typer/kodeverkForPanel';
import FaktaBegrunnelseTextField from '../felles/FaktaBegrunnelseTextField';
import SubmitButton from '../felles/SubmitButton';
import { hasAvklaringsbehov, isAvklaringsbehovOpen } from '../felles/avklaringsbehovUtil';
import VurderAktiviteterPanel from './VurderAktiviteterPanel';
import {
Expand Down Expand Up @@ -280,7 +280,7 @@ const AvklareAktiviteterField: FunctionComponent<OwnProps> = ({
isDirty={fieldIsDirty}
isSubmitting={submitDisabled}
isReadOnly={readOnly || (isAvklaringsbehovClosed && !fieldIsDirty)}
hasEmptyRequiredFields={finnesFeilForBegrunnelse}
hasErrors={finnesFeilForBegrunnelse}
/>
{!!dirtyFields && fieldIsDirty && (
<Button
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { getIntlMock } from '@navikt/ft-utils-test';
import { getIntlMock } from '@ft-frontend-saksbehandling/ft-utils-test';
import { compareAndeler, ulikeAndelerErrorMessage, validateUlikeAndeler } from './ValidateAndelerUtils';
import messages from '../../../i18n/nb_NO.json';
import AndelFieldValue from '../../typer/FieldValues';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import {
Beregningsgrunnlag,
Vilkarperiode,
} from '@navikt/ft-types';
import { SubmitButton } from '@ft-frontend-saksbehandling/internal-components';
import { VerticalSpacer } from '@navikt/ft-ui-komponenter';
import React, { FunctionComponent } from 'react';
import { FieldErrors, UseFormGetValues, useFormContext } from 'react-hook-form';
Expand All @@ -13,7 +14,6 @@ import KodeverkForPanel from '../../typer/kodeverkForPanel';
import { formNameVurderFaktaBeregning } from '../BeregningFormUtils';
import { findBegrunnelse } from '../avklareAktiviteter/avklareAktiviteterHjelpefunksjoner';
import FaktaBegrunnelseTextField from '../felles/FaktaBegrunnelseTextField';
import SubmitButton from '../felles/SubmitButton';
import { erOverstyringAvBeregningsgrunnlag, hasAksjonspunkt, isAksjonspunktClosed } from './BgFaktaUtils';
import FaktaForATFLOgSNPanel from './FaktaForATFLOgSNPanel';
import VurderFaktaContext, { BeregningsgrunnlagIndexContext, GetErrorsContext } from './VurderFaktaContext';
Expand Down Expand Up @@ -105,18 +105,19 @@ const VurderFaktaBeregningField: FunctionComponent<OwnProps> = ({
)}
<VerticalSpacer twentyPx />
<SubmitButton
isDisabled={!!verdiForAvklarAktivitetErEndret}
isSubmittable={erSubmittable(
submittable &&
harIkkeEndringerIAvklarMedFlereAksjonspunkter(verdiForAvklarAktivitetErEndret, avklaringsbehov) &&
!isAksjonspunktClosed(avklaringsbehov),
true,
finnesFeilForBegrunnelse(beregningsgrunnlagIndeks, errors),
)}
isSubmittable={
erSubmittable(
submittable &&
harIkkeEndringerIAvklarMedFlereAksjonspunkter(verdiForAvklarAktivitetErEndret, avklaringsbehov) &&
!isAksjonspunktClosed(avklaringsbehov),
true,
finnesFeilForBegrunnelse(beregningsgrunnlagIndeks, errors),
) && !verdiForAvklarAktivitetErEndret
}
isReadOnly={readOnly || !skalVurderes}
isDirty={isDirty}
isSubmitting={submitDisabled}
hasEmptyRequiredFields={finnesFeilForBegrunnelse(beregningsgrunnlagIndeks, errors)}
hasErrors={finnesFeilForBegrunnelse(beregningsgrunnlagIndeks, errors)}
/>
</>
)}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { konverterKodeverkTilKode } from '@navikt/ft-utils-test';
import { konverterKodeverkTilKode } from '@ft-frontend-saksbehandling/ft-utils-test';

const beregningsgrunnlag = {
beregningsgrunnlagListe: [
Expand Down
5 changes: 3 additions & 2 deletions packages/fakta-beregning/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@
"storybook": "storybook dev --quiet -p 7001"
},
"dependencies": {
"@ft-frontend-saksbehandling/internal-components": "workspace:^",
"@navikt/aksel-icons": "7.1.0",
"@navikt/ds-css": "7.1.0",
"@navikt/ds-react": "7.1.0",
Expand Down Expand Up @@ -62,8 +63,8 @@
"react-intl": "6.6.8"
},
"devDependencies": {
"@navikt/ft-storybook-utils": "^2.4.1",
"@navikt/ft-utils-test": "^2.4.1",
"@ft-frontend-saksbehandling/ft-utils-test": "workspace:^",
"@ft-frontend-saksbehandling/storybook-utils": "workspace:^",
"@storybook/addon-actions": "8.3.3",
"@storybook/react": "8.3.3",
"@storybook/react-vite": "8.3.3",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import {
Inntektskategori,
OpptjeningAktivitetType,
} from '@navikt/ft-kodeverk';
import { alleKodeverk as alleKodeverkMock } from '@navikt/ft-storybook-utils';
import { alleKodeverk as alleKodeverkMock } from '@ft-frontend-saksbehandling/storybook-utils';
import {
AndelForFaktaOmBeregning,
BeregningAvklaringsbehov,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@ import React, { FunctionComponent, useState } from 'react';

import { Button, ErrorMessage, Label } from '@navikt/ds-react';
import { FormattedMessage, IntlShape, useIntl } from 'react-intl';
import { SubmitButton } from '@ft-frontend-saksbehandling/internal-components';
import { useCustomValidation } from '@navikt/ft-form-hooks';
import {
ArbeidsgiverOpplysningerPerId,
Expand All @@ -26,7 +27,6 @@ import { BeregningAktiviteterTransformedValues } from '../../typer/interface/Ber
import FaktaBeregningAvklaringsbehovCode from '../../typer/interface/FaktaBeregningAvklaringsbehovCode';
import { hasAvklaringsbehov, isAvklaringsbehovOpen } from '../felles/avklaringsbehovUtil';
import FaktaBegrunnelseTextField from '../felles/FaktaBegrunnelseTextField';
import SubmitButton from '../felles/SubmitButton';
import {
erSubmittable,
findBegrunnelse,
Expand Down Expand Up @@ -282,7 +282,7 @@ const AvklareAktiviteterField: FunctionComponent<OwnProps> = ({
isDirty={fieldIsDirty}
isSubmitting={submitDisabled}
isReadOnly={readOnly || (isAvklaringsbehovClosed && !fieldIsDirty)}
hasEmptyRequiredFields={finnesFeilForBegrunnelse}
hasErrors={finnesFeilForBegrunnelse}
/>
</FlexColumn>
{!!dirtyFields && fieldIsDirty && (
Expand Down
66 changes: 0 additions & 66 deletions packages/fakta-beregning/src/components/felles/SubmitButton.tsx

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { getIntlMock } from '@navikt/ft-utils-test';
import { getIntlMock } from '@ft-frontend-saksbehandling/ft-utils-test';
import { compareAndeler, ulikeAndelerErrorMessage, validateUlikeAndeler } from './ValidateAndelerUtils';
import messages from '../../../i18n/nb_NO.json';
import AndelFieldValue from '../../typer/FieldValues';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,12 +7,12 @@ import {
} from '@navikt/ft-types';
import { AksjonspunktHelpTextHTML, VerticalSpacer } from '@navikt/ft-ui-komponenter';
import { isAksjonspunktOpen } from '@navikt/ft-kodeverk';
import { SubmitButton } from '@ft-frontend-saksbehandling/internal-components';
import { FormattedMessage } from 'react-intl';
import { FieldErrors, UseFormGetValues, useFormContext } from 'react-hook-form';
import FaktaForATFLOgSNPanel from './FaktaForATFLOgSNPanel';
import FaktaBegrunnelseTextField from '../felles/FaktaBegrunnelseTextField';
import { formNameVurderFaktaBeregning } from '../BeregningFormUtils';
import SubmitButton from '../felles/SubmitButton';
import FaktaBeregningAvklaringsbehovCode from '../../typer/interface/FaktaBeregningAvklaringsbehovCode';
import { erOverstyringAvBeregningsgrunnlag } from './BgFaktaUtils';
import VurderFaktaBeregningFormValues from '../../typer/VurderFaktaBeregningFormValues';
Expand Down Expand Up @@ -141,20 +141,20 @@ const VurderFaktaBeregningField: FunctionComponent<OwnProps> = ({
/>
)}
<VerticalSpacer twentyPx />
{/* @ts-ignore */}
<SubmitButton
isDisabled={!!verdiForAvklarAktivitetErEndret}
isSubmittable={erSubmittable(
submittable &&
harIkkeEndringerIAvklarMedFlereAksjonspunkter(verdiForAvklarAktivitetErEndret, avklaringsbehov) &&
!isAksjonspunktClosed(avklaringsbehov),
true,
finnesFeilForBegrunnelse(beregningsgrunnlagIndeks, errors),
)}
isSubmittable={
erSubmittable(
submittable &&
harIkkeEndringerIAvklarMedFlereAksjonspunkter(verdiForAvklarAktivitetErEndret, avklaringsbehov) &&
!isAksjonspunktClosed(avklaringsbehov),
true,
finnesFeilForBegrunnelse(beregningsgrunnlagIndeks, errors),
) && !verdiForAvklarAktivitetErEndret
}
isReadOnly={readOnly || !skalVurderes}
isDirty={isDirty}
isSubmitting={submitDisabled}
hasEmptyRequiredFields={finnesFeilForBegrunnelse(beregningsgrunnlagIndeks, errors)}
hasErrors={finnesFeilForBegrunnelse(beregningsgrunnlagIndeks, errors)}
/>
</>
)}
Expand Down
2 changes: 1 addition & 1 deletion packages/fakta-beregning/testdata/bgUtenAvklaringsbehov.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { konverterKodeverkTilKode } from '@navikt/ft-utils-test';
import { konverterKodeverkTilKode } from '@ft-frontend-saksbehandling/ft-utils-test';

const beregningsgrunnlag = {
beregningsgrunnlagListe: [
Expand Down
4 changes: 2 additions & 2 deletions packages/fakta-fordel-beregningsgrunnlag/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -60,8 +60,8 @@
"uuid": "9.0.1"
},
"devDependencies": {
"@navikt/ft-storybook-utils": "^2.4.1",
"@navikt/ft-utils-test": "^2.4.1",
"@ft-frontend-saksbehandling/ft-utils-test": "workspace:^",
"@ft-frontend-saksbehandling/storybook-utils": "workspace:^",
"@storybook/addon-actions": "8.3.3",
"@storybook/react": "8.3.3",
"@storybook/react-vite": "8.3.3",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ import {
FordelBeregningsgrunnlagPeriode,
Vilkar,
} from '@navikt/ft-types';
import { alleKodeverk } from '@navikt/ft-storybook-utils';
import { alleKodeverk } from '@ft-frontend-saksbehandling/storybook-utils';

import {
bgMedDelvisRefusjon as vurderDelvisRefBG,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { getIntlMock } from '@navikt/ft-utils-test';
import { getIntlMock } from '@ft-frontend-saksbehandling/ft-utils-test';
import {
validateSumFastsattBelop,
validateTotalRefusjonPrArbeidsforhold,
Expand Down
Loading

0 comments on commit bfadcf2

Please sign in to comment.