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 på nye felter i Behandling. #415

Merged
merged 6 commits into from
Feb 14, 2024
Merged

Conversation

ramrock93
Copy link
Collaborator

@ramrock93 ramrock93 commented Feb 13, 2024

Disse endringene vil medføre til deserialiseringsfeil for allerede persistere behandlinger uten disse feltene.
Det kan løses enten via republisering av de dataene, eller ved å gjøre dem nullable.

  • aksjonspunkt.tidsfrist: når behandlingen går av vent
  • behandling.opprettetTidspunkt og avsluttetTidspunkt
  • behandling.behandlingResultat: resultat på behandling hvis avsluttet. Bruker eget enum fordi ønsker ikke å eksponere alle resultattypene
  • bruker æøå i enum slik som i k9-sak

Dette vil medføre parsingfeil for allerede persistere behandlinger uten opprettetDato.
Det kan løses enten via republisering av de dataene, eller ved å gjøre opprettetDato nullable.
@ramrock93 ramrock93 requested a review from qtips February 13, 2024 09:50
@ramrock93 ramrock93 requested a review from a team as a code owner February 13, 2024 09:50
- aksjonspunkt.tidsfrist: når behandlingen går av vent
- behandling.opprettetTidspunkt og avsluttetTidspunkt
- behandling.behandlingResultat: resultat på behandling hvis avsluttet. Bruker eget enum fordi ønsker ikke å eksponere alle resultattypene

bruker æøå i enum slik som i k9-sak
endrer til å bruke tidspunkt på opprettet fordi vi har den. Bruker zonedatetime da vi allerede bruker det i søknad.
@ramrock93 ramrock93 changed the title Legger på opprettet og avsluttet datoer i Behandling. Legger på nye felter i Behandling. Feb 13, 2024
Endrer konstantene til å bruke Period da det er ment for dager og større enheter. Duration er ment for timer. Endrer også til å bruke string slik som i k9-sak fordi det tydlig viser format på perioden.
@ramrock93 ramrock93 merged commit 0ba9f77 into master Feb 14, 2024
@ramrock93 ramrock93 deleted the innsyn-behandlingsdatoer branch February 14, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants