Saksnummer #793
Merged
Saksnummer #793
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
hestad
reviewed
Jan 31, 2025
app/src/main/kotlin/no/nav/tiltakspenger/vedtak/repository/søknad/SøknadDAO.kt
Outdated
Show resolved
Hide resolved
hestad
reviewed
Jan 31, 2025
app/src/main/kotlin/no/nav/tiltakspenger/vedtak/routes/behandling/benk/BehandlingBenkRoutes.kt
Show resolved
Hide resolved
hestad
reviewed
Jan 31, 2025
| post(SØKNAD_PATH) { | ||
| logger.debug { "Mottatt ny søknad på '$SØKNAD_PATH' - Prøver deserialisere og lagre." } | ||
| call.withSystembruker(tokenService = tokenService) { systembruker: Systembruker -> | ||
| val søknadDTO = call.receive<SøknadDTO>() | ||
| logger.debug { "Deserialisert søknad OK med id ${søknadDTO.søknadId}" } | ||
| // Oppretter sak med søknad og lagrer den | ||
| val sakId = sakService.hentForSaksnummer(Saksnummer(søknadDTO.saksnummer), systembruker).id |
There was a problem hiding this comment.
Blir det loop hvis vi flytter dette kallet inn i SøknadService? Jeg synes det er ryddigest og bare gjøre et service kall.
There was a problem hiding this comment.
Etter at jeg gjorde saksnummer og sakid til en del av søknaden må jeg ha saken for å kunne opprette søknaden :/ Da må jeg i så fall dra med meg dto-formatet inn i søknadservicen og gjøre mappingen der.
hestad
reviewed
Jan 31, 2025
.../main/kotlin/no/nav/tiltakspenger/saksbehandling/service/behandling/BehandlingServiceImpl.kt
Outdated
Show resolved
Hide resolved
hestad
approved these changes
Jan 31, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://trello.com/c/5ZSIIqvj/1290-for-%C3%A5-kunne-journalf%C3%B8re-s%C3%B8knader-automatisk-m%C3%A5-s%C3%B8knad-api-kunne-hente-saksnummer-fra-saksbehandling-api-f%C3%B8r-s%C3%B8knaden-blir-sendt
Vi trenger å kunne opprette saksnummer før vi mottar selve søknaden. Det har fått ganske mange konsekvenser. Har sjekket at ting fungerer både i testene (det var overraskende mange! 😅) og at ting fungerer når jeg kjører opp dette lokalt. Det er sikkert lurt å se litt grundig på dette likevel, for her er det ganske sannsynlig at jeg kan ha gjort noe som ødelegger noe.