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

EY-4963 Støtte opprettelse av oppfølgingsoppaver for aktivitetsplikt … #6995

Merged
merged 12 commits into from
Feb 17, 2025

Conversation

sebassonav
Copy link
Contributor

@sebassonav sebassonav commented Feb 11, 2025

…i backend

@sebassonav sebassonav marked this pull request as ready for review February 13, 2025 22:01
@sebassonav sebassonav requested a review from a team as a code owner February 13, 2025 22:01
val oppgave = inTransaction { aktivitetspliktOppgaveService.ferdigstillBrevOgOppgave(oppgaveId, brukerTokenInfo) }
call.respond(oppgave)
route("/api/aktivitetsplikt/oppgave") {
post("/opprett") {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@oyvindsh skal vi kalle denne fra de automatiske jobbene også?

Copy link
Contributor

Choose a reason for hiding this comment

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

Hmm, kanskje? Jobben må i så fall håndtere å få forskjellige forespoerselexceptions tilbake forskjellig fra internfeil, slik at den gir seg hvis den ikke skal opprette og prøver på nytt (kræsjlooper) hvis det er noe feil med behandling. Kanskje det letteste er å ha to endepunkt og så kan sjekkene hentes ut i en felles metode, og så kan oppgavene for de automatiske bare fange exceptions i service der, og saksbehandler-kallet sender de ut.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

se gjerne på #7044
merger de samtidig tenker jeg med denne først
og deretter den ovennevnte så sperrer på brev

Copy link
Contributor

@oyvindsh oyvindsh left a comment

Choose a reason for hiding this comment

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

Nice!

Vi burde ha noen sjekker i de automatiske opprettede oppgavene også, slik at de ikke lager flere under arbeid / lager nye hvis de allerede er ferdigstilt

@sebassonav
Copy link
Contributor Author

Sees på med #7045

@sebassonav sebassonav merged commit a94362d into main Feb 17, 2025
11 checks passed
@sebassonav sebassonav deleted the feature/EY-4963-opprette-aks-flyt-manuelt branch February 17, 2025 11:25
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.

2 participants