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

Legger med hjemmel på status som er fastsatt #580

Merged
merged 1 commit into from
Dec 16, 2024
Merged

Conversation

pekern
Copy link
Contributor

@pekern pekern commented Dec 10, 2024

No description provided.

@pekern pekern requested review from a team as code owners December 10, 2024 13:24
@pekern pekern requested review from espenjv and jolarsen December 10, 2024 13:24
@pekern
Copy link
Contributor Author

pekern commented Dec 10, 2024

Fpsak har behov for å vite hjemmel brukt til å beregne status for bruk i stønadsstatistikk og XML. Legger opp til expand contract her

Copy link
Contributor

@jolarsen jolarsen left a comment

Choose a reason for hiding this comment

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

Er det ikke 1-1 mellom aktivitetsstatus og hjemmel ?
Er det DagP og AAP som lager problem her?

@pekern
Copy link
Contributor Author

pekern commented Dec 12, 2024

Nei, det satt opp til å være ulike hjemler avhengig av hva slags regler man har brukt for statusen, avhengig av om det er besteberegnet eller ikke. Tror det tryggeste er å få disse fra kalkulus, men hvis vi heller vil duplisere litt logikk for å utlede hjemmel in i fpsak er det nok mulig.

@pekern
Copy link
Contributor Author

pekern commented Dec 12, 2024

Er hovedsakelig et problem der vi setter hjemmel = 14_7, den brukes av de fleste statuser, som regel ifm manuell beregning eller manuell besteberegning

@pekern pekern merged commit 0cff048 into master Dec 16, 2024
5 of 6 checks passed
@pekern pekern deleted the feature/as-hjemmel branch December 16, 2024 07:58
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