-
Notifications
You must be signed in to change notification settings - Fork 4
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
Lagt til støtte for å kopiere journalpost #1117
Merged
Merged
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
# Conflicts: # src/main/kotlin/no/nav/k9punsj/integrasjoner/dokarkiv/SafDtos.kt # src/main/kotlin/no/nav/k9punsj/journalpost/JournalpostRoutes.kt # src/main/kotlin/no/nav/k9punsj/journalpost/JournalpostService.kt # src/test/kotlin/no/nav/k9punsj/journalpost/KopierJournalpostRouteTest.kt
# Conflicts: # src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/LokalK9SakService.kt # src/test/kotlin/no/nav/k9punsj/rest/eksternt/k9sak/TestK9SakService.kt
espenjv
reviewed
Sep 17, 2024
src/main/kotlin/no/nav/k9punsj/integrasjoner/dokarkiv/DokarkivGateway.kt
Show resolved
Hide resolved
src/main/kotlin/no/nav/k9punsj/integrasjoner/dokarkiv/DokarkivGateway.kt
Show resolved
Hide resolved
src/main/kotlin/no/nav/k9punsj/integrasjoner/dokarkiv/SafDtos.kt
Outdated
Show resolved
Hide resolved
src/main/kotlin/no/nav/k9punsj/integrasjoner/dokarkiv/SafDtos.kt
Outdated
Show resolved
Hide resolved
espenjv
reviewed
Sep 17, 2024
src/main/kotlin/no/nav/k9punsj/journalpost/JournalpostkopieringService.kt
Outdated
Show resolved
Hide resolved
espenjv
approved these changes
Sep 17, 2024
ramrock93
added a commit
that referenced
this pull request
Sep 23, 2024
Lagt til støtte for å kopiere journalpost (#1117) * WIP Støtte for å kopiere journalpost * Testjobb * Fikser kopiering av journalpost * Refaktorerer ut krav for kopiering. * Fjerner ubrukt felt. * Fjerner rapid og ubrukte klasser. * Fikser circular reference. * Lagrer journalpostkopi gjennom hendelseMottaker. * Etterspørr datoOpprettet ved kall mot SAF. * Legger på TODO... * Fjerner konsumering fra punsjbolle topic. * Fikser feilhåndtering ved kopiering av journalpost. * Rydder opp kode * Bruker enum for string verdier. * Fjerner ubrukt variabel. * Bruker enum for string verdier. * Forenkler logikk. * Returnerer mer detaljert respons ved kopiering av JP.
ramrock93
added a commit
that referenced
this pull request
Sep 27, 2024
) * WIP Støtte for å kopiere journalpost * Testjobb * Fikser kopiering av journalpost * Refaktorerer ut krav for kopiering. * Fjerner ubrukt felt. * Fjerner rapid og ubrukte klasser. * Fikser circular reference. * Lagrer journalpostkopi gjennom hendelseMottaker. * Etterspørr datoOpprettet ved kall mot SAF. * Legger på TODO... * Fjerner konsumering fra punsjbolle topic. * Fikser feilhåndtering ved kopiering av journalpost. * Rydder opp kode * Bruker enum for string verdier. * Fjerner ubrukt variabel. * Bruker enum for string verdier. * Forenkler logikk. * Returnerer mer detaljert respons ved kopiering av JP. * Tilrettelegger for kopiering av JP for alle ytelser. * Lagt til støtte for å kopiere journalpost (#1117) * WIP Støtte for å kopiere journalpost * Testjobb * Fikser kopiering av journalpost * Refaktorerer ut krav for kopiering. * Fjerner ubrukt felt. * Fjerner rapid og ubrukte klasser. * Fikser circular reference. * Lagrer journalpostkopi gjennom hendelseMottaker. * Etterspørr datoOpprettet ved kall mot SAF. * Legger på TODO... * Fjerner konsumering fra punsjbolle topic. * Fikser feilhåndtering ved kopiering av journalpost. * Rydder opp kode * Bruker enum for string verdier. * Fjerner ubrukt variabel. * Bruker enum for string verdier. * Forenkler logikk. * Returnerer mer detaljert respons ved kopiering av JP. --------- Co-authored-by: Ramin Esfandiari <[email protected]> Co-authored-by: Ramin Esfandiari <[email protected]> * Fjerner `fra` felt i kopiering som fører forvirring. Det er kun personen vi kopierer til som er viktig. til person kan være samme person som er registrert på orginal eller en annen person som det skal kopieres til. * Sjekker null eller tom * Legger til støtte for resterende ytelser for kopiering --------- Co-authored-by: albrektsson <[email protected]> Co-authored-by: Emil Albrektsson <[email protected]>
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.
Ved flytte denne funksjonaliteten inn til punsj vil det også gir oss muligheten til å fange opp den kopierte journalpostens nye id, som gjør at vi kan navigere saksbehandler rett til den i punsj, i stedenfor for å søke opp søkeren på nytt i LOS, og prøve å finne den nye journalposten.
I tillegg fjerner dette vår avhengighet til
k9-punsjbolle
ogRapid&Rivers
, og gir oss bedre mulighet til feilhåndtering og en tjeneste mindre å forvalte.BREAKING CHANGE (håndteres av frontend nå)
POST /journalpost/kopier/{journalpost_id}
returnerer nå201 CREATED
(tidligere202 ACCEPTED
) med response som inneholder id til den nye journalposten.