Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Be kun om måned der vi ikke trenger nøyaktig dato #1172

Open
wants to merge 33 commits into
base: main
Choose a base branch
from
Open

Be kun om måned der vi ikke trenger nøyaktig dato #1172

Show file tree
Hide file tree
Changes from 30 commits
Commits
Show all changes
33 commits
Select commit Hold shift + click to select a range
8069d4a
Start på månedårvelger
kristeine Apr 2, 2024
8c21db0
Be kun om måned, ikke nøyaktig dato, for pensjonsperioder utland
kristeine Apr 3, 2024
b992c5a
Fjern funksjon som ikke er i bruk
kristeine Apr 3, 2024
9278a15
Ordne stor forbokstav på månedsnavn i oppsummering
kristeine Apr 3, 2024
2d1b51c
Legg feature toggle foran
kristeine Apr 3, 2024
004dd1e
Nullstill månedsvelger dersom avgrensende verdier krysser valgt verdi
kristeine Apr 3, 2024
c05a453
Endre navn på avgrensende verdier
kristeine Apr 3, 2024
d4e8607
Bruk kun måned som verdi når vi sender inn søknaden
kristeine Apr 3, 2024
0f89e49
Merge branch 'main' into feat/be-kun-om-måned-ikke-dato
kristeine Apr 3, 2024
1b6079b
Støtt formatering både fra isostring og date
kristeine Apr 8, 2024
277eadb
Begynn på feilhåndtering og validering
kristeine Apr 8, 2024
6cf275b
Unngå å parse tom streng til dato
kristeine Apr 9, 2024
a3fe899
Oppdater todos
kristeine Apr 9, 2024
44351b3
Oppdater todos
kristeine Apr 19, 2024
388731b
Merge branch 'main' into feat/be-kun-om-måned-ikke-dato
hanneolsen May 13, 2024
1a2f879
Begynner å legge til Sanity-tekster for feilmeldingene
hanneolsen May 13, 2024
10f32c9
Flytt concurrently fra dependencies til devDependencies
halvorbmundal May 13, 2024
c3d3f5c
Rename dokumenterFiltrertPåPrefix -> dokumenterFiltrertPåType
halvorbmundal May 13, 2024
22576b4
Forenkle santiy transformasjonen
halvorbmundal May 13, 2024
1e88f10
Legg til feilmeldinger for månedvelger
halvorbmundal May 13, 2024
4e55827
Bruk produksjonsdatasett for sanity i localhost
halvorbmundal May 14, 2024
053be5b
Legger til månedsvelger på arbeidsperiode
hanneolsen May 14, 2024
0bce5d9
Endre til månedformat i arbeidsperiode i kontrakten mot backend
halvorbmundal May 14, 2024
3a6d3b7
Legg til månedsformat i ArbeidsperiodeOppsummering
halvorbmundal May 14, 2024
2ec5973
Merge branch 'main' into feat/be-kun-om-måned-ikke-dato
halvorbmundal May 15, 2024
7084b47
Legg til tidligsteValgbareMåned på tom-dato for arbeidsperioder
halvorbmundal May 15, 2024
05d4b1a
Legger til månedsvisning på utbetalingsperiode, fikser måned med ukje…
hanneolsen May 15, 2024
cafa352
Hente toggle fra parameter i hovedfunksjonen i stedet
hanneolsen May 16, 2024
864b7e8
Merge branch 'main' into feat/be-kun-om-måned-ikke-dato
hanneolsen May 16, 2024
524ecad
Legger til logikk for å velge samme fra og til måned, legger til feil…
hanneolsen May 28, 2024
b31038c
Omstrukturerer unleash toggle-sjekk for MånedÅrVelger
hanneolsen Jun 24, 2024
75125f7
Omstrukturerer unleash toggle for UtbetalingerModal
hanneolsen Jun 24, 2024
c9c800b
Fikser typo
hanneolsen Jun 24, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,6 @@
"amplitude-js": "^8.21.9",
"axios": "^1.6.2",
"compression": "^1.7.4",
"concurrently": "^8.2.2",
"constate": "^3.3.2",
"cookie-parser": "^1.4.6",
"cors": "^2.8.5",
Expand Down Expand Up @@ -163,6 +162,7 @@
"buffer": "^6.0.3",
"case-sensitive-paths-webpack-plugin": "2.4.0",
"classnames": "^2.5.1",
"concurrently": "^8.2.2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Brukes kun til test og i byggesteget

