Utbedre tildel oppgave - preprod #2868
Conversation
…t slik at systeme får litt lenger tid.
… Skal ikke vise komponenten for tildeling av oppgave når den har blitt tatt over.
gunnsteingarmo
requested review from
charliemidtlyng,
olekvernberg,
ma10s and
throndi
gunnsteingarmo
changed the title
| }; | ||
|
|
||
| const sjekkOmDetHarGåttMistTiSekunderSidenBehandlingBleOpprettet = ( | ||
| const sjekkOmDetHarGåttMistEttMinuttSidenBehandlingenBleOpprettet = ( |
There was a problem hiding this comment.
Hvorfor trenger vi denne sjekken?
There was a problem hiding this comment.
Når en behandling har blitt opprettet tar det litt tid før ansvarlig saksbehandler blir satt - derfor er ikke sjekken med ansvarlig alltid nok.
| settOppgaveTilSaksbehandler() | ||
| .then(() => { | ||
| hentAnsvarligSaksbehandler.rerun(); | ||
| settHarTildetOppgave(true); |
There was a problem hiding this comment.
Hvorfor trengs denne? Burde ikke hentAnsvarligSaksbehandler.rerun() være nok?
There was a problem hiding this comment.
Den er nødvendig fordi erTilordnetOgInnloggetSaksbehandlerDenSamme ikke blir oppdatert med engang - det finnes sikkert andre måter å gjøre det på, men syntes at denne var enklest.
There was a problem hiding this comment.
Skjønner, tenkte ikke over den...
| @@ -48,6 +48,18 @@ export const Toast: React.FC = () => { | |||
| <AlertSuccess>{toastTilTekst[toast]}</AlertSuccess> | |||
| </ContainerTopMiddle> | |||
| ); | |||
| case EToast.TILDEL_OPPGAVE_VELlYKKET: | |||
| return ( | |||
| <ContainerTopMiddle> | |||
src/frontend/Felles/Toast/Toast.tsx
Outdated
| case EToast.TILDEL_OPPGAVE_FEILET: | ||
| return ( | ||
| <ContainerTopMiddle> | ||
| <AlertError>{toastTilTekst[toast]}</AlertError> |
There was a problem hiding this comment.
Bør vi heller gi ut en fornuftig feilmelding under knappen istedenfor feilmelding i toast?
There was a problem hiding this comment.
Enig - jeg ble litt for ivrig med toasten her 🍞
| navigate(`/behandling/${behandlingId}/arsak-revurdering`); | ||
| }) | ||
| .catch((error) => { | ||
| console.error(error); |
There was a problem hiding this comment.
Hvis vi får inn Ressurs her så kan vi plukke ut feilmeldingen og sette en tilstand som viser den frem.
Men husk å nulle ut feilmelding når man trykker på tildel-knappen :)
…er feilmelding med alert i stedet for toast.
Hvorfor er denne endringen nødvendig? ✨