Skip to content

Commit

Permalink
Merge pull request #383 from navikt/bug/ikke-slett-soknad
Browse files Browse the repository at this point in the history
Bug/ikke slett soknad
  • Loading branch information
nils-arne authored Feb 5, 2025
2 parents f383420 + ffc3937 commit 02da350
Show file tree
Hide file tree
Showing 7 changed files with 34 additions and 13 deletions.
1 change: 1 addition & 0 deletions assets/locales/en/translation.json
Original file line number Diff line number Diff line change
Expand Up @@ -110,6 +110,7 @@
"avbryt": "Cancel",
"slett": "Delete form"
},
"opptatt": "You can not send in or remove the application until the problems below are solved",
"beskjedOmAvvik": "Have you experienced problems submitting documentation? Please upload the documentation again and press Submit."
},
"filvelger": {
Expand Down
3 changes: 2 additions & 1 deletion assets/locales/nb/translation.json
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,8 @@
"avbryt": "Avbryt",
"slett": "Avbryt og slett"
},
"beskjedOmAvvik": "Får du ikke ettersendt dokumentasjon? Da må du laste opp dokumentasjonen på nytt og trykke Send inn."
"beskjedOmAvvik": "Får du ikke ettersendt dokumentasjon? Da må du laste opp dokumentasjonen på nytt og trykke Send inn.",
"opptatt": "Du kan ikke sende inn søknad eller slette søknad før problemene under er løst"
},
"filvelger": {
"defaultText": "Velg dine filer"
Expand Down
3 changes: 2 additions & 1 deletion assets/locales/nn/translation.json
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,8 @@
"avbryt": "Avbryt",
"slett": "Avbryt og slett"
},
"beskjedOmAvvik": "Får du ikkje ettersendt dokumentasjon? Då må du lasta opp dokumentasjonen på nytt og trykkja Send inn."
"beskjedOmAvvik": "Får du ikkje ettersendt dokumentasjon? Då må du lasta opp dokumentasjonen på nytt og trykkja Send inn.",
"opptatt": "Du kan ikkje senda inn søknad eller sletta søknad før problema under er løyste"
},
"filvelger": {
"defaultText": "Velg dine filer"
Expand Down
16 changes: 15 additions & 1 deletion components/LastOppVedlegg.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -205,8 +205,22 @@ function LastOppVedlegg(props: LastOppVedleggdProps) {
</Button>

<Button
loading={isLoading}
onClick={() => {
openSlettSoknadModal();
if (lastOppVedleggHarFeil) {
setLastOppVedleggValideringfokus(true);
setVisLastOppVedleggFeil(true);
return;
}
setIsLoading(true);
ventPaaLagring()
.then(() => {
openSlettSoknadModal();
})
.catch(() => console.error('Feil oppsto ved lagring, så kan ikke slette søknad'))
.finally(() => {
setIsLoading(false);
});
}}
variant="tertiary"
data-cy="slettSoknadKnapp"
Expand Down
14 changes: 8 additions & 6 deletions components/SideValideringProvider.tsx
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
import React, { createContext, useCallback, useContext, useEffect, useReducer, useRef } from 'react';
import { useTranslation } from 'react-i18next';

import { ErrorSummary } from '@navikt/ds-react';

Expand Down Expand Up @@ -82,7 +83,9 @@ export const SideValideringProvider = ({

useEffect(() => {
if (fokus) {
errorRef?.current && errorRef.current.focus();
if (errorRef.current) {
errorRef.current.focus();
}
setFokus(false);
}
}, [fokus, setFokus]);
Expand All @@ -100,6 +103,9 @@ export const SideValideringProvider = ({
const slettValidering = useCallback((validering: ValideringsType) => {
dispatch({ type: ACTIONS.SLETT, validering });
}, []);

const { t } = useTranslation();

return (
<ValideringsContext.Provider
value={{
Expand All @@ -109,11 +115,7 @@ export const SideValideringProvider = ({
}}
>
{harFeil && visValideringsfeil && (
<ErrorSummary
data-cy="valideringsfeil"
heading="Du må fikse disse feilene før du kan sende inn søknad."
ref={errorRef}
>
<ErrorSummary data-cy="valideringsfeil" heading={t('soknad.opptatt')} ref={errorRef}>
{valideringsFeil.map((validering) => (
<ErrorSummary.Item key={validering.valideringsId} href={'#' + validering.komponentId}>
{validering.melding}
Expand Down
6 changes: 4 additions & 2 deletions components/SoknadModalProvider.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@ import { createContext, useCallback, useContext, useState } from 'react';
import { useTranslation } from 'react-i18next';
import { datoOmXDager, formatertDato } from '../utils/dato';
import { navigerTilMinSide } from '../utils/navigerTilMinSide';
import { isLospost } from '../utils/soknad';
import { useErrorMessageContext } from './ErrorMessageProvider';
import { FellesModal } from './FellesModal';
import { useLagringsProsessContext } from './LagringsProsessProvider';
Expand Down Expand Up @@ -102,7 +101,10 @@ export const SoknadModalProvider = ({ children }: SoknadModalProviderProps) => {
heading={t('modal.slett.tittel')}
open={slettSoknadModal}
setOpen={setSlettSoknadModal}
onAccept={slettSoknad}
onAccept={async () => {
await slettSoknad();
setSlettSoknadModal(false);
}}
acceptButtonText={t('modal.slett.accept')}
cancelButtonText={t('modal.slett.cancel')}
isLoading={lagrer}
Expand Down
4 changes: 2 additions & 2 deletions components/VedleggsListe.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -154,10 +154,10 @@ function VedleggsListe({ soknad, setSoknad }: VedleggsListeProps) {
});
};

const slettSoknad = () => {
const slettSoknad = async () => {
if (lagrerNaa()) return;

nyLagringsProsess(axios.delete(`${publicRuntimeConfig.apiUrl}/frontend/v1/soknad/${soknad?.innsendingsId}`))
await nyLagringsProsess(axios.delete(`${publicRuntimeConfig.apiUrl}/frontend/v1/soknad/${soknad?.innsendingsId}`))
.then(() => {
resetState();
navigerTilMinSide();
Expand Down

0 comments on commit 02da350

Please sign in to comment.