AAP-2194 Bruk meldepliktstatus i underveis for å identifisere fritaksjobber#2846
Draft
peterbb wants to merge 1 commit into
Draft
AAP-2194 Bruk meldepliktstatus i underveis for å identifisere fritaksjobber#2846peterbb wants to merge 1 commit into
peterbb wants to merge 1 commit into
Conversation
…jobber Å se på vurderingene av fritak, så tar man ikke hensyn til om brukeren har rett på AAP eller om meldeplikten kanskje er oppfylt på andre måter.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
🔍 Detekt Code Quality ReportTotal Issues: 96
|
| .begrensetTil(sistePasserteMeldeperiode) | ||
| .segmenter().any { (_, underveisperiode) -> | ||
| underveisperiode.utfall == Utfall.OPPFYLT | ||
| && underveisperiode.meldepliktStatus == MeldepliktStatus.FRITAK |
Contributor
There was a problem hiding this comment.
Etter en førstegangsbehandling er vel alle meldepliktstatusene FØR_VEDTAK, hvordan fanger vi opp den første fritakskjøringen for disse? 🤔
Contributor
There was a problem hiding this comment.
Ift OpprettJobbForFritakMeldepliktJobbUtfører så plukker denne ut 143 behandlinger for "sjekk" nå - som ikke er helt katastrofe...
Contributor
Author
There was a problem hiding this comment.
Ja, er derfor jeg satte den tilbake til draft. Nå kjører vi også denne fritaksjobben på saker uten rett til AAP, fordi vi kun ser på vurderingene. Kanskje det er lettere å bare beholde sjekken av fritaksvurderingen, men i tillegg sjekke om brukeren har rett til AAP i perioden.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Edit: Hmm, dette fungerer for å starte første behandling etter innvilgelse, for da er ingen meldeperioder markert med
FRITAK.Når man ser på vurderingene av fritak, så tar vi ikke hensyn til om brukeren har rett på AAP eller om meldeplikten kanskje er oppfylt på andre måter. Ved å ta utgangspunkt i underveis-tabellen, kan vi enklere se når fritak faktisk kan være relevant.
Legger på skrivelås siden vi potensielt muterer saken (oppretter behandling).
Ser det er en sjekk på åpen behandling med vurderingsbehov
FRITAK_MELDEPLIKT, men forstår ikke helt hvordan det er en hensiktsmessig sjekk. Ser for meg at sjekken burde se på alle behandlinger opprettet etter forrige meldeperiode (pluss ~3 dager?), og se om noen av de har vurderingsbehovFRITAK_MELDEPLIKT.Kan også optimalisere utplukket i
OpprettJobbForFritakMeldepliktJobbUtfører, men har ikke gjort det i denne PRen.