-
Notifications
You must be signed in to change notification settings - Fork 0
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
Opprett sak og oppgave i samme river #723
Conversation
Testet OK. |
@@ -20,18 +22,28 @@ object NotifikasjonTekst { | |||
const val STATUS_TEKST_FERDIG = "Mottatt – Se kvittering eller korriger inntektsmelding" | |||
const val STATUS_TEKST_AVBRUTT = "Avbrutt av NAV" | |||
|
|||
fun lenkeFerdigstilt( | |||
fun lenkeAktiv( |
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.
Kanskje denne kan hete lenkeAktivForespoersel så det er likt pattern som de andre?
withData( | ||
mapOf( | ||
"melding med behov" to Pair(Key.BEHOV, BehovType.LAGRE_SELVBESTEMT_IM.toJson()), | ||
"melding med data" to Pair(Key.DATA, "".toJson()), |
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 gjelder bare for meldinger med data med tom verdi antar jeg?
Hva med "melding med data med tom verdi" feks?
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.
Jess, skal fikse. Det ble kopiert fra en river som avviser Key.DATA
blankt.
) | ||
|
||
val oppgaveId = | ||
agNotifikasjonKlient.opprettOppgave( |
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.
Kan vi havne i en situasjon der saken ble opprettet men ikke oppgaven? Vil vi da forsøke på nytt og lage en helt ny sak og en helt ny oppgave?
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.
Du er inne på noe, men pga. at feillytteren ikke tåler meldinger uten behov så vil ikke en feilet oppgaveopprettelse føre til at man prøver på ny. Da står vi uten oppgave, men siden det er sjeldent at kallene mot fager feiler så kan vi leve med risikoen i en liten stund, til vi fikser feillytteren. Før vi gjør det så må vi imidlertid tåle duplikater, som er hvorfor jeg roter inne i notifikasjonsklienten.
Vi har uansett fjernet den største synderen for at saker og oppgaver ikke blir opprettet, som var de gamle servicene.
sykmeldt = | ||
Person( | ||
fnr = Fnr.genererGyldig(), | ||
navn = "Peer Gynt", |
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.
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.
🤨
Vi trenger ikke lenger to rivere da vi ikke har to uavhengige prosesser for å hente data.
Fjerner også river for å slette sak, da vi kan løse det med ny klient mot Fager.