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-24387: Forvalter-endepunkt for å kunne korrigere andeler i fagsaker med avvik. #5168

Merged

Conversation

bragejahren
Copy link
Contributor

@bragejahren bragejahren commented Mar 13, 2025

NAV-24387

💰 Hva skal gjøres, og hvorfor?

Etter en forholdsvis vellykket konsistensavstemming, hvor vi benyttet UtbetalingsTidslinjeService for å finne periodene vi skulle sende til Oppdrag, ble det besluttet at vi skulle forsøke å bruke UtbetalingsTidslinjeService til å finne og korrigere andeler med avvik.

Legger her til et forvalter-endepunkt som lar oss sende inn en liste over fagsakId'er vi vet at har avvik, for å korrigere tilhørende andeler i siste iverksatte behandling med feil periodeId, forrigePeriodeId eller kildeBehandlingId.

For å sikre at endepunktet ikke kjøres ved et uhell er det lagt på en toggle som må være på for at endepunktet skal gjøre noe i tillegg til en dryRun parameter i request-objektet som default er satt til true, slik at vi kan se hva resultatet av en kjøring vil bli før noe blir persistert til db.

For alle andeler som korrigeres, lagrer vi "originalen" i en ny tabell "PatchetAndelTilkjentYtelse" i tilfelle vi skulle ønske å finne ut hvordan en bestemt andel så ut tidligere. Denne tabellen er tenkt at skal slettes etterhvert. Litt avhengig av hvordan patchingen går og om vi ser at vi vil ha behov for å benytte endepunktet flere ganger i fremtiden.

🔎️ Er det noe spesielt du ønsker tilbakemelding om?

✅ Checklist

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

@bragejahren bragejahren requested a review from a team as a code owner March 13, 2025 13:19
@bragejahren bragejahren changed the title NAV-24387: Forvalter-endepunkt for å kunne patche fagsaker med avvik. NAV-24387: Forvalter-endepunkt for å kunne korrigere andeler i fagsaker med avvik. Mar 13, 2025
val nasjonaltPeriodebeløp: Int?,
@Column(name = "differanseberegnet_periodebelop")
val differanseberegnetPeriodebeløp: Int? = null,
) : BaseEntitet()
Copy link
Contributor

Choose a reason for hiding this comment

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

Gir det mening å ha baseentitet her? Feltene i baseentitet vil settes til din bruker og når raden i PATCHET_ANDEL_TILKJENT_YTELSE opprettes, men egentlig er man vel interessert i det som faktisk var på aty'en.

Copy link
Contributor

@stigebil stigebil left a comment

Choose a reason for hiding this comment

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

En liten kommentar på basenetitet, men ser generelt bra ut

bragejahren and others added 2 commits March 20, 2025 14:48
…eg kan lagre ned de faktiske verdiene som finnes på original AndelTilkjentYtelse
@bragejahren bragejahren enabled auto-merge March 20, 2025 14:06
@bragejahren bragejahren added this pull request to the merge queue Mar 20, 2025
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
3.0% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

Merged via the queue into main with commit 07f7ca2 Mar 20, 2025
7 of 8 checks passed
@bragejahren bragejahren deleted the feat/forvalter-endepunkt-for-patching-av-fagsaker-med-avvik branch March 20, 2025 14:15
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