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-22311: Bestemmer arbeidsfordelingsenhet utenfor OpprettOppgave-task #4799

Conversation

bragejahren
Copy link
Contributor

Favro: NAV-22311

💰 Hva skal gjøres, og hvorfor?

Tidligere tilpasset/justerte vi arbeidsfordeling på behandling i OpprettOppgaveTask slik at BehandleSak-oppgave fikk korrekt kombinasjon av enhet og saksbehandler. I og med at OpprettOppgaveTask, kjøres asynkront, blir ikke arbeidsfordeling på behandling oppdatert før saksbehandler blir rutet til "Registrer søknad"-steget. Dette fører til at det ser ut som at behandlingen er satt på feil enhet, selv om den egentlig er riktig (dukket opp korrekt dersom saksbehandler oppdaterer siden).

Tilpasser derfor arbeidsfordelingen i henhold til saksbehandlers tilganger tidligere, og utenfor OpprettOppgaveTask i fastsettBehandlendeEnhet-metoden i ArbeidsfordelingService. Dette gir også mer mening da det er i fastsettBehandlendeEnhet vi henter "geografisk behandlende enhet", som vi nå validerer tilgangen til.

✅ Checklist

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

…ndeEnhet fremfor i opprettOppgave. OppgaveArbeidsfordelingService -> TilpassArbeidsfordelingService.
@bragejahren bragejahren requested a review from a team as a code owner October 1, 2024 10:00
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.

Synes det ser bra ut! 🙌

Copy link
Contributor

@thoalm thoalm left a comment

Choose a reason for hiding this comment

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

LGTM! 🤩

Noen kommentarer du kan vurdere

@bragejahren bragejahren added this pull request to the merge queue Oct 8, 2024
Merged via the queue into main with commit 171f085 Oct 8, 2024
5 checks passed
@bragejahren bragejahren deleted the NAV_22311/bestemmer-arbeidsfordelingsenhet-utenfor-opprett-oppgave-task branch October 8, 2024 07:46
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