Fikser erFosterbarn-feil vi sporadisk ser i Sentry

[hanneolsen]

Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-24631

💰 Hva forsøker du å løse i denne PR'en

Vi har med jevne mellomrom sett feil slik som denne i Sentry: Cannot read properties of undefined (reading 'erFosterbarn'). Frem til nå har vi ikke klart å finne ut hvorfor dette skjer.

Når vi oppretter skjema med checkboxer for barn antar vi at disse barna har blitt valgt og satt in memory. Vi bruker denne dataen til å sette riktig label på checkboxen i det relevante skjemaet. Problemet er når denne dataen ikke er satt.

Hvordan reprodusere feilen

Måten å reprodusere feilen på er å starte en ny søknad og fylle den ut inntil minst /om/barn/1. Så må man slette søknaden og starte en ny søknad. Så går man tilbake i nettleseren helt til man kommer til /om-barna. Der vil det nå feile siden dataen om hvilke barn ikke er satt.

Fiks

Legger skjemaet for et barn i egen komponent, slik at vi kan sjekke om all data finnes og eventuell redirecte hvis ikke fra en useEffect. Hooken useOmBarnaDine fra OmBarna-skjemaet blir da kalt i riktig rekkefølge etter at sjekken og eventuell redirect har blitt gjort.

🔎️ Er det noe spesielt du ønsker å fremheve?

Ingenting spesielt

✅ Checklist

Har du husket alle punktene i listen?

• Jeg har testet mine endringer i henhold til akseptansekriteriene/skissene 🕵️
• Jeg har testet endringene mine i mobilstørrelse, zoom 200%, skalerer riktig med endret tekststørrelse i browser 📱
• Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇
• Jeg har fikset en bug, og skrevet regresjonstest for denne
• Jeg har endret søknadskontrakten og modellversjon i Miljø.ts

Jeg har ikke skrevet tester fordi:
Ikke veldig relevant

🤷‍♀ ️Hvor er det lurt å starte?

Alt i ett

💬 Ønsker du en muntlig gjennomgang?

• Ja
• Nei

👀 Screen shots

Ingen visuelle endringer

[bragejahren]

Trenger man å bruke useEffect her? Ville det fungert å bare bruke

if (manglerData) {
    navigate('/velg-barn')
}

[hanneolsen]

Nei dessverre, det har sannsynligvis endret seg i nyere versjonen av react-router. Jeg prøvde det først, men da fungerte redirecten kun spontant og jeg fikk melding i konsollet om at navigate burde ligge inni en useEffect. :/ Jeg prøvde å finne noe offisiell dokumentasjon på kravet nå men fant ingenting, noe som er litt rart. Men av testingen min av koden og av warningen jeg fikk i konsollet burde denne ligge inni en useEffect.

[bragejahren]

Ser bra ut! ✅ La inn et lite spørsmål 😊