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

Hent første vedtaksperiode som faktisk har en fom dato før vi sjekker etter endringstidspunkt #4606

Conversation

UyQuangNguyen
Copy link
Contributor

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

Vi henter endringstidspunkt i denne koden ved å hente alle sorterte vedtaksperioder (ved å bruke fom), og deretter så tar vi den tidligste fommen. Problemet med dette er at hvis fom er null i et vedtaksperiode som f.eks ved avslag, så vil den bli plassert først i lista. (se https://pl.kotl.in/EI5fbYvTC)

Henter derfor første som faktisk har fom.

@UyQuangNguyen UyQuangNguyen requested a review from a team as a code owner June 4, 2024 15:08
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.

🙌

@UyQuangNguyen UyQuangNguyen enabled auto-merge June 5, 2024 06:28
@UyQuangNguyen UyQuangNguyen added this pull request to the merge queue Jun 5, 2024
Merged via the queue into main with commit 0935e8a Jun 5, 2024
5 checks passed
@UyQuangNguyen UyQuangNguyen deleted the Hent_første_vedtaksperiode_som_faktisk_har_fom_dato_før_vi_sjekker_etter_endringstidspunkt branch June 5, 2024 06:44
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