Skip to content

Conversation

@palfi
Copy link
Collaborator

@palfi palfi commented Nov 4, 2025

Ny v2 tjeneste
Flyttet oppgitt tilknytning til medlemdto
Flyttet felter rundt startdato/dekningsgrad for fakta om saken til ytelsefordelingdto. Kanskje et naturlig hjem, men også åpen ny tjeneste/dto. Fakta om saken trenger blanding av YF og søknad (mottatt dato for søker og annen part)

@palfi palfi requested a review from jolarsen November 4, 2025 13:15
@palfi palfi requested a review from a team as a code owner November 4, 2025 13:15
record Annenpart(@NotNull Set<PersonadresseDto> adresser, @NotNull Set<Region> regioner, @NotNull Set<Personstatus> personstatuser) {
}

record OppgittUtenlandsopphold(@NotNull boolean oppholdSistePeriode, @NotNull boolean oppholdNestePeriode,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Trenger man disse booleansene? Det går vel på oppgitt Nei (ingen periode) eller Ja (med perioder som kan være tom)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Det vet jeg ikke, tidligere_opphold fra databasen som brukes for å finne disse

@palfi palfi force-pushed the feature/TFP-6443 branch 2 times, most recently from 75f8093 to c61657f Compare November 5, 2025 08:34
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 5, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
30.8% Coverage on New Code (required ≥ 60%)
B Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@palfi palfi merged commit d706650 into master Nov 5, 2025
9 of 10 checks passed
@palfi palfi deleted the feature/TFP-6443 branch November 5, 2025 13:37
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