Gjenbruk barnetilsynsøknad steg 1 #1533
Conversation
…or å kunne fortsette hvis den ikke finner noe.
…nto hent-ut-samlivsbrudd-dato-ole
olekvernberg
requested review from
charliemidtlyng,
ViktorGSolberg,
ma10s,
throndi and
gunnsteingarmo
| // eslint-disable-next-line | ||
| }, []); | ||
|
|
||
| useEffect(() => { | ||
| if (toggles[ToggleName.hentBarnetilsynSøknad]) { |
There was a problem hiding this comment.
Hva skjer om denne trigges før fetching blir false? 🤔
Bør vi ha if(fetching === false && toggles && toggles[...]) ?
There was a problem hiding this comment.
Forrige søknad hentes ikke dersom toggles ikke er ferdig hentet, fordi toggle er undefined. Men useEffect blir gjort på nytt når fetching er ferdig og satt til false, så tenker det er nok?
Så hvertfall ut til å funke i preprod
There was a problem hiding this comment.
hvis toggles er undefined så krasjer jo appen med Cannot read properties of undefined ?
There was a problem hiding this comment.
Glem det - toggles er aldri undefined - den er jo {} tomt objekt :)
|
|
||
| const settMedlemskap = (medlemskap: IMedlemskap) => { | ||
| settSøknad((prevSoknad: ISøknad) => { | ||
| return { | ||
| ...prevSoknad, | ||
| medlemskap: medlemskap, | ||
| medlemskap: | ||
| Object.keys(medlemskap).length !== 0 |
There was a problem hiding this comment.
Den vil inneholde noe når bruker velger et alternativ på medlemskap spørsmålene. Ved gjenbruk skal svar fra forrige søknad settes, men bruker overstyrer så fort noen alternativer på medlemskap velges.
| søkerBorPåRegistrertAdresseEllerHarMeldtAdresseendring: boolean | ||
| ): boolean => { | ||
| return !!( | ||
| søkerBorPåRegistrertAdresseEllerHarMeldtAdresseendring && |
There was a problem hiding this comment.
Har dere testet alle permutasjonene her? :)
There was a problem hiding this comment.
Vi har testet grundig på barnetilsyn med featuretoggle slått av, men ser for oss at vi kommer til å teste litt til på overgangsstønad.
src/utils/søknad.ts
Outdated
| return response.data; | ||
| } catch (error) { | ||
| console.error('Feil med å hente data fra forrige søknad: ', error); | ||
| throw error; |
There was a problem hiding this comment.
Håndterer vi denne noe sted?
There was a problem hiding this comment.
Vi dropper det i første omgang. Men vi ønsker å vise en feilmelding eller noe i fremtiden - når vi legger til det at søker skal gjøre et valg om de ønsker å gjenbruke tidligere søknad.
|
Testet alle permutasjoner av sivilstand. Ser bra ut! |
Hvorfor er denne endringen nødvendig? ✨
Gjenbruk barnetilsynsøknad: Første steg med featuretoggle som skal være skrudd av i prod til alle stegene er implementert.
Det er ikke lagt inn noe varsel eller lignende om verdier som blir gjenbrukt.
Data hentes fra mottak via soknad-api. Det er opprettet PR i mottak og soknad-api.
Skal squashe før merge.
Det er tegnet opp hvilke spørsmål som blir vist ut ifra hvilken sivilstand bruker har:
Favro