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.

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

Open
wants to merge 33 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from 17 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
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
Original file line number Diff line number Diff line change
@@ -0,0 +1,124 @@
import React, { useEffect, useState } from 'react';

import { formatISO } from 'date-fns';

import { MonthPicker, useMonthpicker } from '@navikt/ds-react';
import { Felt } from '@navikt/familie-skjema';

import { useApp } from '../../../context/AppContext';
import { useSpråk } from '../../../context/SpråkContext';
import { ISODateString } from '../../../typer/common';
import { ESanitySteg } from '../../../typer/sanity/sanity';
import { formaterDatoKunMåned } from '../../../utils/dato';

interface IProps {
tidligsteValgbareMåned?: Date;
senesteValgbareMåned?: Date;
label: React.ReactNode;
felt: Felt<ISODateString>;
visFeilmeldinger?: boolean;
}

/* TODO
- vurder å lagre som Date og gjøre parsing/format mot mellomlagring og innsending - i neste runde
- legge feilmeldinger i sanity slik at vi får i18n: støtte flettefelt i fellestekst i formateringsfeilmeldinger, gjøres i familie-baks-soknad-sanity
*/

export enum Feilmelding {
FØR_MIN_DATO = 'FØR_MIN_DATO',
ETTER_MAKS_DATO = 'ETTER_MAKS_DATO',
UGYLDIG_DATO = 'UGYLDIG_DATO',
}

export const MånedÅrVelger: React.FC<IProps> = ({
Copy link
Contributor

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 😊

tidligsteValgbareMåned,
senesteValgbareMåned,
label,
felt,
visFeilmeldinger = false,
}) => {
const { valgtLocale } = useSpråk();
const { tekster, plainTekst } = useApp();
const [error, setError] = useState<Feilmelding | undefined>(undefined);

const nullstillOgSettFeilmelding = (feilmelding: Feilmelding) => {
if (error !== feilmelding) {
setError(feilmelding);
felt.nullstill();
}
};

const feilmeldingForDatoFørMinDato = () => {
// TODO må få i18n
if (tidligsteValgbareMåned) {
return plainTekst(
tekster()[ESanitySteg.FELLES].formateringsfeilmeldinger
.tilDatoKanIkkeVaereFoerFraDato
);
}
return plainTekst(tekster()[ESanitySteg.FELLES].formateringsfeilmeldinger.ugyldigManed);
};

const feilmeldingForDatoEtterMaksDato = () => {
// TODO må få i18n
if (senesteValgbareMåned) {
return `Du kan ikke velge en dato etter ${formaterDatoKunMåned(senesteValgbareMåned, valgtLocale)}`;
}
return plainTekst(tekster()[ESanitySteg.FELLES].formateringsfeilmeldinger.ugyldigManed);
};

const feilmeldinger: Record<Feilmelding, string> = {
UGYLDIG_DATO: 'Du må velge en gyldig dato', // TODO må få i18n
FØR_MIN_DATO: feilmeldingForDatoFørMinDato(),
ETTER_MAKS_DATO: feilmeldingForDatoEtterMaksDato(),
};

const { monthpickerProps, inputProps, reset, selectedMonth } = useMonthpicker({
fromDate: tidligsteValgbareMåned,
toDate: senesteValgbareMåned,
locale: valgtLocale,
onMonthChange: (dato: Date | undefined): void => {
if (dato === undefined) {
felt.nullstill();
} else {
felt.validerOgSettFelt(formatISO(dato, { representation: 'date' }));
}
},
onValidate: val => {
if (val.isBefore) {
nullstillOgSettFeilmelding(Feilmelding.FØR_MIN_DATO);
} else if (val.isAfter) {
nullstillOgSettFeilmelding(Feilmelding.ETTER_MAKS_DATO);
} else if (val.isEmpty || val.isDisabled || !val.isValidMonth) {
nullstillOgSettFeilmelding(Feilmelding.UGYLDIG_DATO);
} else {
setError(undefined);
}
},
});

useEffect(() => {
if (selectedMonth) {
if (
(!!tidligsteValgbareMåned && tidligsteValgbareMåned > selectedMonth) ||
(!!senesteValgbareMåned && senesteValgbareMåned < selectedMonth)
) {
reset();
}
}
}, [tidligsteValgbareMåned, senesteValgbareMåned]);
Copy link
Contributor

Choose a reason for hiding this comment

The 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 😆

image


return (
<MonthPicker {...monthpickerProps}>
<MonthPicker.Input
{...inputProps}
label={label}
error={
error && visFeilmeldinger
? feilmeldinger[error]
: felt.hentNavBaseSkjemaProps(visFeilmeldinger).error
}
/>
</MonthPicker>
);
};
Original file line number Diff line number Diff line change
@@ -1,7 +1,11 @@
import React from 'react';

