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

Fix: Omskriving av OPPDATER_UTVIDET_KLASSEKODE-behandling for å kunne fikse fagsaker med feil periodeIder #5071

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

Conversation

bragejahren
Copy link
Contributor

NAV-24233

💰 Hva skal gjøres, og hvorfor?

For å få korrigert fagsaker med feil offsets på siste iverksatte behandling skriver vi her om logikken i OPPDATER_UTVIDET_KLASSEKODE-behandlinger. Nå sørger vi for at vi sender alle andeler fra og med andelen som overlapper med inneværende mnd og fremover for alle fagsaker som er berørt. I verste fall vil dette innebære at vi sender andeler fra og med 01.07.23 og frem over på nytt, og vi vil derfor få 0-utbetalinger fra 01.07.23 og for alle måneder frem til nå.

Fagsaker som blir plukket ut for kjøring er fagsaker som tidligere har hatt en OPPDATER_UTVIDET_KLASSEKODE-behandling og som har blitt revurdert i etterkant. Noen av disse vil ikke ha feil offset og vil bli ignorert i selve jobben.

En spørring 11.02.25 sier at det er 995 fagsaker som vil bli plukket ut, og det vil da altså være et subset av disse vi vil kjøre behandlingen for.

✅ Checklist

  • Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️
  • Jeg har config- eller sql-endringer.
  • Jeg har skrevet tester.

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

…nde tester da vi ikke lenger splitter utvidet andelene i OPPDATER_UTVIDET_KLASSEKODE behandling
…e korrigere fagsaker med feil offsets i siste iverksatte behandling
… noen enkel måte å skrive om testen til å passe omskrivning av autovedtaket.
@bragejahren bragejahren requested a review from a team as a code owner February 11, 2025 14:35
Comment on lines +21 to +27
// Fjerner alle andeler som overlapper eller kommer etter inneværende måned
if (it.stønadFom <= inneværendeMåned && it.stønadTom >= inneværendeMåned) {
null
} else if (it.stønadFom >= inneværendeMåned) {
null
} else {
// Tar med alle andeler som kommer før
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ved å fjerne alle utvidet andeler fra forrige iverksatte behandling som overlapper med inneværende måned eller har fom senere vil utbetalingsgenerator tolke det som at vi ønsker å legge til nye andeler i denne perioden. Dette vil igjen innebære at alle disse andelene som egentlig er helt like vil få nye offsets/periodeId'er og disse vil nå være oppdaterte og korrekte.

@bragejahren bragejahren changed the title Fix: Omskriving av OPPDATER_UTVIDET_KLASSEKODE-behandling for å kunne fikse fagsaker med feil offsets Fix: Omskriving av OPPDATER_UTVIDET_KLASSEKODE-behandling for å kunne fikse fagsaker med feil periodeIder Feb 11, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
31.4% Coverage on New Code (required ≥ 70%)

See analysis details on SonarQube Cloud

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.

1 participant