IS-2601: Tildel veileder #1428
IS-2601: Tildel veileder #1428
Conversation
| import { SYFOVEILEDER_ROOT } from "../../src/apiConstants"; | ||
| import { ANNEN_VEILEDER, VEILEDER_DEFAULT } from "../common/mockConstants"; | ||
|
|
||
| const veiledereMock = [VEILEDER_DEFAULT, ANNEN_VEILEDER]; |
There was a problem hiding this comment.
Har ryddet litt i mockingen her og fjernet duplikat veilederMock-kode
There was a problem hiding this comment.
Denne koden er flyttet til TildeltVeileder-komponenten og ut av personkortet.
andersrognstad
force-pushed
the
IS-2601-tildel-veileder
branch
from
541f0d1
to
3290502
Compare
src/components/TildelVeileder.tsx
Outdated
|
|
||
| const StyledCombobox = styled(UNSAFE_Combobox)` | ||
| .navds-combobox__list { | ||
| max-height: 15rem; |
There was a problem hiding this comment.
Setter en max-høyde på combobox-lista slik at den ikke forsvinner ut av modalen.
| enhet: valgtEnhet, | ||
| }; | ||
|
|
||
| return post(path, { |
There was a problem hiding this comment.
APIet som finnes i syfooversiktsrv tar inn en liste med tilknytninger (siden man i oversikten kan velge flere personer). I tillegg er enhet: string påkrevd uten at den brukes til noe i APIet. Vi kan vurdere å lage et eget endepunkt i syfooversiktsrv (uten liste og enhet) som bare brukes herfra. Da kan vi også enklere telle i grafana hvor mange tildelinger som skjer fra syfomodiaperson.
There was a problem hiding this comment.
Høres ut som noe vi kan gjøre ja, men evt forbedre etterhvert.
Hvis enhet hadde vært mulig å sende inn og at den faktisk ble brukt til noe, så har vi vel løst hele "ruting" problematikken internt hos oss i syfo? 😬 Så vi er jo åpenbart ikke langt unna å løse noe sånt, men det er en veldig lokal løsning på et større problem da.
There was a problem hiding this comment.
Vi hadde et ubrukt endepunkt i syfooversiktsrv så tok og ryddet litt der nå. Fikser eget api for syfomodiaperson samtidig, kommer PR :)
| @@ -230,43 +217,4 @@ describe("PersonkortHeader", () => { | |||
|
|
|||
| expect(screen.getByText("Under arbeidsrettet oppfølging")).to.exist; | |||
| }); | |||
|
|
|||
| describe("TildeltVeileder", () => { | |||
There was a problem hiding this comment.
Flyttet dette til TildeltVeilederTest
andersrognstad
force-pushed
the
IS-2601-tildel-veileder
branch
3 times, most recently
from
82c6997
to
10284c3
Compare
andersrognstad
force-pushed
the
IS-2601-tildel-veileder
branch
from
10284c3
to
80aeecb
Compare
| @@ -1,6 +1,7 @@ | |||
| import { NarmesteLederRelasjonStatus } from "../../src/data/leder/ledereTypes"; | |||
| import { RSContext } from "../../src/data/modiacontext/modiacontextTypes"; | |||
| import { Veileder } from "../../src/data/veilederinfo/types/Veileder"; | |||
| import { VeilederBrukerKnytningDTO } from "@/data/veilederbrukerknytning/useGetVeilederBrukerKnytning"; | |||
There was a problem hiding this comment.
Trodde ikke sånne imports var lov i mock? Kjører det lokalt nå?
| open, | ||
| handleClose, | ||
| }: ChangeTildeltVeilederModalProps) { | ||
| const [isError, setIsError] = useState(false); |
There was a problem hiding this comment.
Kan evt skrive det om til et form senere?
There was a problem hiding this comment.
Ja, føltes litt overkill med bare et input-felt, men er en mulighet hvis det blir mer komplekst senere 👍
| enhet: valgtEnhet, | ||
| }; | ||
|
|
||
| return post(path, { |
There was a problem hiding this comment.
Høres ut som noe vi kan gjøre ja, men evt forbedre etterhvert.
Hvis enhet hadde vært mulig å sende inn og at den faktisk ble brukt til noe, så har vi vel løst hele "ruting" problematikken internt hos oss i syfo? 😬 Så vi er jo åpenbart ikke langt unna å løse noe sånt, men det er en veldig lokal løsning på et større problem da.
Hva har blitt lagt til✨🌈
Flytter informasjon om tildelt veileder øverst til høyre over personkortet. Lagt til endre-knapp (bak feature-toggle) som åpner modal hvor man kan velge veileder fra enheten og tildele personen til valgt veileder.
Laget feature-toggle først fordi jeg tenkte å ikke gjøre alt i en PR 😅 Nå ble det likevel det, men kan jo være greit å kunne skru det enkelt av dersom det får noen uønskede effekter i prod.
Screenshots 📸✨
Har gjort modalen litt høy for å få plass til en del av innholdet i comboboxen når den ekspanderes, men det ser litt rart ut når den ikke er ekspandert :/ Tar gjerne mot forslag til hvordan løse det bedre :)