Be kun om måned der vi ikke trenger nøyaktig dato #1172
Conversation
| 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.
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 😆
| @@ -163,6 +162,7 @@ | |||
| "buffer": "^6.0.3", | |||
| "case-sensitive-paths-webpack-plugin": "2.4.0", | |||
| "classnames": "^2.5.1", | |||
| "concurrently": "^8.2.2", | |||
There was a problem hiding this comment.
Brukes kun til test og i byggesteget
…nt verdi på arbeidsperiode
…meldinger for fremtid og fortid
| if (dato === undefined) { | ||
| felt.nullstill(); | ||
| } else { | ||
| if (dagIMåneden === DagIMåneden.FØRSTE_DAG) { |
There was a problem hiding this comment.
For datakvaliteten sin del er det fint om vi lagrer fra og til som første og siste dag i måneden i databasen. I PDFen og oppsummeringen vår er det kun måneden og året som er listet opp, men vi lagrer en dato til databasen. Derfor returnerer vi første eller siste dag i måneden her. Første dag i måneden er den som default blir returnert fra Aksel sin komponent, som vi bruker her.
hanneolsen
changed the title
| 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 | ||
| )} | ||
| /> | ||
| </> | ||
| ))} |
There was a problem hiding this comment.
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 🙈
There was a problem hiding this comment.
Enig i dette, både lettere å lese og enklere å fjerne koden når toggelen skal slettes 🚀
There was a problem hiding this comment.
Enig i dette, fikset det nå 😊
| {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() | ||
| } | ||
| felt={pensjonFraDato} | ||
| visFeilmeldinger={skjema.visFeilmeldinger} | ||
| dagIMåneden={DagIMåneden.FØRSTE_DAG} | ||
| kanIkkeVæreFremtid={true} | ||
| /> | ||
| ) : ( | ||
| <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 | ||
| } | ||
| senesteValgbareMåned={sisteDagDenneMåneden()} | ||
| felt={pensjonTilDato} | ||
| visFeilmeldinger={skjema.visFeilmeldinger} | ||
| dagIMåneden={DagIMåneden.SISTE_DAG} | ||
| kanIkkeVæreFremtid={periodenErAvsluttet} | ||
| kanIkkeVæreFortid={!periodenErAvsluttet} | ||
| /> | ||
| ) : ( | ||
| <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 | ||
| /> | ||
| ))} |
There was a problem hiding this comment.
Samme som kommentaren over. Skulle gjerne sett at fom og tom med ny MånedÅrVelger lå inne i samme if(toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO]) og at fom og tom med dato lå sammen i else.
| )} | ||
| /> | ||
| } | ||
| senesteValgbareMåned={ | ||
| periodenErAvsluttet ? gårsdagensDato() : dagensDato() | ||
| } | ||
| felt={utbetalingFraDato} | ||
| visFeilmeldinger={skjema.visFeilmeldinger} | ||
| dagIMåneden={DagIMåneden.FØRSTE_DAG} | ||
| kanIkkeVæreFremtid={true} | ||
| /> | ||
| ) : ( | ||
| <Datovelger | ||
| skjema={skjema} | ||
| felt={utbetalingTilDato} | ||
| felt={utbetalingFraDato} | ||
| label={ | ||
| <SpråkTekst | ||
| id={hentSpørsmålTekstId( | ||
| UtbetalingerSpørsmålId.utbetalingTilDato | ||
| UtbetalingerSpørsmålId.utbetalingFraDato | ||
| )} | ||
| /> | ||
| } | ||
| avgrensMaxDato={periodenErAvsluttet ? dagensDato() : undefined} | ||
| avgrensMinDato={minTilDatoForUtbetalingEllerArbeidsperiode( | ||
| periodenErAvsluttet, | ||
| utbetalingFraDato.verdi | ||
| )} | ||
| disabled={utbetalingTilDatoUkjent.verdi === ESvar.JA} | ||
| /> | ||
| <SkjemaCheckbox | ||
| labelSpråkTekstId={hentSpørsmålTekstId( | ||
| UtbetalingerSpørsmålId.utbetalingTilDatoVetIkke | ||
| )} | ||
| felt={utbetalingTilDatoUkjent} | ||
| avgrensMaxDato={periodenErAvsluttet ? gårsdagensDato() : dagensDato()} | ||
| /> | ||
| </> | ||
| )} | ||
|
|
||
| {toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO] ? ( | ||
| <> | ||
| <MånedÅrVelger | ||
| label={ | ||
| <SpråkTekst | ||
| id={hentSpørsmålTekstId( | ||
| UtbetalingerSpørsmålId.utbetalingTilDato | ||
| )} | ||
| values={{ ...(barn && { barn: barn.navn }) }} | ||
| /> | ||
| } | ||
| tidligsteValgbareMåned={minTilDatoForUtbetalingEllerArbeidsperiode( | ||
| periodenErAvsluttet, | ||
| utbetalingFraDato.verdi | ||
| )} | ||
| senesteValgbareMåned={ | ||
| periodenErAvsluttet ? sisteDagDenneMåneden() : undefined | ||
| } | ||
| felt={utbetalingTilDato} | ||
| visFeilmeldinger={skjema.visFeilmeldinger} | ||
| dagIMåneden={DagIMåneden.SISTE_DAG} | ||
| kanIkkeVæreFremtid={periodenErAvsluttet} | ||
| kanIkkeVæreFortid={!periodenErAvsluttet} | ||
| disabled={utbetalingTilDatoUkjent.verdi === ESvar.JA} | ||
| /> | ||
| <SkjemaCheckbox | ||
| labelSpråkTekstId={hentSpørsmålTekstId( | ||
| UtbetalingerSpørsmålId.utbetalingTilDatoVetIkke | ||
| )} | ||
| felt={utbetalingTilDatoUkjent} | ||
| /> | ||
| </> | ||
| ) : ( | ||
| <> | ||
| <Datovelger | ||
| skjema={skjema} | ||
| felt={utbetalingTilDato} | ||
| label={ | ||
| <SpråkTekst | ||
| id={hentSpørsmålTekstId( | ||
| UtbetalingerSpørsmålId.utbetalingTilDato | ||
| )} | ||
| /> | ||
| } | ||
| avgrensMaxDato={periodenErAvsluttet ? dagensDato() : undefined} | ||
| avgrensMinDato={minTilDatoForUtbetalingEllerArbeidsperiode( | ||
| periodenErAvsluttet, | ||
| utbetalingFraDato.verdi | ||
| )} | ||
| disabled={utbetalingTilDatoUkjent.verdi === ESvar.JA} | ||
| /> | ||
| <SkjemaCheckbox | ||
| labelSpråkTekstId={hentSpørsmålTekstId( | ||
| UtbetalingerSpørsmålId.utbetalingTilDatoVetIkke | ||
| )} | ||
| felt={utbetalingTilDatoUkjent} | ||
| /> | ||
| </> | ||
| )} |
There was a problem hiding this comment.
Samme som over. Ny MånedÅrVelger for fom og tom i if og gammel Datovelger i else. SkjemaCheckbox utenfor.
| @@ -57,7 +57,7 @@ const Miljø = (): MiljøProps => { | |||
| }; | |||
| } else { | |||
| return { | |||
| sanityDataset: 'test', | |||
| sanityDataset: 'production', | |||
There was a problem hiding this comment.
Meningen at denne skulle endres til production?
There was a problem hiding this comment.
Jepp, Halvor og jeg diskuterte oss frem til at vi ønsker å bruke prodmiljøet som default, slik at vi kan se hvordan ting reelt blir seende ut i prod når vi utvikler. Vi har fremdeles muligheten til å bytte til testmiljøet hvis vi vil teste ut noe lokalt.
| /* TODO | ||
| - vurder å lagre som Date og gjøre parsing/format mot mellomlagring og innsending - i neste runde | ||
| */ |
There was a problem hiding this comment.
Er det meningen at denne skal ligge igjen?
There was a problem hiding this comment.
Jepp, det er ikke noe jeg tenkte å se på i denne omgangen. Som vi diskuterte for lenge siden så hadde det kanskje vært fint å få gjort, men ikke akkurat nå. Denne endringen til månedsvelger er stor nok allerede. 😃
| kanIkkeVæreFortid?: boolean; | ||
| kanIkkeVæreFremtid?: boolean; |
There was a problem hiding this comment.
Lurer på om jeg liker navnene kanKunVelgeFortid og kanKunVelgeFremtid bedre 😊
| senesteValgbareMåned={ | ||
| periodenErAvsluttet ? gårsdagensDato() : dagensDato() | ||
| } |
There was a problem hiding this comment.
Blir dette fortsatt riktig nå med månedvelger?
There was a problem hiding this comment.
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? 🤔
| if (dato === undefined) { | ||
| felt.nullstill(); | ||
| } else { | ||
| if (dagIMåneden === DagIMåneden.FØRSTE_DAG) { | ||
| felt.validerOgSettFelt(formatISO(dato, { representation: 'date' })); | ||
| } else { | ||
| felt.validerOgSettFelt( | ||
| formatISO(lastDayOfMonth(dato), { representation: 'date' }) | ||
| ); | ||
| } | ||
| } |
There was a problem hiding this comment.
| if (dato === undefined) { | |
| felt.nullstill(); | |
| } else { | |
| if (dagIMåneden === DagIMåneden.FØRSTE_DAG) { | |
| felt.validerOgSettFelt(formatISO(dato, { representation: 'date' })); | |
| } else { | |
| felt.validerOgSettFelt( | |
| formatISO(lastDayOfMonth(dato), { representation: 'date' }) | |
| ); | |
| } | |
| } | |
| if (dato === undefined) { | |
| felt.nullstill(); | |
| } else { | |
| felt.validerOfSettFelt(formatISO(formatterTilRiktigDagIMåneden(dato))) | |
| } |
Må da legge til denne funksjonen et annet sted i samme fil
const formatterTilRiktigDagIMåneden = (dato: Date): Date => {
if (dagIMåneden === DagIMåneden.FØRSTE_DAG) return startOfMonth(dato);
else if (dagIMåneden === DagIMåneden.SISTE_DAG) return endOfMonth(dato);
else return dato;
};
There was a problem hiding this comment.
Hadde forsåvidt også vært nice å gjøre formattering av ISO-dato på samme måte som i ks- og ba-sak-frontend 😄
Litt overkill for akkurat dette eksemplet kanskje 😅 kan evt droppe å ha med så mange datoformater
Da må dette legges til i dato.ts-filen:
export enum Datoformat {
DATO = 'dd.MM.yyyy',
DATO_FORKORTTET = 'dd.MM.yy',
DATO_FORLENGET = 'PPP',
DATO_FORLENGET_MED_TID = 'PPPp',
ISO_MÅNED = 'yyyy-MM',
ISO_DAG = 'yyyy-MM-dd',
DATO_TID = 'dd.MM.yy HH:mm',
DATO_TID_SEKUNDER = 'dd.MM.yy HH:mm:ss',
MÅNED_ÅR = 'MM.yyyy',
MÅNED_ÅR_NAVN = 'MMMM yyyy',
MÅNED_ÅR_KORTNAVN = 'MMM yyyy',
MÅNED_NAVN = 'MMM',
}
interface DateTilFormatertStringProps {
date?: Date;
tilFormat: Datoformat;
defaultString?: string;
}
export const dateTilFormatertString = ({
date,
tilFormat,
defaultString = '',
}: DateTilFormatertStringProps): string => {
return date && isValid(date) ? format(date, tilFormat) : defaultString;
};
export const dateTilIsoDatoString = (dato?: Date): IsoDatoString =>
dateTilFormatertString({ date: dato, tilFormat: Datoformat.ISO_DAG, defaultString: '' });
Også kan du da bytte ut
felt.validerOfSettFelt(formatISO(formatterTilRiktigDagIMåneden(dato)))
med
felt.validerOfSettFelt(dateTilIsoDatoString(formatterTilRiktigDagIMåneden(dato))
| pensjonFraDato.verdi !== '' | ||
| ? parseISO(pensjonFraDato.verdi) | ||
| : undefined | ||
| } |
There was a problem hiding this comment.
Dette kunne evt ha blitt gjort med å lage en funksjon som isoStringTilDate som du finner i dato.ts-filen i ba-sak-frontend
| SISTE_DAG = 'SISTE_DAG', | ||
| } | ||
|
|
||
| export const MånedÅrVelger: React.FC<IProps> = ({ |
There was a problem hiding this comment.
Jeg liker navnet Månedvelger bedre så det stemmer overens med ba-sak-frontend og ks-sak-frontend 😊
| søknadsvar={ | ||
| toggles[EFeatureToggle.BE_OM_MÅNED_IKKE_DATO] | ||
| ? uppercaseFørsteBokstav( | ||
| formaterDatostringKunMåned(utbetalingFraDato.svar, valgtLocale) | ||
| ) | ||
| : formaterDato(utbetalingFraDato.svar) | ||
| } |
There was a problem hiding this comment.
Vil du legge med screenshots av hvordan dette ser ut før og nå?
| if (svar === AlternativtSvarForInput.UKJENT) { | ||
| return vetIkkeTekst; |
There was a problem hiding this comment.
Vil denne noen gang treffe inn? Klarer ikke helt å skjønne når svar blir satt til UKJENT 😅
src/frontend/utils/dato.tsx
Outdated
There was a problem hiding this comment.
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
| datoKanIkkeVareIFortid: LocaleRecordString; | ||
| datoKanIkkeVareIFremtid: LocaleRecordString; |
There was a problem hiding this comment.
Foretrekker om vi også endrer disse til å ikke ha "ikke" i seg 😅
Altså kanKunVelgeFortid og kanKunVelgeFremtid
There was a problem hiding this comment.
Det var det jeg også ville ha i starten, men logikken på disse er som regel at man kan velge a) dagens dato eller i fortid, eller b) dagens dato og i fremtid. Det blir i så fall veldig lange variabelnavn. Derfor gikk jeg for "ikke"-varianten 😅
src/frontend/utils/sanity.ts
Outdated
| ) as IFrittståendeOrdTekstinnhold, | ||
| dokumenterFiltrertPåType(fellesDokumenter, frittståendeOrdPrefix) | ||
| ), | ||
| formateringsfeilmeldinger: struktrerInnholdForFelles( |
There was a problem hiding this comment.
Vil du rette opp skrivefeilen som er i dette funksjonsnavnet når du først er her inne? Mangler en u 🫶 strukturerInnholdForFelles
There was a problem hiding this comment.
Jepp! Skal fikse det nå 😊
💰 Hva forsøker du å løse i denne PR'en
Favro: Ikke spørre om nøyaktig dato der vi kun trenger måned
Legger til en feature toggle som gjør at vi kun ber om måned der vi ikke trenger nøyaktig dato for å saksbehandle. Dette gjelder for arbeidsperioder, pensjonsperioder og utbetalingsperioder.
🔎️ Er det noe spesielt du ønsker å fremheve?
Er det noe du er bekymret eller usikker på? Beskriv det gjerne her.
Tenker å kjøre gjennom hele flyten i preprod inkl se på PDF i Gosys før vi legger til samme støtte for flere felter og til slutt aktiverer toggle i prod.
Det er viktig å teste validering på disse feltene. Det er disse casene som skal gjelde for feltene:
✅ Checklist
Har du husket alle punktene i listen?
Jeg har ikke skrevet tester fordi:
🤷♀ ️Hvor er det lurt å starte?
Jeg ville ha lest commit for commit
💬 Ønsker du en muntlig gjennomgang?
Jeg tror det kan være lurt med en muntlig gjennomgang her siden denne oppgaven er skikkelig tunga rett i munnen.
👀 Screen shots