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

Feat/søknad endepunkt v9 #1348

Merged
merged 12 commits into from
Sep 3, 2024
Merged

Feat/søknad endepunkt v9 #1348

merged 12 commits into from
Sep 3, 2024

Conversation

bragejahren
Copy link
Contributor

Favro: NAV-22010

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

Kontrakt for søknad er oppdatert og vi må nå gå mot nytt endepunkt i familie-baks-soknad-api.

  • Endrer kontraktVersjon til 9.
  • Fjerner versjon (V8) fra alle type-navn og filnavn da vi aldri vil operere med flere versjoner av søknaden i frontend.
  • Legger til teksten skjermetAdresse i teksterUtenomSpørsmål, slik at den kan brukes i PDF.
  • Støtte for å legge til sanity-tekster i søknads-kontrakt før innsending.

Endrer ikke modellversjon da data fra søknaden er lik som tidligere. Eneste forskjellen er at vi nå legger ved teksten skjermetAdresse i teksterUtenomSpørsmål ved innsending. Når vi ikke endrer på modellversjon vil man kunne fortsette på søknaden dersom man hadde begynt før denne endringen blir rullet ut.

✅ Checklist

  • 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.
  • 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:
Ingen nye logikk-endringer

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

@bragejahren bragejahren merged commit c5b2186 into main Sep 3, 2024
5 checks passed
@bragejahren bragejahren deleted the feat/søknad-endepunkt-v9 branch September 3, 2024 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants