-
Notifications
You must be signed in to change notification settings - Fork 1
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
Hotfix: Fange tilfeller hvor det finnes flere saker av samme type #6423
base: main
Are you sure you want to change the base?
Conversation
} catch (e: InternfeilException) { | ||
val saker = sakService.finnSaker(beroertFnr) | ||
|
||
if (saker.size > 1) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Si det her skjer i produksjon, vil vi ikke potensielt nå da sende ut et brev selvom bruker allerede har sendt en søknad?
edit: Det kjøres en sjekk i kontrollpunktene i jobben på om bruker har en iverksatt behandling og da avbrytes den.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, men er det vanlig at det sendes ut dødshendelser flere ganger på samme person?
Tenkte kanskje det er mest sannsynlig at det opprettes ny hendelse fordi et annet familiemedlem dør, og da er det jo uansett hensiktsmessig at det blir sendt brev, selv om det potensielt er koblet til "feil sak".
Alternative løsninger blir å sjekke om det finnes behandling, i stedet for iverksatte som det gjøres i dag.
Eller skrive om kontrollpunktene til å håndtere en liste med saker.
Oddsen for at dette skjer er vel strengt tatt veldig lav nå som det ikke lenger er mulig å opprette flere saker på samme person. Finnes potensielt et par tilfeller hvor noen har flere saker i produksjon allerede, ettersom det ikke var noen sperre før.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Det er vel bare å sjekke i databasen det :D
Har ikke hørt noe om det i hvert fall.
Det er sikkert ikke krise, men det kan jo potensielt bli at bruker tror at en søknad de har sendt inn ikke er sendt inn allikevel som gjør bruker usikker. Samtidig om det skjer sjeldent så er det vel noe vi kan ignorere...
Men burde nok konsolidere behandlingssjekken for saker og den for kontrollpunktene?
Kan jo splitte ut hentSakMedBehandlinger
for en med flere saker som denne bruker og
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Litt usikker på om jeg forsto problemstillingen her. Selv om bruker har sendt inn en søknad, vil det vel fortsatt sendes et brev all tid denne søknaden ikke er ferdig behandlet?
Denne sjekken som legges på vil vel kun gjøre at de som skulle ha to eller flere saker pga splitt ifm fnr-endring ikke feiler ved kjøring?
sikkerLogg.error("Det finnes flere saker på bruker med ident=$beroertFnr: ${saker.joinToString()} ") | ||
|
||
val sakMedBehandlinger = behandlingService.hentSakMedBehandlinger(saker) | ||
sikkerLogg.error("Bruker sak ${sakMedBehandlinger.sak.id} for dødshendelsen") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Denne trenger kanskje ikke å være sikkerlogg eller error?
} catch (e: InternfeilException) { | ||
val saker = sakService.finnSaker(beroertFnr) | ||
|
||
if (saker.size > 1) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Litt usikker på om jeg forsto problemstillingen her. Selv om bruker har sendt inn en søknad, vil det vel fortsatt sendes et brev all tid denne søknaden ikke er ferdig behandlet?
Denne sjekken som legges på vil vel kun gjøre at de som skulle ha to eller flere saker pga splitt ifm fnr-endring ikke feiler ved kjøring?
No description provided.