-
Notifications
You must be signed in to change notification settings - Fork 2
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.
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
Be kun om måned der vi ikke trenger nøyaktig dato #1172
base: main
Are you sure you want to change the base?
The head ref may contain hidden characters: "feat/be-kun-om-m\u00E5ned-ikke-dato"
Changes from 9 commits
8069d4a
8c21db0
b992c5a
9278a15
2d1b51c
004dd1e
c05a453
d4e8607
0f89e49
1b6079b
277eadb
6cf275b
a3fe899
44351b3
388731b
1a2f879
10f32c9
c3d3f5c
22576b4
1e88f10
4e55827
053be5b
0bce5d9
3a6d3b7
2ec5973
7084b47
05d4b1a
cafa352
864b7e8
524ecad
b31038c
75125f7
c9c800b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,44 @@ | ||
import React, { useEffect } from 'react'; | ||
|
||
import { MonthPicker, useMonthpicker } from '@navikt/ds-react'; | ||
import { useSprakContext } from '@navikt/familie-sprakvelger'; | ||
|
||
interface IProps { | ||
tidligsteValgbareMåned?: Date; | ||
senesteValgbareMåned?: Date; | ||
label: React.ReactNode; | ||
onChange: (date: Date) => void; | ||
} | ||
|
||
export const MånedÅrVelger: React.FC<IProps> = ({ | ||
tidligsteValgbareMåned, | ||
senesteValgbareMåned, | ||
label, | ||
onChange, | ||
}) => { | ||
const [valgtLocale] = useSprakContext(); | ||
|
||
const { monthpickerProps, inputProps, reset, selectedMonth } = useMonthpicker({ | ||
fromDate: tidligsteValgbareMåned, | ||
toDate: senesteValgbareMåned, | ||
locale: valgtLocale, | ||
onMonthChange: (date?: Date) => date && onChange(date), | ||
}); | ||
|
||
useEffect(() => { | ||
if (selectedMonth) { | ||
if ( | ||
(!!tidligsteValgbareMåned && tidligsteValgbareMåned > selectedMonth) || | ||
(!!senesteValgbareMåned && senesteValgbareMåned < selectedMonth) | ||
) { | ||
reset(); | ||
} | ||
} | ||
}, [tidligsteValgbareMåned, senesteValgbareMåned]); | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Logikken for å resette et felt basert på et annet felt funker kun hvis man velger datoer ved å klikke i månedsvelgeren. Jeg får lov til å skrive inn en ugyldig dato manuelt, og det kan virke som det underliggende skjemafeltet som holder på verdien ikke blir satt, noe som gjør at feilmeldingen bruker får er litt rar 🙃 Lurer på om vi kan få en del gratis her hvis vi lager en implementasjon av Månedvelgeren som kombinerer sånn det er gjort for Månedvelger i ba-sak-frontend og for Datovelgeren her i ba-soknad. Syns eeegentlig det smudeste hadde vært å skrive om sånn at skjemafeltet ikke holdt på en ISOString, men heller et Date-objekt (sånn som vi har skrevet om i ba-sak og ks-sak) - men da må man i så fall også sørge for at det blir gjort om til isostring før man sender det til backend (og her også kanskje noe greier med mellomlagring, men er ikke så kjent med koblingen mellom skjemafeltene og det som mellomlagres) Mye greier å skrive, så kanskje like greit å ta en muntlig prat i morgen i stedet 😆 |
||
|
||
return ( | ||
<MonthPicker {...monthpickerProps}> | ||
<MonthPicker.Input {...inputProps} label={label} /> | ||
</MonthPicker> | ||
); | ||
}; |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,11 @@ | ||
import React from 'react'; | ||
|
||
import { formatISO, parseISO } from 'date-fns'; | ||
|
||
import { ESvar } from '@navikt/familie-form-elements'; | ||
|
||
import { useFeatureToggles } from '../../../context/FeatureToggleContext'; | ||
import { EFeatureToggle } from '../../../typer/feature-toggles'; | ||
import { IPensjonsperiode } from '../../../typer/perioder'; | ||
import { PersonType } from '../../../typer/personType'; | ||
import { dagensDato, gårsdagensDato } from '../../../utils/dato'; | ||
|
@@ -10,6 +14,7 @@ import Datovelger from '../Datovelger/Datovelger'; | |
import { LandDropdown } from '../Dropdowns/LandDropdown'; | ||
import JaNeiSpm from '../JaNeiSpm/JaNeiSpm'; | ||
import KomponentGruppe from '../KomponentGruppe/KomponentGruppe'; | ||
import { MånedÅrVelger } from '../MånedÅrVelger/MånedÅrVelger'; | ||
import { SkjemaFeiloppsummering } from '../SkjemaFeiloppsummering/SkjemaFeiloppsummering'; | ||
import SkjemaModal from '../SkjemaModal/SkjemaModal'; | ||
import SpråkTekst from '../SpråkTekst/SpråkTekst'; | ||
|
@@ -42,6 +47,8 @@ export const PensjonModal: React.FC<Props> = ({ | |
erDød, | ||
}); | ||
|
||
const { toggles } = useFeatureToggles(); | ||
|
||
const { mottarPensjonNå, pensjonTilDato, pensjonFraDato, pensjonsland } = skjema.felter; | ||
|
||
const onLeggTil = () => { | ||
|
@@ -119,34 +126,77 @@ export const PensjonModal: React.FC<Props> = ({ | |
/> | ||
)} | ||
|
||
{pensjonFraDato.erSynlig && ( | ||
<Datovelger | ||
felt={pensjonFraDato} | ||
label={ | ||
<SpråkTekst | ||
id={hentSpørsmålTekstId(PensjonsperiodeSpørsmålId.fraDatoPensjon)} | ||
values={{ ...(barn && { barn: barn.navn }) }} | ||
/> | ||
} | ||
skjema={skjema} | ||
avgrensMaxDato={periodenErAvsluttet ? gårsdagensDato() : dagensDato()} | ||
/> | ||
)} | ||
{pensjonTilDato.erSynlig && ( | ||
<Datovelger | ||
felt={pensjonTilDato} | ||
label={ | ||
<SpråkTekst | ||
id={hentSpørsmålTekstId(PensjonsperiodeSpørsmålId.tilDatoPensjon)} | ||
values={{ ...(barn && { barn: barn.navn }) }} | ||
/> | ||
} | ||
skjema={skjema} | ||
avgrensMaxDato={dagensDato()} | ||
tilhørendeFraOgMedFelt={pensjonFraDato} | ||
dynamisk | ||
/> | ||
)} | ||
{pensjonFraDato.erSynlig && | ||
(toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO] ? ( | ||
<MånedÅrVelger | ||
label={ | ||
<SpråkTekst | ||
id={hentSpørsmålTekstId( | ||
PensjonsperiodeSpørsmålId.fraDatoPensjon | ||
)} | ||
values={{ ...(barn && { barn: barn.navn }) }} | ||
/> | ||
} | ||
senesteValgbareMåned={ | ||
periodenErAvsluttet ? gårsdagensDato() : dagensDato() | ||
} | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Blir dette fortsatt riktig nå med månedvelger? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ja, det tror jeg. Men det kan hende det er noe jeg overser? Det enkle caset: Caset rundt månedsskifter: Er det noe annet jeg ikke har tenkt på her? 🤔 |
||
onChange={dato => | ||
pensjonFraDato.validerOgSettFelt( | ||
formatISO(dato, { representation: 'date' }) | ||
) | ||
} | ||
/> | ||
) : ( | ||
<Datovelger | ||
felt={pensjonFraDato} | ||
label={ | ||
<SpråkTekst | ||
id={hentSpørsmålTekstId( | ||
PensjonsperiodeSpørsmålId.fraDatoPensjon | ||
)} | ||
values={{ ...(barn && { barn: barn.navn }) }} | ||
/> | ||
} | ||
skjema={skjema} | ||
avgrensMaxDato={periodenErAvsluttet ? gårsdagensDato() : dagensDato()} | ||
/> | ||
))} | ||
{pensjonTilDato.erSynlig && | ||
(toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO] ? ( | ||
<MånedÅrVelger | ||
label={ | ||
<SpråkTekst | ||
id={hentSpørsmålTekstId( | ||
PensjonsperiodeSpørsmålId.tilDatoPensjon | ||
)} | ||
values={{ ...(barn && { barn: barn.navn }) }} | ||
/> | ||
} | ||
tidligsteValgbareMåned={parseISO(pensjonFraDato.verdi)} | ||
senesteValgbareMåned={dagensDato()} | ||
onChange={dato => | ||
pensjonTilDato.validerOgSettFelt( | ||
formatISO(dato, { representation: 'date' }) | ||
) | ||
} | ||
/> | ||
) : ( | ||
<Datovelger | ||
felt={pensjonTilDato} | ||
label={ | ||
<SpråkTekst | ||
id={hentSpørsmålTekstId( | ||
PensjonsperiodeSpørsmålId.tilDatoPensjon | ||
)} | ||
values={{ ...(barn && { barn: barn.navn }) }} | ||
/> | ||
} | ||
skjema={skjema} | ||
avgrensMaxDato={dagensDato()} | ||
tilhørendeFraOgMedFelt={pensjonFraDato} | ||
dynamisk | ||
/> | ||
))} | ||
</KomponentGruppe> | ||
{visFeiloppsummering(skjema) && <SkjemaFeiloppsummering skjema={skjema} />} | ||
</SkjemaModal> | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Hadde egentlig vært litt nice å gjøre om funksjonene i denne filen til å ligne på hvordan vi gjør det i ks- og ba-sak og i familie-tilbake så blir det enklere forvaltning |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jeg liker navnet
Månedvelger
bedre så det stemmer overens med ba-sak-frontend og ks-sak-frontend 😊