Job for å prosessere klagevedtak fra KA #631
Conversation
| Klagevedtak.Utfall.AVVIST, | ||
| -> { | ||
| log.error("Utfall: ${klagevedtak.utfall} fra Klageinstans er ikke håndtert.") | ||
| klagevedtakRepo.markerSomFeil(klagevedtak.id) |
There was a problem hiding this comment.
Kan legge til en kommentar her om at tanken er at kjente FEIL kan settes tilbake til UPROSESSERT manuelt av oss i databasen etter vi har lagt til ny logikk som støtter disse tilfellene. Alternativt kan man ha en db-migreringsjobb sammen med kodeendringen.
service/src/main/kotlin/no/nav/su/se/bakover/service/klage/KlagevedtakServiceImpl.kt
Outdated
Show resolved
Hide resolved
| @@ -113,6 +113,10 @@ internal class OppgaveHttpClient( | |||
| "--- ${ | |||
| Tidspunkt.now(clock).toOppgaveFormat() | |||
| } - Opprettet av Supplerende Stønad ---\nSaksnummer : ${config.saksreferanse}" | |||
| is OppgaveConfig.Klage.Vedtak -> | |||
There was a problem hiding this comment.
| is OppgaveConfig.Klage.Vedtak -> | |
| is OppgaveConfig.Klageinstans.Vedtak -> |
There was a problem hiding this comment.
Da blir det också OppgaveConfig.Klageinstans.Saksbehandler og OppgaveConfig.Klageinstans.Attestering for saksbehandlings & attesteringsoppgaver for klagene som behandles. Blir det ikke rart? Klageinstansen er ikke involvert i de.
Kanske kunde endre navnet fra Vedtak til VedtakFraKlageinstans eller noe istedet?
There was a problem hiding this comment.
Hmm. Hvorfor trenger vi .Attestering? Kan ikke attesteringsoppgaven heller gå inn i den vanlige flyten vår.
There was a problem hiding this comment.
Per nå så ser vår oppgave-config klasse såhär for Klage:
sealed class Klage(..) {
data class Saksbehandler(...): Klage()
data class Attestering(...): Klage()
data class Vedtak(...): Klage()
}
Hvis vi endrer Klage til Klageinstans så påverkas alla de tre. Men kanske jag kunne trekt ut Vedtak og lagt det som en egen liksom? Är de det du tänker? 😊
client/src/main/kotlin/no/nav/su/se/bakover/client/oppgave/OppgavebeskrivelseMapper.kt
Outdated
Show resolved
Hide resolved
client/src/main/kotlin/no/nav/su/se/bakover/client/oppgave/OppgavebeskrivelseMapper.kt
Outdated
Show resolved
Hide resolved
client/src/main/kotlin/no/nav/su/se/bakover/client/oppgave/OppgavebeskrivelseMapper.kt
Outdated
Show resolved
Hide resolved
client/src/main/kotlin/no/nav/su/se/bakover/client/oppgave/OppgavebeskrivelseMapper.kt
Outdated
Show resolved
Hide resolved
client/src/main/kotlin/no/nav/su/se/bakover/client/oppgave/OppgavebeskrivelseMapper.kt
Outdated
Show resolved
Hide resolved
| data class Handling( | ||
| override val saksnummer: Saksnummer, | ||
| override val aktørId: AktørId, | ||
| override val journalpostId: JournalpostId, |
There was a problem hiding this comment.
Tror denne er optional fra Klageinstansen. aka. nullable
There was a problem hiding this comment.
Den står som required i deras schema.
Ref: https://github.com/navikt/kabal-api/blob/main/docs/schema/klagevedtak-fattet.json (vedtaksbrevReferanse)
| KlagevedtakUtfall.UGUNST -> OppgaveConfig.Klage.Vedtak.Handling( | ||
| saksnummer = klage.saksnummer, | ||
| aktørId = aktørId, | ||
| journalpostId = JournalpostId(klagevedtak.vedtaksbrevReferanse), |
There was a problem hiding this comment.
vedtaksbrevReferanse burde være nullable. Hvor i overgangen vil dette smelle? Kanskje legge inn en test som har vedtaksbrevReferanse som null 😁
Co-authored-by: John Andre Hestad <[email protected]>
No description provided.