"copy-webpack-plugin": "^12.0.2",
"crypto-browserify": "^3.12.0",
"css-loader": "^7.1.1",
Expand Down
123 changes: 90 additions & 33 deletions src/frontend/components/Felleskomponenter/Arbeidsperiode/ArbeidsperiodeModal.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,19 @@ import React from 'react';

import { ESvar } from '@navikt/familie-form-elements';

import { useFeatureToggles } from '../../../context/FeatureToggleContext';
import { EFeatureToggle } from '../../../typer/feature-toggles';
import { IArbeidsperiode } from '../../../typer/perioder';
import { PersonType } from '../../../typer/personType';
import { dagensDato, gårsdagensDato } from '../../../utils/dato';
import { dagensDato, gårsdagensDato, sisteDagDenneMåneden } from '../../../utils/dato';
import { trimWhiteSpace, visFeiloppsummering } from '../../../utils/hjelpefunksjoner';
import { minTilDatoForUtbetalingEllerArbeidsperiode } from '../../../utils/perioder';
import { svarForSpørsmålMedUkjent } from '../../../utils/spørsmål';
import Datovelger from '../Datovelger/Datovelger';
import { LandDropdown } from '../Dropdowns/LandDropdown';
import JaNeiSpm from '../JaNeiSpm/JaNeiSpm';
import KomponentGruppe from '../KomponentGruppe/KomponentGruppe';
import { DagIMåneden, MånedÅrVelger } from '../MånedÅrVelger/MånedÅrVelger';
import { SkjemaCheckbox } from '../SkjemaCheckbox/SkjemaCheckbox';
import { SkjemaFeiloppsummering } from '../SkjemaFeiloppsummering/SkjemaFeiloppsummering';
import { SkjemaFeltInput } from '../SkjemaFeltInput/SkjemaFeltInput';
Expand Down Expand Up @@ -49,6 +52,8 @@ export const ArbeidsperiodeModal: React.FC<ArbeidsperiodeModalProps> = ({
tilDatoArbeidsperiodeUkjent,
} = skjema.felter;

const { toggles } = useFeatureToggles();

const onLeggTil = () => {
if (!validerFelterOgVisFeilmelding()) {
return false;
Expand Down Expand Up @@ -140,48 +145,100 @@ export const ArbeidsperiodeModal: React.FC<ArbeidsperiodeModalProps> = ({
)}
/>
)}
{fraDatoArbeidsperiode.erSynlig && (
<Datovelger
felt={skjema.felter.fraDatoArbeidsperiode}
skjema={skjema}
label={
<SpråkTekst
id={hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.fraDatoArbeidsperiode
)}
/>
}
avgrensMaxDato={periodenErAvsluttet ? gårsdagensDato() : dagensDato()}
/>
)}
{tilDatoArbeidsperiode.erSynlig && (
<>
{fraDatoArbeidsperiode.erSynlig &&
(toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO] ? (
<MånedÅrVelger
label={
<SpråkTekst
id={hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.fraDatoArbeidsperiode
)}
/>
}
senesteValgbareMåned={
periodenErAvsluttet ? gårsdagensDato() : dagensDato()
}
felt={skjema.felter.fraDatoArbeidsperiode}
visFeilmeldinger={skjema.visFeilmeldinger}
dagIMåneden={DagIMåneden.FØRSTE_DAG}
kanIkkeVæreFremtid={true}
/>
) : (
<Datovelger
felt={skjema.felter.tilDatoArbeidsperiode}
felt={skjema.felter.fraDatoArbeidsperiode}
skjema={skjema}
label={
<SpråkTekst
id={hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiode
ArbeidsperiodeSpørsmålsId.fraDatoArbeidsperiode
)}
/>
}
avgrensMinDato={minTilDatoForUtbetalingEllerArbeidsperiode(
periodenErAvsluttet,
skjema.felter.fraDatoArbeidsperiode.verdi
)}
avgrensMaxDato={periodenErAvsluttet ? dagensDato() : undefined}
disabled={skjema.felter.tilDatoArbeidsperiodeUkjent.verdi === ESvar.JA}
avgrensMaxDato={periodenErAvsluttet ? gårsdagensDato() : dagensDato()}
/>
))}
{tilDatoArbeidsperiode.erSynlig &&
(toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO] ? (
<>
<MånedÅrVelger
label={
<SpråkTekst
id={hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiode
)}
/>
}
tidligsteValgbareMåned={minTilDatoForUtbetalingEllerArbeidsperiode(
periodenErAvsluttet,
skjema.felter.fraDatoArbeidsperiode.verdi
)}
senesteValgbareMåned={
periodenErAvsluttet ? sisteDagDenneMåneden() : undefined
}
felt={skjema.felter.tilDatoArbeidsperiode}
visFeilmeldinger={skjema.visFeilmeldinger}
dagIMåneden={DagIMåneden.SISTE_DAG}
kanIkkeVæreFremtid={periodenErAvsluttet}
kanIkkeVæreFortid={!periodenErAvsluttet}
/>

<SkjemaCheckbox
felt={skjema.felter.tilDatoArbeidsperiodeUkjent}
labelSpråkTekstId={hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiodeVetIkke
)}
/>
</>
)}
<SkjemaCheckbox
felt={skjema.felter.tilDatoArbeidsperiodeUkjent}
labelSpråkTekstId={hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiodeVetIkke
)}
/>
</>
) : (
<>
<Datovelger
felt={skjema.felter.tilDatoArbeidsperiode}
skjema={skjema}
label={
<SpråkTekst
id={hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiode
)}
/>
}
avgrensMinDato={minTilDatoForUtbetalingEllerArbeidsperiode(
periodenErAvsluttet,
skjema.felter.fraDatoArbeidsperiode.verdi
)}
avgrensMaxDato={periodenErAvsluttet ? dagensDato() : undefined}
disabled={
skjema.felter.tilDatoArbeidsperiodeUkjent.verdi === ESvar.JA
}
/>

