Skip to content

Tiltaksdeltaker-fiksing #839

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

Merged
merged 6 commits into from
Mar 3, 2025
Merged

Tiltaksdeltaker-fiksing #839

merged 6 commits into from
Mar 3, 2025

Conversation

tiafiring
Copy link
Contributor

Rydder opp i filtrering og mutering av tiltaksdeltakelser.

  • fjerner overskriving av tiltakstypenavnet ved adressebeskyttelse (man mottar ikke lenger gjennomføringsnavn her)
  • bruker logikk fra libs for å mappe deltakerstatuser fra kafka
  • støtter at fom/tom kan mangle for deltakelsen og legger datoene flatt i stedet for som en periode
  • filtrerer bort deltakelser som ikke gir rett på tiltakspenger, og deltakelser der både fom og tom mangler hvis statusen ikke er venter på oppstart (deltakelser med andre statuser mener jeg ikke er relevante å vurdere hvis de mangler datoer)

https://trello.com/c/xIpArfVl/1119-hvordan-filtrere-mutere-tiltaksdeltagelser-fra-registere

@tiafiring tiafiring requested a review from a team as a code owner February 28, 2025 12:21
@tiafiring tiafiring merged commit ae5620a into main Mar 3, 2025
4 checks passed
@tiafiring tiafiring deleted the tiltaksdeltaker-fiksing branch March 3, 2025 06:27
import java.time.LocalDate
import java.util.UUID

class TiltaksdeltagelseTest {
Copy link
Contributor

@Henreich Henreich Mar 3, 2025

Choose a reason for hiding this comment

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

Datosammenligninger blir alltid krøll i hodet mitt, så jeg synes det var kjempebra med mange tester her 💯

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