-
Notifications
You must be signed in to change notification settings - Fork 1
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
IS-2649: Show content of oppfolgingsoppgave in oversikt #542
IS-2649: Show content of oppfolgingsoppgave in oversikt #542
Conversation
d30fbaa
to
6310dd1
Compare
6310dd1
to
ec3b1f5
Compare
src/api/types/personoversiktTypes.ts
Outdated
@@ -107,3 +107,32 @@ export enum Oppfolgingsgrunn { | |||
VURDER_ANNEN_YTELSE = 'VURDER_ANNEN_YTELSE', | |||
ANNET = 'ANNET', | |||
} | |||
|
|||
export function oppfolgingsgrunnToString( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nå vet jeg dette er bare flyttet, men er ikke dette akkurat det samme som ligger i src/data/oppfolgingsoppgave/types.ts
l.53 (oppfolgingsgrunnToText)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forstår ikke helt spørsmålet 😅 Det er bare flyttet ja. Så har vi fjernet en unødvendig default:
case i switch casen
ec3b1f5
to
ea1bdde
Compare
ea1bdde
to
12494da
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ikonet er bare klikkbart når veileder er i "Min oversikt"
I videoen er man i enhetens oversikt? Men ser jo i koden at det ser riktig ut 🤷🏼♂️
Ja sorry, litt misvisende 😅 Ingen lokalt som lå med en oppfølgingsoppgave. Burde stått noe om det for å spare dere forvirringen 🙈 |
Hva har blitt lagt til✨🌈
Screenshots 📸✨
Screen.Recording.2024-11-07.at.12.16.34.mov