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.

Sign up for GitHub

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

Jump to bottom

Rydd opp i valutajustering validering #4599

Merged
merged 18 commits into from
Jun 4, 2024
Merged

Rydd opp i valutajustering validering #4599

merged 18 commits into from
Jun 4, 2024

Conversation

MagnusTonnessen
Copy link
Contributor

@MagnusTonnessen MagnusTonnessen commented Jun 3, 2024 •
edited by halvorbmundal
Loading

💰 Hva skal gjøres, og hvorfor?

Favro: NAV-21237

Et par forberedelser før vi gjør månedlig valutajustering live.

  • Bruker fagsak id i løypa for månedlig valutakurs
  • Endre sql-spørring for hva vi skal se på ved månedlig valutajustering. Nå ser vi på fagsaker der det er en valutakurs som er utdatert og som fremdeles har løpende ytelse.
  • Fikser bug på rekkefølge oppdatere kompetanser og legge til nye rader på valutakurser for gjeldende måned.

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

Er det noe du er usikker på eller ønsker å diskutere? Beskriv det gjerne her eller kommenter koden det gjelder.

✅ 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

MagnusTonnessen and others added 16 commits June 3, 2024 12:26
@MagnusTonnessen
Flytt sjekk av valutakurser inn i utførMånedligValutajustering
b7eb609
@MagnusTonnessen
Slett ubrukt funksjon og tester
a09ef15
@MagnusTonnessen
Ikke oppdater valutakurser tilbake i tid ved automatisk behandling
ceda17e
@MagnusTonnessen
Legg til mulighet for å teste valutakurser i cucumber
868a4ea
@MagnusTonnessen
Fikse mock
ac176fd
@MagnusTonnessen
Legg til test
abb049a
@MagnusTonnessen
Gjør så vi oppdaterer valutakursene etter at kompetansene er tilpasse…
a6e0288 
…t regelverket
@MagnusTonnessen
Endre til å bruke fagsak id når vi skal opprette månedligValutajuster…
14baf31 
…ingTasker
@halvorbmundal
Fiks test. Søkers aktivitetsland må være det samme som utbetalingslan…
48e9801 
…det i utenlandske periodebeløp
@halvorbmundal
v.tom == null -> v.tom IS null
3ce4297
@halvorbmundal
Fagsakstatus -> Status
636ea85
@halvorbmundal
Send inn fagsakId
795cee7
@halvorbmundal
Faksakstatus -> Status
816d098
@halvorbmundal
Flytt kompetanse før tilkjent ytelse
bee75d8
@halvorbmundal
Legg til utenlansk periodebeløp i testverktøy
2cdf073
@halvorbmundal
Legg til valutakurser i testverktøy
50704ff
halvorbmundal
halvorbmundal approved these changes Jun 4, 2024
View reviewed changes
Copy link
Contributor

@halvorbmundal halvorbmundal left a comment

Choose a reason for hiding this comment

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

LGTM

@halvorbmundal halvorbmundal marked this pull request as ready for review June 4, 2024 11:31
@halvorbmundal halvorbmundal requested review from a team as code owners June 4, 2024 11:31
@halvorbmundal halvorbmundal enabled auto-merge June 4, 2024 11:35
@halvorbmundal halvorbmundal added this pull request to the merge queue Jun 4, 2024
Merged via the queue into main with commit 2681815 Jun 4, 2024
5 checks passed
@halvorbmundal halvorbmundal deleted the feat/rydd-opp-i-valutajustering-validering branch June 4, 2024 11:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@halvorbmundal halvorbmundal halvorbmundal approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MagnusTonnessen @halvorbmundal