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

Opprett sak og oppgave i samme river #723

Merged
merged 2 commits into from
Oct 23, 2024

Conversation

bjerga
Copy link
Contributor

@bjerga bjerga commented Oct 23, 2024

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.

@bjerga bjerga requested a review from a team as a code owner October 23, 2024 13:09
@bjerga
Copy link
Contributor Author

bjerga commented Oct 23, 2024

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(
Copy link
Contributor

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()),
Copy link
Contributor

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?

Copy link
Contributor Author

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(
Copy link
Contributor

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?

Copy link
Contributor Author

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",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤨

@bjerga bjerga merged commit d30da7b into main Oct 23, 2024
21 checks passed
@bjerga bjerga deleted the dev/opprett-sak-og-oppgave-samme-river branch October 23, 2024 14:47
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.

3 participants