import { 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';
Expand All @@ -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';
Expand Down Expand Up @@ -42,6 +47,8 @@ export const PensjonModal: React.FC<Props> = ({
erDød,
});

const { toggles } = useFeatureToggles();

const { mottarPensjonNå, pensjonTilDato, pensjonFraDato, pensjonsland } = skjema.felter;

const onLeggTil = () => {
Expand Down Expand Up @@ -119,34 +126,75 @@ 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()
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Blir dette fortsatt riktig nå med månedvelger?

Copy link
Contributor

Choose a reason for hiding this comment

The 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?
Datoen som vi sender inn til månedsvelgeren brukes kun til å bestemme hvilken måned som skal settes.

Det enkle caset:
Hvis dagens dato er 15. juni 2024 og perioden ikke er avsluttet, sender vi med dagens dato. Da er det juni 2024 som kommer frem som sist valgbare måned. I det tilfellet kunne denne logikken kanskje endres på.

Caset rundt månedsskifter:
Hvis dagens dato er 1. juni 2024 og perioden er avsluttet, skal man etter den eksisterende logikken kun få velge frem til gårsdagen, 31. mai. Da vil mai 2024 være sist valgbare måned.

Er det noe annet jeg ikke har tenkt på her? 🤔

felt={pensjonFraDato}
visFeilmeldinger={skjema.visFeilmeldinger}
/>
) : (
<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={
pensjonFraDato.verdi !== ''
? parseISO(pensjonFraDato.verdi)
: undefined
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Dette kunne evt ha blitt gjort med å lage en funksjon som isoStringTilDate som du finner i dato.ts-filen i ba-sak-frontend

senesteValgbareMåned={dagensDato()}
felt={pensjonTilDato}
visFeilmeldinger={skjema.visFeilmeldinger}
/>
) : (
<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>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,15 @@ import React from 'react';

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

import { useFeatureToggles } from '../../../context/FeatureToggleContext';
import { useSpråk } from '../../../context/SpråkContext';
import { IBarnMedISøknad } from '../../../typer/barn';
import { EFeatureToggle } from '../../../typer/feature-toggles';
import { IPensjonsperiode } 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 { 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 @@ -42,6 +45,7 @@ export const PensjonsperiodeOppsummering: React.FC<PensjonsperiodeOppsummeringPr
barn = undefined,
}) => {
const { valgtLocale } = useSpråk();
const { toggles } = useFeatureToggles();
const { mottarPensjonNå, pensjonsland, pensjonFra, pensjonTil } = pensjonsperiode;

const periodenErAvsluttet =
Expand Down Expand Up @@ -85,13 +89,25 @@ export const PensjonsperiodeOppsummering: React.FC<PensjonsperiodeOppsummeringPr
{pensjonFra.svar && (
<OppsummeringFelt
tittel={spørsmålSpråkTekst(PensjonsperiodeSpørsmålId.fraDatoPensjon)}
søknadsvar={formaterDato(pensjonFra.svar)}
søknadsvar={
toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO]
? uppercaseFørsteBokstav(
formaterDatostringKunMåned(pensjonFra.svar, valgtLocale)
)
: formaterDato(pensjonFra.svar)
}
/>
)}
{pensjonTil.svar && (
<OppsummeringFelt
tittel={spørsmålSpråkTekst(PensjonsperiodeSpørsmålId.tilDatoPensjon)}
søknadsvar={formaterDato(pensjonTil.svar)}
søknadsvar={
toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO]
? uppercaseFørsteBokstav(
formaterDatostringKunMåned(pensjonTil.svar, valgtLocale)
)
: formaterDato(pensjonTil.svar)
}
/>
)}
</PeriodeOppsummering>
Expand Down
9 changes: 8 additions & 1 deletion src/frontend/hooks/useSendInnSkjema.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,9 @@ import { RessursStatus } from '@navikt/familie-typer';
import Miljø from '../../shared-utils/Miljø';
import { erModellMismatchResponsRessurs } from '../../shared-utils/modellversjon';
import { useApp } from '../context/AppContext';
import { useFeatureToggles } from '../context/FeatureToggleContext';
import { useSpråk } from '../context/SpråkContext';
import { EFeatureToggle } from '../typer/feature-toggles';
import { ISøknadKontraktV8 } from '../typer/kontrakt/v8';
import { dataISøknadKontraktFormatV8 } from '../utils/mappingTilKontrakt/søknadV8';
import { sendInn } from '../utils/sendInnSkjema';
Expand All @@ -16,10 +18,15 @@ export const useSendInnSkjema = (): {
const { axiosRequest, søknad, settInnsendingStatus, settSisteModellVersjon } = useApp();
const { soknadApiProxyUrl } = Miljø();
const { valgtLocale } = useSpråk();
const { toggles } = useFeatureToggles();
const sendInnSkjemaV8 = async (): Promise<[boolean, ISøknadKontraktV8]> => {
settInnsendingStatus({ status: RessursStatus.HENTER });

const formatert: ISøknadKontraktV8 = dataISøknadKontraktFormatV8(valgtLocale, søknad);
const formatert: ISøknadKontraktV8 = dataISøknadKontraktFormatV8(
valgtLocale,
søknad,
toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO] === true
);

const res = await sendInn<ISøknadKontraktV8>(
formatert,
Expand Down
5 changes: 4 additions & 1 deletion src/frontend/typer/feature-toggles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,13 @@ export enum EKillSwitchToggle {
SOKNAD = 'familie-ba-soknad.disable-soknad',
}

export enum EFeatureToggle {} // EKSEMPEL = 'EKSEMPEL',
export enum EFeatureToggle {
BE_OM_MÅNED_IKKE_DATO = 'BE_OM_MÅNED_IKKE_DATO',
}

export const ToggleKeys: Record<EFeatureToggle, string> = {
// [EFeatureToggle.EKSEMPEL]: 'familie-ba-soknad.eksempel',
[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO]: 'familie-ba-soknad.be-om-mnd-ikke-dato',
};

export type EAllFeatureToggles = Record<EFeatureToggle, boolean>;
Expand Down
1 change: 1 addition & 0 deletions src/frontend/typer/sanity/sanity.ts
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ export enum Typografi {
}

export const frittståendeOrdPrefix = 'FRITTSTAENDEORD';
export const formateringsfeilmeldingerPrefix = 'FORMATERINGSFEILMELDINGER';

export interface ISanitySpørsmålDokument extends SanityDokument {
sporsmal: LocaleRecordBlock;
Expand Down
6 changes: 6 additions & 0 deletions src/frontend/typer/sanity/tekstInnhold.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ export interface ITekstinnhold {

export interface IFellesTekstInnhold {
frittståendeOrd: IFrittståendeOrdTekstinnhold;
formateringsfeilmeldinger: IFormateringsfeilmeldingerTekstinnhold;
}

export interface IFrittståendeOrdTekstinnhold {
Expand All @@ -31,3 +32,8 @@ export interface IForsideTekstinnhold {
utvidetBarnetrygdAlert: LocaleRecordBlock;
soekerDuUtvidet: ISanitySpørsmålDokument;
}

export interface IFormateringsfeilmeldingerTekstinnhold {
tilDatoKanIkkeVaereFoerFraDato: LocaleRecordString;
ugyldigManed: LocaleRecordString;
}
Loading