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

Vi ønsker å kunne begrunne avslagsperiode for søker selvom det bare finnes uregistrert barn og ingen andre på søknad #4039

Merged
merged 4 commits into from
Oct 3, 2023

Conversation

UyQuangNguyen
Copy link
Contributor

Favrokort: https://favro.com/organization/98c34fb974ce445eac854de0/1844bbac3b6605eacc8f5543?card=NAV-14370
Vi har en feil der vi ikke kan begrunne avslag på de ordinære vilkårene til søker dersom det bare eksisterer uregistrert barn.

Før:
førrrr

Etter:
etterrr

…innes uregistrert barn og ingen andre på søknad
@UyQuangNguyen UyQuangNguyen requested a review from a team as a code owner October 2, 2023 22:16
): Tidslinje<List<VilkårResultat>, Måned> {
val forskjøvedeVilkårResultaterForPerson = this.tilForskjøvedeVilkårTidslinjer(person.fødselsdato).kombiner { it }

return when (vilkårRolle) {
PersonType.SØKER -> forskjøvedeVilkårResultaterForPerson.map { vilkårResultater ->
vilkårResultater?.filtrerErIkkeOrdinærtFor(vilkårRolle)?.takeIf { it.isNotEmpty() }
if (bareSøkerOgUregistrertBarn) {
vilkårResultater?.toList()?.takeIf { it.isNotEmpty() }
Copy link
Contributor

Choose a reason for hiding this comment

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

Ligger det kun vilkår på søker når det er uregistrert barn? Er det grunnen til at vi ikke trenger filtrerErIkkeOrdinærtFor()?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Stemmer, vanligvis blir vilkårene til søker filtrert bort (bortsett fra utvidet) når det eksisterer et annet barn med vilkår.
I dette tilfellet funker ikke den logikken siden det ikke finnes et annet barn med vilkår.

@UyQuangNguyen UyQuangNguyen enabled auto-merge (squash) October 3, 2023 10:23
@UyQuangNguyen UyQuangNguyen merged commit 351289f into main Oct 3, 2023
@UyQuangNguyen UyQuangNguyen deleted the fiks_manglende_avslagsperiode branch October 3, 2023 10:34
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