<SkjemaCheckbox
felt={skjema.felter.tilDatoArbeidsperiodeUkjent}
labelSpråkTekstId={hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiodeVetIkke
)}
/>
</>
))}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kunne vurdert å legge if (toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO]) "ytterst" også ha både fom og tom med ny MånedÅr sammen og ellers fom og tom med dato i else. Synes det blir litt enklere å få oversikt over hva som er nytt og gammelt på den måten.

SkjemaCheckbox kan vel også ligge utenfor feature toggle if'en dersom den er lik i begge tilfeller? Med forbehold om at det er noe her jeg har misset 🙈

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Enig i dette, både lettere å lese og enklere å fjerne koden når toggelen skal slettes 🚀

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Enig i dette, fikset det nå 😊

</KomponentGruppe>
{visFeiloppsummering(skjema) && <SkjemaFeiloppsummering skjema={skjema} />}
</SkjemaModal>
Expand Down
25 changes: 20 additions & 5 deletions src/frontend/components/Felleskomponenter/Arbeidsperiode/ArbeidsperiodeOppsummering.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,12 +4,14 @@ import { useIntl } from 'react-intl';

import { ESvar } from '@navikt/familie-form-elements';

import { useFeatureToggles } from '../../../context/FeatureToggleContext';
import { useSpråk } from '../../../context/SpråkContext';
import { EFeatureToggle } from '../../../typer/feature-toggles';
import { IArbeidsperiode } from '../../../typer/perioder';
import { PersonType } from '../../../typer/personType';
import { formaterDato } from '../../../utils/dato';
import { formaterDato, formaterDatostringKunMåned } from '../../../utils/dato';
import { landkodeTilSpråk } from '../../../utils/språk';
import { formaterDatoMedUkjent } from '../../../utils/visning';
import { formaterMånedMedUkjent, uppercaseFørsteBokstav } from '../../../utils/visning';
import { OppsummeringFelt } from '../../SøknadsSteg/Oppsummering/OppsummeringFelt';
import PeriodeOppsummering from '../PeriodeOppsummering/PeriodeOppsummering';
import SpråkTekst from '../SpråkTekst/SpråkTekst';
Expand Down Expand Up @@ -44,6 +46,8 @@ export const ArbeidsperiodeOppsummering: React.FC<ArbeidsperiodeOppsummeringProp
}) => {
const { valgtLocale } = useSpråk();
const intl = useIntl();
const { toggles } = useFeatureToggles();

const { formatMessage } = intl;
const {
arbeidsperiodeAvsluttet,
Expand Down Expand Up @@ -93,19 +97,30 @@ export const ArbeidsperiodeOppsummering: React.FC<ArbeidsperiodeOppsummeringProp
{fraDatoArbeidsperiode.svar && (
<OppsummeringFelt
tittel={spørsmålSpråkTekst(ArbeidsperiodeSpørsmålsId.fraDatoArbeidsperiode)}
søknadsvar={formaterDato(fraDatoArbeidsperiode.svar)}
søknadsvar={
toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO]
? uppercaseFørsteBokstav(
formaterDatostringKunMåned(
fraDatoArbeidsperiode.svar,
valgtLocale
)
)
: formaterDato(fraDatoArbeidsperiode.svar)
}
/>
)}
{tilDatoArbeidsperiode.svar && (
<OppsummeringFelt
tittel={spørsmålSpråkTekst(ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiode)}
søknadsvar={formaterDatoMedUkjent(
søknadsvar={formaterMånedMedUkjent(
tilDatoArbeidsperiode.svar,
formatMessage({
id: hentSpørsmålTekstId(
ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiodeVetIkke
),
})
}),
toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO],
valgtLocale
)}
/>
)}
Expand Down
12 changes: 11 additions & 1 deletion src/frontend/components/Felleskomponenter/Arbeidsperiode/useArbeidsperiodeSkjema.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,20 @@ import { ESvar } from '@navikt/familie-form-elements';
import { useFelt, useSkjema, Valideringsstatus } from '@navikt/familie-skjema';

import { useEøs } from '../../../context/EøsContext';
import { useFeatureToggles } from '../../../context/FeatureToggleContext';
import useDatovelgerFelt from '../../../hooks/useDatovelgerFelt';
import useDatovelgerFeltMedUkjent from '../../../hooks/useDatovelgerFeltMedUkjent';
import useInputFelt from '../../../hooks/useInputFelt';
import useJaNeiSpmFelt from '../../../hooks/useJaNeiSpmFelt';
import useLanddropdownFelt from '../../../hooks/useLanddropdownFelt';
import { EFeatureToggle } from '../../../typer/feature-toggles';
import { PersonType } from '../../../typer/personType';
import { IArbeidsperioderFeltTyper } from '../../../typer/skjema';
import {
dagensDato,
erSammeDatoSomDagensDato,
gårsdagensDato,
sisteDagDenneMåneden,
stringTilDate,
} from '../../../utils/dato';
import { minTilDatoForUtbetalingEllerArbeidsperiode } from '../../../utils/perioder';
Expand All @@ -35,6 +38,7 @@ export const useArbeidsperiodeSkjema = (
erDød = false
) => {
const { erEøsLand } = useEøs();
const { toggles } = useFeatureToggles();

const andreForelderErDød = personType === PersonType.AndreForelder && erDød;

Expand Down Expand Up @@ -85,6 +89,10 @@ export const useArbeidsperiodeSkjema = (
avhengigheter: { arbeidsperiodeAvsluttet, arbeidsperiodeLand },
});

const tilArbeidsperiodeSluttdatoAvgrensning = toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO]
? sisteDagDenneMåneden()
: dagensDato();

const tilDatoArbeidsperiode = useDatovelgerFeltMedUkjent({
feltId: ArbeidsperiodeSpørsmålsId.tilDatoArbeidsperiode,
initiellVerdi: '',
Expand All @@ -94,7 +102,9 @@ export const useArbeidsperiodeSkjema = (
? !!erEøsLand(arbeidsperiodeLand.verdi)
: arbeidsperiodeAvsluttet.valideringsstatus === Valideringsstatus.OK ||
andreForelderErDød,
sluttdatoAvgrensning: periodenErAvsluttet ? dagensDato() : undefined,
sluttdatoAvgrensning: periodenErAvsluttet
? tilArbeidsperiodeSluttdatoAvgrensning
: undefined,
startdatoAvgrensning: minTilDatoForUtbetalingEllerArbeidsperiode(
periodenErAvsluttet,
fraDatoArbeidsperiode.verdi
Expand Down
Loading
Loading