Skip to content

Commit

Permalink
Revert "Unngå doble serverkall pga <LoadingPanel /> (#5380)" (#5438)
Browse files Browse the repository at this point in the history
This reverts commit ccf5a0c.
  • Loading branch information
hallvardastark authored Dec 12, 2023
1 parent 834e2cc commit e129c8c
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 12 deletions.
3 changes: 0 additions & 3 deletions packages/rest-api-hooks/src/RestApiState.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,3 @@ enum RestApiState {
}

export default RestApiState;

export const isRequestNotDone = (state: RestApiState): boolean =>
state === RestApiState.NOT_STARTED || state === RestApiState.LOADING;
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@ import React, { useCallback, useMemo } from 'react';
import { useLocation } from 'react-router-dom';
import moment from 'moment';

import { RestApiState } from '@k9-sak-web/rest-api-hooks';
import HistorikkSakIndex from '@fpsak-frontend/sak-historikk';
import { KodeverkMedNavn, Historikkinnslag } from '@k9-sak-web/types';
import { LoadingPanel, usePrevious } from '@fpsak-frontend/shared-components';

import { isRequestNotDone } from "@k9-sak-web/rest-api-hooks/src/RestApiState";
import useBehandlingEndret from '../../behandling/useBehandlingEndret';
import { K9sakApiKeys, restApiHooks } from '../../data/k9sakApi';
import { pathToBehandling, createLocationForSkjermlenke } from '../../app/paths';
Expand Down Expand Up @@ -109,9 +109,9 @@ const HistorikkIndex = ({ saksnummer, behandlingId, behandlingVersjon }: OwnProp
);

if (
isRequestNotDone(historikkK9SakState) ||
(skalBrukeFpTilbakeHistorikk && isRequestNotDone(historikkTilbakeState)) ||
(skalBrukeKlageHistorikk && isRequestNotDone(historikkKlageState))
historikkK9SakState === RestApiState.LOADING ||
(skalBrukeFpTilbakeHistorikk && historikkTilbakeState === RestApiState.LOADING) ||
(skalBrukeKlageHistorikk && historikkKlageState === RestApiState.LOADING)
) {
return <LoadingPanel />;
}
Expand Down
11 changes: 6 additions & 5 deletions packages/sak-app/src/fagsak/FagsakIndex.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import {
LoadingPanel,
Punsjstripe,
} from '@fpsak-frontend/shared-components';
import { RestApiState } from '@k9-sak-web/rest-api-hooks';
import { Merknadkode } from '@k9-sak-web/sak-meny-marker-behandling';
import Soknadsperiodestripe from '@k9-sak-web/sak-soknadsperiodestripe';
import {
Expand All @@ -27,7 +28,6 @@ import React, { useCallback, useMemo, useState } from 'react';
import { Navigate, Route, Routes, useLocation } from 'react-router-dom';
// eslint-disable-next-line @typescript-eslint/no-unused-vars
import { QueryClient, QueryClientProvider } from 'react-query';
import { isRequestNotDone } from "@k9-sak-web/rest-api-hooks/src/RestApiState";
import {
behandlingerRoutePath,
erBehandlingValgt,
Expand Down Expand Up @@ -216,13 +216,14 @@ const FagsakIndex = () => {
const erHastesak = merknaderFraLos && merknaderFraLos.merknadKoder?.includes(Merknadkode.HASTESAK);

if (!fagsak) {
if (isRequestNotDone(fagsakState)) {
if (fagsakState === RestApiState.NOT_STARTED || fagsakState === RestApiState.LOADING) {
return <LoadingPanel />;
}
return <Navigate to={pathToMissingPage()} />;
}

const harIkkeHentetfagsakPersonData = isRequestNotDone(fagsakPersonState);
const harIkkeHentetfagsakPersonData =
fagsakPersonState === RestApiState.LOADING || fagsakPersonState === RestApiState.NOT_STARTED;

if (harIkkeHentetfagsakPersonData || !harFerdighentetfagsakRettigheter) {
return <LoadingPanel />;
Expand Down Expand Up @@ -273,7 +274,7 @@ const FagsakIndex = () => {
/>
}
supportContent={() => {
if (isRequestNotDone(personopplysningerState)) {
if (personopplysningerState === RestApiState.LOADING) {
return <LoadingPanel />;
}

Expand All @@ -298,7 +299,7 @@ const FagsakIndex = () => {
return null;
}

if (isRequestNotDone(personopplysningerState)) {
if (personopplysningerState === RestApiState.LOADING) {
return <LoadingPanel />;
}

Expand Down

0 comments on commit e129c8c

Please sign in to comment.