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

Nav 23319 ingen kravgrunnlag #1599

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Conversation

amiraliw
Copy link
Contributor

@amiraliw amiraliw commented Jan 15, 2025

Løsningen er en batch som kjøres hver dag kl.06. Batchen henter alle behandlinger som er eldre enn 8 uker og har ingen kravgrunnlag

SELECT beh.* FROM behandling beh LEFT JOIN Kravgrunnlag431 k431 ON beh.id = k431.behandling_id WHERE k431.id IS NULL AND beh.status != 'AVSLUTTET' AND beh.opprettet_dato < CURRENT_TIMESTAMP - INTERVAL '8 weeks'

Batchen sjekker videre om et brev ble sendt ved opprettelse i behandlingen. Hvis et brev var sendt så opprettes en oppgaveTask hos fagsystemet slik at de henlegger behandlingen manuelt og ev. sende et brev på det.
Hvis ingen brev ble sendt, henlegges behandlingen automatisk ved å kjøre behandlingService.henleggBehandling().

      Har spurt om følgende i en kommentar i koden. Det hadde vært fint om dere svarer på og jeg ev. fikser det før jeg tester i preprod:
      
      // Lurer på om oppgaver til saksbehandlere opprettes slik?
      // Var også litt usikker på Oppgave type her!
      // Hvordan skal forklares at behandlingen kanskje skal henlegges pga manglende kravgrunlagg?
      // Er det riktig måte å sjekke om en lignende task for denne ikke ble opprettet i går? for å unngå flere opprettelse av samme task

LEFT JOIN Kravgrunnlag431 k431 ON beh.id = k431.behandling_id
WHERE k431.id IS NULL
AND beh.status != 'AVSLUTTET'
AND beh.opprettet_dato < CURRENT_TIMESTAMP - INTERVAL '8 weeks'
Copy link
Contributor

Choose a reason for hiding this comment

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

Hadde det vært ryddig å sende inn dato-verdien som et parameter for å lettere forstå koden?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

det hadde vært ryddig å sende inn dato som et parameter, ja. Det er gjort nå :)

@@ -309,13 +309,14 @@ class BehandlingService(
fun henleggBehandling(
behandlingId: UUID,
henleggelsesbrevFritekstDto: HenleggelsesbrevFritekstDto,
fraRydderBatch: Boolean? = null,
Copy link
Contributor

Choose a reason for hiding this comment

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

Trengs denne? Eller kan vi bruke henleggelsesbrevFritekstDto istedenfor?

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 har rett. Det trengs ikke. jeg kan bruke behandlingsresultatstype i Dto-en i if-sjekken videre i kanHenleggeBehandling ().

) {
val behandling = behandlingRepository.findByIdOrThrow(behandlingId)
sjekkOmBehandlingAlleredeErAvsluttet(behandling)

val behandlingsresultatstype = henleggelsesbrevFritekstDto.behandlingsresultatstype

if (!kanHenleggeBehandling(behandling, behandlingsresultatstype)) {
if (fraRydderBatch == null && !kanHenleggeBehandling(behandling, behandlingsresultatstype)) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Se ovenfor

Comment on lines +50 to +55
// Sjekk om en oppgave allerede finnes
// Opprette en oppgave i fagsystem for å henlegge manuelt og ev. sende et brev.
// Lurer på om oppgaver til saksbehandlere opprettes slik?
// Var også litt usikker på Oppgave type her!
// Hvordan skal forklares at behandlingen kanskje skal henlegges pga manglende kravgrunlagg?
// Er det riktig måte å sjekke om en lignende task for denne ikke ble opprettet i går? for å unngå flere opprettelse av samme task
Copy link
Contributor

Choose a reason for hiding this comment

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

Bare så jeg forstår det rett:
Det som ønskes oppnådd er å opprette en oppgave som en saksbehandler skal plukke - og dermed henlegge behandlingen manuelt..?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ja, tanken er akkurat det. Det er til behandlingene som et brev ble sendt for. Slik at saksbehandleren kan sende et brev på henleggelsen hvis han/hun vil.
Alle andre behandlinger kan henlegges automatisk.

Copy link
Contributor

Choose a reason for hiding this comment

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

Skjønner.
På sikt går det an å vurdere om disse skal henlegges automatisk og at man sender ut et generisk brev. Ser iallefall ut som det blir et generisk brev hvis jeg henlegger pga feilregistrert behandling i dag;

ref:
https://familie-tilbake-frontend.ansatt.dev.nav.no/fagsystem/EF/fagsak/200054975/behandling/c4b18896-f99e-476f-83de-a6a9f69433c5

Screenshot 2025-01-17 at 09 00 40

…gelsesbrevFritekstDto. Har også sendt utgått dato som parameter til sqls-spørringen
// Sjekk om en oppgave allerede finnes
// Opprette en oppgave i fagsystem for å henlegge manuelt og ev. sende et brev.
// Lurer på om oppgaver til saksbehandlere opprettes slik?
// Var også litt usikker på Oppgave type her!
Copy link
Contributor

Choose a reason for hiding this comment

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

Jeg sjekker med fagressurs ift oppgavetype

Copy link
Contributor

Choose a reason for hiding this comment

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

Tilbakemelding fra fagressurs;
Oppgavetype: Vurder henvendelse

Forslag til tekst (denne kan dere også diskutere)

Tilbakekrevingsbehandlingen for stønad ... (opprettet DD.MM.YYYY) ble opprettet for over n uker siden og har ikke mottatt kravgrunnlag. 

Med mindre det er foretatt en revurdering med tilbakekrevingsbeløp i dag eller de siste dagene for stønaden, så vil det ikke oppstå et kravgrunnlag i dette tilfellet. Tilbakekrevingsbehandlingen kan derfor henlegges manuelt. 

if (finnesAlleredeEnTask == null ||
!finnesAlleredeEnTask.metadata.getProperty("oppgavetype").equals(Oppgavetype.VurderHenvendelse)
) {
oppgaveTaskService.opprettOppgaveTask(behandling, Oppgavetype.VurderHenvendelse)
Copy link
Contributor

Choose a reason for hiding this comment

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

Bør vi lage en annen tasktype per behandling, for å entydig vise at det er DETTE vi skal lage?

RyddBehandlingUtenKravgrunnlagTask ?

Hvis denne tasken finnes fra før så gjør vi ingenting, men hvis den ikke finnes så oppretter vi oppgave eller rydder den automatisk?

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