-
Notifications
You must be signed in to change notification settings - Fork 2
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
Send varselbrev dersom bruker har 10% eller mer endring i inntekt #2765
Conversation
…drivende må teste
src/main/kotlin/no/nav/familie/ef/sak/selvstendig/NæringsinntektKontrollBrev.kt
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Persisterer vi noe ned i vår database som tilsier at vi kan "finne igjen" de vi har prosessert i denne jobben?
Hvis nei - burde vi det?
src/main/kotlin/no/nav/familie/ef/sak/selvstendig/NæringsinntektKontrollBrev.kt
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dette ser bra ut, hadde vært fint med en test ✅
Ikke annet enn i tasks som blir borte etter hvert. Det er kanskje greit med en historikk, slik at man ser utfallet av kjøringen lenger tilbake i tid? EDIT: Tar det i egen PR |
…n skal sette tittel manuelt. Lagt til enhetstest som tester at brev som skal sendes av iverksett inneholder riktige verdier.
src/main/kotlin/no/nav/familie/ef/sak/selvstendig/NæringsinntektKontrollService.kt
Outdated
Show resolved
Hide resolved
… avklaringer og testing når inntekt for selvstendig næringsdrivende er tilgjengelige.
Hvorfor er denne endringen nødvendig? ✨
Sender varselbrev dersom bruker har 10% endring eller mer. Etter diskusjon med fag, vil det avhenge av testing og videre avklaringer om dette vil være verdt å ta i bruk. Brevmalen inneholder mange detaljer som gjør det mer tidkrevende å lage en mal som vil være generell nok til å passe alle. Dagens mal for de med 10% endring i inntekt sier f.eks. noe om hvilken inntekt som ble lagt til grunn i nyeste revurdering, enten om det er g-omregning eller en vanlig revurdering. I tillegg til informasjon om hvor personinntekt kommer fra, som navn på arbeidsgiver, periode og beløp. Brevmalen under ble bare brukt som utgangspunkt for å teste at brevet virker med flettefelt (kun forventetInntekt ble lagd som flettefelt).
Fag avklarer om brevet kan endres og forkortes slik at det ikke blir nødvendig med så mye informasjon. Disse avklaringene i kombinasjon med testing i høst hvor man kan få en indikasjon på hvor mange dette vil gjelde, vil avgjøre om det er verdt å sende et brev automatisk, eller om man heller bare oppdaterer oppgaven med relevant data som er funnet, slik at saksbehandler kan enklere sende et brev selv.
Det er opprettet favro-kort for hva som må gjøres når det skal testes nærmere høsten, hvor forhåpentligvis tallene for selvstendig næringsdrivende er tilgjengelige: https://favro.com/organization/98c34fb974ce445eac854de0/a64c6aad9b0d61ef6c0290bd?card=NAV-24146
Det er også opprettet en enhetstest som vil feile 1. oktober for å gi en påminnelse om at dette må testes før selve kjøringen i prod.