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

Gjenbruk inngangsvilkår v2 #2727

Merged
merged 88 commits into from
Dec 13, 2024
Merged

Gjenbruk inngangsvilkår v2 #2727

merged 88 commits into from
Dec 13, 2024

Conversation

gunnsteingarmo
Copy link
Contributor

@gunnsteingarmo gunnsteingarmo commented Nov 27, 2024

Hvorfor er denne endringen nødvendig? ✨

Det skal være mulig å kun gjenbruke ett bestemt inngangsvilkår i stedet for alle sammen.

Frontend PR
Favro
Favro for å unngå gjenbruk som ikke finnes

gunnsteingarmo and others added 30 commits November 19, 2024 10:39
…år. Sender med vilkår id i stedet for vilkår type.
…gjenbrukes, og gjenbruk av vilkårsvurdering for ett vilkår.
… gjenbruk-inngangsvilkår-v2

# Conflicts:
#	src/main/kotlin/no/nav/familie/ef/sak/vilkår/VurderingController.kt
#	src/main/kotlin/no/nav/familie/ef/sak/vilkår/dto/VilkårsvurderingDto.kt
gunnsteingarmo and others added 2 commits December 5, 2024 16:33
…liste med ider for vikårsvurderings som kan gjenbrukes.
@gunnsteingarmo
Copy link
Contributor Author

Vi har gått bort fra å sette property "kanGjenbrukes" på dto. Nå har vi et nytt endepunkt som returnere alle vilkårsvurderinger som kan gjenbrukes.

Copy link
Contributor

@ViktorGSolberg ViktorGSolberg 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 hadde vært fint med noen integrasjonstester i VurderingControllerTest 😊

gunnsteingarmo and others added 4 commits December 11, 2024 14:43
…på parameter 'utledGjenbrukbareVilkårsvurderinger' slik at det blir mer konsekvent. Små endringer i 'hentGjenbrukbareVilkårsvurderinger' skal sjekke first or null. Fjernet ubrukt endepunkt 'gjenbruk'.
@gunnsteingarmo gunnsteingarmo merged commit 9e46f89 into main Dec 13, 2024
2 checks passed
@gunnsteingarmo gunnsteingarmo deleted the gjenbruk-inngangsvilkår-v2 branch December 13, 2024 12:55
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.

5 participants