Conversation
…perioder ikke er like.
Rendre avsnitt på nytt ved sammenslåing av perioder/angring
…ioder for å unngå å gjøre mange unødvendige kall.
gunnsteingarmo
requested review from
charliemidtlyng,
ViktorGSolberg,
olekvernberg,
ma10s and
throndi
…rPerioderSlaattSammen. Bruker respons fra hentSjekkLikhetPerioder for å sette verdi på state.
…ariabler for mer konsekvent navngivning.
| }, [hentSjekkLikhetPerioder]); | ||
| const fetch = async () => { | ||
| const likhetResult = await hentSjekkLikhetPerioder(); | ||
| settErPerioderSammenslått(likhetResult); |
There was a problem hiding this comment.
Burde vi ikke hente skal_sammenslaa_perioder verdien fra backend her?
There was a problem hiding this comment.
Vi trenger sjekken for å avgjøre om knappen skal vises. Vi må sjekke på hentSjekkLikhetPerioder hvis vi angrer sammenslåing og deretter ønsker å slå sammen igjen.
There was a problem hiding this comment.
Okay, men da lurer jeg på dette scenariet.
Jeg kommer til brevsiden og "angrer sammenslåing".
Jeg lukker behandlingen og åpner den på nytt senere.
Da vil jo "erPerioderLike" være true, men jeg har valgt å splitte periodene allikevel.
Mulig det er noe jeg ikke forstår, men slik jeg leser flyten så blir det i frontend seende ut som vi har slått sammen periodene, mens det i backend ikke er gjort dette.
There was a problem hiding this comment.
Du har rett - som vanlig!😄 Har laget et endepunkt i backend for å hente verdien, denne blir brukt for å sette korrekt state og vise riktig tekst på knappen.
…lik at knapper ikke kan trykkes på mange ganger, som kan føre til feil.
…n state og tekst som skal settes. Denne ble feil hvis man lukket nettleservinduet og åpnet behandlingen igjen.
…' dersom 'Ja' velges på 'Skal særlige grunner gi reduksjon av beløpet'. Skal vi verdie på 'Skal det tillegges renter' i stedet for hardkotet tekst.
Favro
Default hvis perioder er like og derfor har blitt slått sammen:
Etter å ha angret sammenslåing:
PDF før:
PDF sammenslått:
