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-22778: Korrigerer kall til integrasjoner ved endring av behandlede enhet #4829

Conversation

thoalm
Copy link
Contributor

@thoalm thoalm commented Oct 15, 2024

💰 Hva skal gjøres, og hvorfor?

Favro: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-22778

Patch-endepunktet fungere ikke som tenkt i familie-integrasjoner. Det vil ikke nullstille mappeId og tilordnetRessurs når de er satt til null da koden er satt opp til å ingorere null felt.

Bytter om til metoden som ble brukt tidligre for å "patch" oppgaven, men utvider det nå til å også nullstill tilordnetRessurs

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

Nei

✅ 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 👇

Jeg har ikke skrevet tester fordi:
Hotfix for et problem, skal endres i med en gang etter merge

💬 Ønsker du en muntlig gjennomgang?

  • Ja
  • Nei

@thoalm thoalm force-pushed the NAV_22778_korriger_kall_til_integrasjoner_ved_endring_av_behandlede_enhet branch from 65a9ec2 to 01ecd34 Compare October 15, 2024 14:01
@thoalm thoalm force-pushed the NAV_22778_korriger_kall_til_integrasjoner_ved_endring_av_behandlede_enhet branch from 01ecd34 to f0c2793 Compare October 15, 2024 14:11
@UyQuangNguyen UyQuangNguyen marked this pull request as ready for review October 16, 2024 07:20
@UyQuangNguyen UyQuangNguyen requested a review from a team as a code owner October 16, 2024 07:20
Copy link

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.

integrasjoner trenger å fikses først, men denne er ok

@UyQuangNguyen UyQuangNguyen added this pull request to the merge queue Oct 18, 2024
Merged via the queue into main with commit 9981d2a Oct 18, 2024
8 of 14 checks passed
@UyQuangNguyen UyQuangNguyen deleted the NAV_22778_korriger_kall_til_integrasjoner_ved_endring_av_behandlede_enhet branch October 18, 2024 09:26
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.

4 participants