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

IS-2752: Add endpoint for historikk #182

Merged
merged 2 commits into from
Oct 28, 2024
Merged

Conversation

andersrognstad
Copy link
Contributor

Trenger nytt endepunkt for å hente dialogmøtekandidat-historikk (inkl unntak og ikke aktuell-vurderinger) for en person slik at dette kan vises på Historikk-siden i modia.

@andersrognstad andersrognstad requested a review from a team as a code owner October 25, 2024 08:08
)

// Disse dekkes av dialogmote-historikk og unntak/ikke-aktuell-historikk
DialogmotekandidatEndringArsak.DIALOGMOTE_FERDIGSTILT, DialogmotekandidatEndringArsak.DIALOGMOTE_LUKKET, DialogmotekandidatEndringArsak.UNNTAK, DialogmotekandidatEndringArsak.IKKE_AKTUELL -> null
Copy link
Contributor Author

Choose a reason for hiding this comment

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

DIALOGMOTE_FERDIGSTILT og DIALOGMOTE_LUKKET trenger vi ikke ta med her siden de vil dekkes av historikken på dialogmøter. Eller tenker dere at de bør tas med i APIet for kompletthetens skyld?

Copy link
Contributor Author

@andersrognstad andersrognstad Oct 25, 2024

Choose a reason for hiding this comment

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

UNNTAK og IKKE_AKTUELL er også "kandidat-endringer" siden de fører til at personen ikke er kandidat, men vi lagrer selve vurderingen i egne tabeller som mappes under (hvor vi også har data på veileder som gjorde vurderingen)

Copy link
Contributor

@eirikdahlen eirikdahlen Oct 28, 2024

Choose a reason for hiding this comment

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

Blir kanskje litt på siden, men hvorfor vet isdialogmotekandidat om møtet er ferdigstilt og sånt? Det er jo isdialogmote-domene?

Litt usikker på kompletthet, hvis det egt ikke hører hjemme her så trengs det ikke i apiet.

Copy link
Contributor

Choose a reason for hiding this comment

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

Slik jeg leser koden er en arbeidstaker dialogmøtekandidat helt til et dialogmøte er ferdigstilt eller lukket (med mindre det settes unntak).

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah okey, ja det gir mening at endringene trigger noe. Tenker da at siden endringene egentlig hører hjemme i isdialogmote så trenger ikke APIet å returnere de her?

Copy link
Contributor Author

@andersrognstad andersrognstad Oct 28, 2024

Choose a reason for hiding this comment

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

Ja, stemmer det @geir-waagboe sier, det er derfor vi må vite om det her. Tror vi da er enige om at vi ikke tar med de endringene i APIet 👍

val ikkeAktuell = ikkeAktuellService.getIkkeAktuellList(
personIdent = personIdent,
)
val historikkDTOS = HistorikkDTO.createHistorikkDTOs(
Copy link
Contributor

Choose a reason for hiding this comment

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

historikkDTOS -> historikkDTOs?

@@ -54,5 +59,32 @@ fun Route.registerDialogmotekandidatApi(
call.respond(kandidatDTO)
}
}
get(kandidatApiHistorikkPath) {
val personIdent = personIdentHeader()?.let { personIdent ->
Copy link
Contributor

Choose a reason for hiding this comment

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

personIdent -> personident?

Copy link
Contributor

@geir-waagboe geir-waagboe left a comment

Choose a reason for hiding this comment

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

Ser bra ut!

@andersrognstad andersrognstad merged commit a8ef807 into master Oct 28, 2024
3 checks passed
@andersrognstad andersrognstad deleted the IS-2752-historikk branch October 28, 2024 13:41
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