IS-2884: Move unntak-arsaker to ikke-aktuell

[andersrognstad]

Hva har blitt lagt til✨🌈

Flytter årsakene "friskmeldt" og "arbeidsforhold opphørt" fra unntak til ikke aktuell. Årsakene må fortsatt ligge der for å kunne vise historiske unntak.

Screenshots 📸✨

Unntak nå:

Ikke aktuell nå:

[eirikdahlen]

Kunne man delt det opp på en måte som gjør det enda mer tydelig at noe har skjedd her?:
CreateUnntakArsak { ... }  sånn som her, og så en DepricatedUnntakArsak, og så heller ha UnntakArsak = CreateUnntakArsak & DepricatedUnntakArsak eller noe sånt?

[andersrognstad]

union / intersection typer er ganske hårete med enums i typescript. Måtte eventuelt endret til å bruke vanlig string literal type:
eks:
type CreateUnntakArsak = "MEDISINSKE_GRUNNER" | "INNLEGGELSE_INSTITUSJON" | "FORVENTET_FRISKMELDING_INNEN_28UKER" | "DOKUMENTERT_TILTAK_FRISKMELDING"
type DeprecatedUnntakArsak = "FRISKMELDT" | "ARBEIDSFORHOLD_OPPHORT"
type UnntakArsak = CreateUnntakArsak | DeprecatedUnntakArsak
Da bruker man bare stringen (eks "FRISKMELDT") der hvor den brukes i stedet for UnntakArsak.FRISKMELDT, så det er litt mindre eksplisitt i bruk, men synes kanskje det er bedre likevel i dette tilfellet 🤔

[eirikdahlen]

Ja okey, da kan vi beholde det sånn 👍🏼

[vetlesolgaard]

Ble også litt forvirret av slik det er nå. Går dette?👇😊 Fikk ikke testet det, men får ingen kjørefeil i alle fall

export enum ValidUnntakArsak {
  MEDISINSKE_GRUNNER = "MEDISINSKE_GRUNNER",
  INNLEGGELSE_INSTITUSJON = "INNLEGGELSE_INSTITUSJON",
  FORVENTET_FRISKMELDING_INNEN_28UKER = "FORVENTET_FRISKMELDING_INNEN_28UKER",
  DOKUMENTERT_TILTAK_FRISKMELDING = "DOKUMENTERT_TILTAK_FRISKMELDING",
}

export enum DeprecatedUnntakArsak {
  FRISKMELDT = "FRISKMELDT",
  ARBEIDSFORHOLD_OPPHORT = "ARBEIDSFORHOLD_OPPHORT",
}

export type UnntakArsak = ValidUnntakArsak | DeprecatedUnntakArsak;

[andersrognstad]

Ser ut som det funka 😍

[andersrognstad]

Kan merge denne på mandag etter å ha gitt Lars og evt dvh en heads up.