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

Spesifiser starttidspunkt for utbetalingstabell ved årlig kontroll #4740

Merged
merged 6 commits into from
Aug 28, 2024

Conversation

MagnusTonnessen
Copy link
Contributor

💰 Hva skal gjøres, og hvorfor?

Favro: NAV-22065

Ved behandlingsårsak årlig kontroll skal utbetalingstabellen vise utbetalinger tilbake i tid med fom etter disse reglene:

  • Hvis det er gjort endringer tidligere enn 1. januar i fjor $\rightarrow$ Endringstidspunkt
  • Hvis det er gjort endring senere enn 1. januar i fjor $\rightarrow$ Seneste av 1. januar i fjor og første utbetaling i fjor
  • Hvis det ikke er gjort noen endring $\rightarrow$ Seneste av 1. januar i fjor og første utbetaling i fjor

✅ Checklist

Har du husket alle punktene i listen?

  • Jeg har testet mine endringer i henhold til akseptansekriteriene 🕵️
  • Jeg har config- eller sql-endringer. I så fall, husk manuell deploy til miljø for å verifisere endringene.
  • Jeg har skrevet tester. Hvis du ikke har skrevet tester, beskriv hvorfor under 👇

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

…etalinger tilbake til endringstidspunkt, hvis det er gjort endringer, eller det seneste av 1. januar i fjor og innvilgelsestidspunkt
@MagnusTonnessen MagnusTonnessen requested a review from a team as a code owner August 26, 2024 12:02
}

@Test
fun `finnStarttidspunktForUtbetalingstabell returnerer første januar i fjor selv om endringstidspunkt er senere`() {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
fun `finnStarttidspunktForUtbetalingstabell returnerer første januar i fjor selv om endringstidspunkt er senere`() {
fun `finnStarttidspunktForUtbetalingstabell returnerer første januar i fjor ved ÅRLIG_KONTROLL selv om endringstidspunkt er senere`() {

Copy link
Contributor

@bragejahren bragejahren left a comment

Choose a reason for hiding this comment

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

Litt småpirk i navngivning av tester for å gjøre det enda tydeligere hva som testes, men ellers ser det veldig bra ut 👍 ✅

@MagnusTonnessen MagnusTonnessen added this pull request to the merge queue Aug 28, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 28, 2024
@MagnusTonnessen MagnusTonnessen added this pull request to the merge queue Aug 28, 2024
Merged via the queue into main with commit 066a56b Aug 28, 2024
5 checks passed
@MagnusTonnessen MagnusTonnessen deleted the feat/utbetalingstabell-årlig-kontroll branch August 28, 2024 12:04
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