IS-3073: Be om oppfølgingsplan ved flere arbeidsgivere #1635
Conversation
|
|
||
| const texts = { | ||
| aktivForesporsel: | ||
| "Obs! Det ble bedt om oppfølgingsplan fra denne arbeidsgiveren", | ||
| "Obs! Det ble bedt om oppfølgingsplan fra denne arbeidsgiveren", // TODO: Adjust to multiple arbeidsgivere ? |
There was a problem hiding this comment.
Må vi endre ordlyden eller logikken rundt når denne vises når vi støtter flere arbeidsgivere?
There was a problem hiding this comment.
Jeg tenker det, og kanskje det etterhvert dukker opp et historikk-behov her?
Evt burde vi nok populere historikk-siden med dette ganske snart også 🤔 Må ikke glemme å holde den vedlike.
There was a problem hiding this comment.
Lurt med historikk ja 👍
Den aktiv forspørsel-advarselen vises jo nå hvis det finnes en forespørsel for det aktive oppfølgingstilfellet. Så kunne i først omgang bare endret teksten til det, evt vise navn på arbeidsgiveren det er forespurt fra.
There was a problem hiding this comment.
Tilpasset alert til å inkludere navn på virksomhet nå 👍
andersrognstad
force-pushed
the
IS-3073
branch
3 times, most recently
from
28fc97f to
485fdb8
Compare
andersrognstad
changed the title
There was a problem hiding this comment.
Denne ble litt stor etter hvert så flyttet det som går direkte på BeOmOppfolgingsplan-funksjonalitet til BeOmOppfolgingsplanTest
|
Det er også noe med logikken for når man kan be om oppfølgingsplan vi kanskje må se på her. Nå er |
andersrognstad
force-pushed
the
IS-3073
branch
2 times, most recently
from
925abdc to
36cd72b
Compare
andersrognstad
force-pushed
the
IS-3073
branch
from
36cd72b to
516ccf6
Compare
| <Controller | ||
| name="narmesteLeder" | ||
| control={control} |
There was a problem hiding this comment.
Hva hjelper Controller oss med?
There was a problem hiding this comment.
Vil helst ha NarmesteLederRelasjonDTO som type på skjemafeltet, men <Radio> kan ikke ha NarmesteLederRelasjonDTO som type på value. Derfor må vi finne nærmeste leder basert på value i onChange under og sette denne som verdi i field.onChange(selectedNarmesteLeder);
Ut i fra det jeg fant på nettet og chatgpt virket <Controller> som måten å gjøre det på når man vil ha en custom onChange :)
Jeg tenker det høres ut som noe vi vil legge opp til ja, at vi sjekker hvor du har aktiv oppfølgingsplan fra. Det må kanskje på plass først? |
|
andersrognstad
force-pushed
the
IS-3073
branch
from
411adda to
e20ffcb
Compare
Hva har blitt lagt til✨🌈
Legger til valg av arbeidsgiver når det er flere aktive nærmeste ledere for det aktive oppfølgingstilfellet.
Når det bare er én nærmeste leder blir det som før, dvs valget vises ikke:
Dersom det er flere vises radio-knapper med validering:
Tanker? Burde vi f.eks vise radio-knapp uansett om det er én eller flere?
TODO