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-23391: Setter riktig kategori og underkategori på behandlingstema for saksoversikt #4914

Conversation

thoalm
Copy link
Contributor

@thoalm thoalm commented Nov 22, 2024

💰 Hva skal gjøres, og hvorfor?

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

Behandlingstema for saksoversikt skal settes til det samme som er satt på den aktive behandlingen, evt. den siste iverksatte behandlingen om en aktiv behandling ikke finnes.

I tillegg har man tilpasset logikken for å utlede behandlingstema på diverse steder gjennom behandlingsløpet, f.eks. når man beveger seg fra vilkårsvurderingsteget. Nå skal behandlingstema oppdatere seg korrekt, og ikke unødvendig overskrive når saksbehandler har endret det manuelt.

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

Litt usikker på om oppdateringen til koden rundt verdikjedetesten blir helt korrekt. Det gir mening i mitt hode, men greit at noen andre dobbeltsjekker det.

Se https://github.com/navikt/familie-ba-sak/pull/4914/files#diff-8a57dc26690dd9c9f4c3f0585472af321e7b74b1133e5b22903d5465afb208c6R480

✅ 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

@thoalm thoalm requested a review from a team as a code owner November 22, 2024 12:26
@thoalm thoalm force-pushed the NAV_23391_setter_riktig_kategori_og_underkategori_på_behandlingstema_fagsak branch 2 times, most recently from e404559 to a066baf Compare November 25, 2024 09:39
@thoalm thoalm changed the title NAV-23391: Setter riktig kategori og underkategori på behandlingstema for fagsak NAV-23391: Setter riktig kategori og underkategori på behandlingstema for saksoversikt Nov 25, 2024
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.

@thoalm thoalm force-pushed the NAV_23391_setter_riktig_kategori_og_underkategori_på_behandlingstema_fagsak branch 4 times, most recently from 0de5cc1 to 41c98a8 Compare December 1, 2024 10:22
@thoalm thoalm force-pushed the NAV_23391_setter_riktig_kategori_og_underkategori_på_behandlingstema_fagsak branch from 41c98a8 to 8f32de9 Compare December 1, 2024 10:23
@thoalm thoalm requested review from stigebil and removed request for stigebil December 1, 2024 12:27
@thoalm thoalm self-assigned this Dec 1, 2024
@thoalm thoalm requested review from a team and stigebil December 1, 2024 12:28
Copy link
Contributor

@UyQuangNguyen UyQuangNguyen left a comment

Choose a reason for hiding this comment

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

Veldig veldig bra jobba ⭐

Ekstra stor ⭐ for å legge til tester der det ikke fantes!

@thoalm thoalm requested a review from UyQuangNguyen December 8, 2024 11:03
Copy link
Contributor

@UyQuangNguyen UyQuangNguyen left a comment

Choose a reason for hiding this comment

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

Småpirk fra meg, men ser fortsatt veldig bra ut ⭐

@thoalm thoalm added this pull request to the merge queue Dec 9, 2024
Merged via the queue into main with commit e9af1bb Dec 9, 2024
9 of 10 checks passed
@thoalm thoalm deleted the NAV_23391_setter_riktig_kategori_og_underkategori_på_behandlingstema_fagsak branch December 9, 2024 13:49
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.

3 participants