Doedsfall hendelser #710
Doedsfall hendelser #710
Conversation
…ed tilsvarende plugin utviklet av en utvikler i NAV
…tomatisk blir generert avro-filer
… i en vedtaksperiode
…i databasen med referanse til tasken som har blitt opprettet
| @@ -29,7 +29,7 @@ plugins { | |||
| id("io.spring.dependency-management") version "1.1.7" | |||
| kotlin("plugin.spring") version "2.1.20" | |||
|
|
|||
| id("com.github.davidmc24.gradle.plugin.avro") version "1.9.1" | |||
| id("io.github.androa.gradle.plugin.avro") version "0.0.12" | |||
There was a problem hiding this comment.
Gammel plugin blir ikke oppdatert lenger (kan forøvrig ikke se den var i bruk heller)
There was a problem hiding this comment.
Den ble tatt inn når vi skulle lytte på journalhendelser men så gikk vi for å bruke dependency
implementation("no.nav.teamdokumenthandtering:teamdokumenthandtering-avro-schemas:$joarkHendelseVersion")
Vi tenkte det var en fordel om man fikk dependabot-varsel om endringer i skjeamaet
Men jeg glømte bort å fjernecom.github.davidmc24.gradle.plugin.avro 😬
Publiserer pdl en pakke for disse hendelsene? Ulempen med det er ju at man trekker inn en del kode man ev ikke har bruk for
There was a problem hiding this comment.
Jeg prøvde å se etter det før jeg dro inn avro-filene, men nei de publiserer ikke noe selv.
Men ja, det legger jo til noe kompleksitet her. Er jo kanskje best å dytte dette ut i tilleggsstonader-kontrakter? Kjipeste med å bygge java-klassene i repoet er at man må bygge prosjektet før det kompilerer. Samme om man kjører en gradle clean, så må man bygge det igjen 😅
There was a problem hiding this comment.
Hmm ja, av den grunnen er det kanskje fint å trekke det ut i kontrakter, då får man det nog til å fungere ut av boksen
AStrand94
force-pushed
the
doedsfall-hendelser
branch
from
fcc24bb to
cdc9d58
Compare
# Conflicts: # .github/workflows/codeql.yml
| @@ -29,7 +29,7 @@ plugins { | |||
| id("io.spring.dependency-management") version "1.1.7" | |||
| kotlin("plugin.spring") version "2.1.20" | |||
|
|
|||
| id("com.github.davidmc24.gradle.plugin.avro") version "1.9.1" | |||
| id("io.github.androa.gradle.plugin.avro") version "0.0.12" | |||
There was a problem hiding this comment.
Den ble tatt inn når vi skulle lytte på journalhendelser men så gikk vi for å bruke dependency
implementation("no.nav.teamdokumenthandtering:teamdokumenthandtering-avro-schemas:$joarkHendelseVersion")
Vi tenkte det var en fordel om man fikk dependabot-varsel om endringer i skjeamaet
Men jeg glømte bort å fjernecom.github.davidmc24.gradle.plugin.avro 😬
Publiserer pdl en pakke for disse hendelsene? Ulempen med det er ju at man trekker inn en del kode man ev ikke har bruk for
| erAnnullering(), | ||
| ) | ||
|
|
||
| private fun Personhendelse.erAnnullering() = this.endringstype == Endringstype.ANNULLERT |
There was a problem hiding this comment.
Må vi håndtere "ENDRING" på en eller annen måte? 👀
There was a problem hiding this comment.
Ah, ser vi må håndtere korrigering og ja https://pdl-docs.ansatt.nav.no/ekstern/index.html#_d%C3%B8dsfall
En mulighet kan bare være å ignorere det, for vi sender ikke med dødsdato i oppgaven per nå 👀
...in/kotlin/no/nav/tilleggsstonader/sak/hendelser/personhendelse/dødsfall/DødsfallHåndterer.kt
Outdated
Show resolved
Hide resolved
...in/kotlin/no/nav/tilleggsstonader/sak/hendelser/personhendelse/dødsfall/DødsfallHåndterer.kt
Outdated
Show resolved
Hide resolved
|
|
||
| val person = personService.hentPersonUtenBarn(dødsfallHendelse.personidenter.first()) | ||
|
|
||
| if (personErFortsattDød(person.søker)) { |
There was a problem hiding this comment.
Her kan det ev. kommit en korrigering på dødsdato. Burde man hente dødsdato fra PDL i stedet for hendelsen?
Burde vi sjekke for at personen fortsatt har aktivt vedtak? Eks hvis en saksbehandler plukket opp at personen er død et annet sted før vi opprettet oppgaven.
Jeg tenker nødvendigvis ikke det, hvis vi oppretter en ekstra oppgave om dette er det ikke noe krise - kommer ikke skje ofte
There was a problem hiding this comment.
Ja jeg tenker og at det ikke er krise med en ekstra oppgave. Sannsynligheten for at det skal skje er vel fort ganske liten, så den ekstra kompleksiteten er kanskje ikke verdt å ta høyde for 🤷
| oppgave = | ||
| OpprettOppgave( | ||
| oppgavetype = Oppgavetype.VurderLivshendelse, | ||
| beskrivelse = data.beskrivelse ?: "Person død, vurder aktive stønader", |
...in/kotlin/no/nav/tilleggsstonader/sak/hendelser/personhendelse/dødsfall/DødsfallHåndterer.kt
Outdated
Show resolved
Hide resolved
...n/kotlin/no/nav/tilleggsstonader/sak/hendelser/personhendelse/PersonhendelseKafkaListener.kt
Outdated
Show resolved
Hide resolved
…sisterende beskrivelse i gosys-stil
…for å sette eksplisitt id på listenercontainer
Hvorfor er denne endringen nødvendig? ✨
Legger til at vi lytter på personhendelser fra leesah (liver er en strøm av hendelser)-topicet, og filtrerer ut dødshendelser. Dette slik at vi kan plukke opp og varsle saksbehandler om bruker har aktive stønader. Varslet i form av en oppgave med type
VurderLivshendelse. Oppretter en task som kjører og oppretter oppgave 1 uke etter vi har mottatt dødshendelsen. Tasken sjekker om personen fortsatt er død.Må også håndtere annulleringer av dødsfall. Så utfall av lytting på topic kan bli:
Leesah-topicet har veldig mange forskjellige typer personhendelser, og har da altså ganske mye data. Leser her batchvis (journalpost-hendelser topic leser vi enkeltvis), med 500 records om gangen.