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

Tar i bruk V9 av endepunkt for innsending av søknad #1340

Merged
merged 8 commits into from
Sep 3, 2024

Conversation

bragejahren
Copy link
Contributor

@bragejahren bragejahren commented Aug 29, 2024

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 added a commit to navikt/familie-baks-soknad-api that referenced this pull request Sep 2, 2024
…v adressebeskyttelse på alle barn (#283)

Favro:
[NAV-22010](https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-22010)

### 💰 Hva forsøker du å løse i denne PR'en
Per nå viser vi søkers adresse i PDF dersom søker selv ikke har
adressebeskyttelse. Dersom søker ikke har adressebeskyttelse men et av
barna det søkes for har det, viser vi fortsatt søkers adresse og
informasjon om at barna bor med søker. Dette er svært uheldig.

For å fikse dette har vi oppdatert søknadskontraktene til barnetrygd og
kontantstøtte med feltet `finnesPersonMedAdressebeskyttelse` som skal gi
HTML-generator informasjon om når vi kan vise adresse og når adresse
skal bli satt til "Skjermet adresse". Feltet utledes før
`baks-soknad-api` sender søknaden videre til `baks-mottak`, ved at vi
gjør oppslag på alle barn det søkes om mot PDL for å finne ut om noen
har adressebeskyttelse. Dersom søker eller et av barna har
adressebeskyttelse setter vi feltet `finnesPersonMedAdressebeskyttelse`
til `true`.

Relaterte endringer:
*
[familie-ba-soknad](navikt/familie-ba-soknad#1340)
*
[familie-ks-soknad](navikt/familie-ks-soknad#622)
*
[familie-baks-mottak](navikt/familie-baks-mottak#1150)

### ✅ Checklist
- [ ] Jeg har testet mine endringer i henhold til akseptansekriteriene
🕵️
- [x] Jeg har skrevet tester.


### 💬 Ønsker du en muntlig gjennomgang?
- [ ] Ja
- [x] Nei
@bragejahren bragejahren merged commit cc5f6c3 into main Sep 3, 2024
5 checks passed
@bragejahren bragejahren deleted the feat/søknad-endepunkt-v9 branch September 3, 2024 06:42
@bragejahren bragejahren restored the feat/søknad-endepunkt-v9 branch September 3, 2024 06:59
bragejahren added a commit that referenced this pull request Sep 3, 2024
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