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

Send varselbrev dersom bruker har 10% eller mer endring i inntekt #2765

Merged
merged 11 commits into from
Feb 4, 2025

Conversation

olekvernberg
Copy link
Contributor

@olekvernberg olekvernberg commented Feb 2, 2025

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.

Screenshot 2025-02-02 at 10 30 20

Copy link
Contributor

@charliemidtlyng charliemidtlyng left a 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?

Copy link
Contributor

@ViktorGSolberg ViktorGSolberg left a 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 ✅

@olekvernberg
Copy link
Contributor Author

olekvernberg commented Feb 3, 2025

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?

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?
I såfall ser jeg for meg å lagre ned oppgaveid, fagsakid, tidspunkt for kjøring og utfallet av kjøringen.

EDIT: Tar det i egen PR

@olekvernberg olekvernberg merged commit 28d828a into main Feb 4, 2025
2 checks passed
@olekvernberg olekvernberg deleted the send-varselbrev branch February 4, 2025 15:13
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.

4 participants