Fiks datosjekk i for godkjenning av tilskuddsperiode

Når backenden returnerte år -99999 for kanBesluttesFom, som den alltid gjør for første tilskuddsperiode, oppstod det en feil i sjekken for om godkjenn-knappen skulle enables eller ikke. Backenden har nå blitt endret til å returnere et nytt bool- felt "kanBehandles", som gjør at frontenden ikke lenger behøver å gjøre "datomatematikk" for å avgjøre om knappen skal være enabled eller ikke.
navikt · Oct 28, 2024 · 5023696 · 5023696
1 parent e0d714f
commit 5023696
Show file tree

Hide file tree

Showing 6 changed files with 51 additions and 33 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/BeslutterSide/beslutterTilskuddsperioder/BeslutterTilskuddsperioder.tsx b/src/BeslutterSide/beslutterTilskuddsperioder/BeslutterTilskuddsperioder.tsx
@@ -13,7 +13,6 @@ import { Periode, TilskuddsperiodeContext } from '@/BeslutterSide/BeslutterSide'
 import { Returårsaker, TilskuddsPeriode } from '@/types/avtale';
 import { tilskuddsperiodeReturÅrsakTekst } from '@/messages';
 import TilskuddsperiodeReturModal from '@/BeslutterSide/beslutterPanel/TilskuddsperiodeVisAvslag';
-import moment from 'moment';
 
 interface Props {
     startAnimering: () => void;
@@ -51,22 +50,17 @@ const BeslutterTilskuddsPerioder: FunctionComponent<Props> = (props) => {
 
     const hentAvslattInfoTilskuddsperiode = (periode: TilskuddsPeriode): JSX.Element => {
         return (
-            <>
-                <BodyShort size="small">
-                    Tilskuddsperioden ble returnert av
-                    <span className={cls.element('bold')}>{' ' + periode.avslåttAvNavIdent + ' '}</span> den
-                    <span className={cls.element('bold')}>
-                        {' ' + formatterDato(periode.avslåttTidspunkt ?? '', NORSK_DATO_FORMAT) + ' '}
-                    </span>
-                    med følgende årsak(er):
-                    <span className={cls.element('bold')}>
-                        {' ' + hentReturÅrsaker(periode) + ' '}
-                        {''}
-                    </span>
-                    med forklaringen:
-                    <span className={cls.element('bold')}>{' ' + periode.avslagsforklaring}</span>
-                </BodyShort>
-            </>
+            <BodyShort size="small">
+                Tilskuddsperioden ble returnert av
+                <span className={cls.element('bold')}>{' ' + periode.avslåttAvNavIdent + ' '}</span> den
+                <span className={cls.element('bold')}>
+                    {' ' + formatterDato(periode.avslåttTidspunkt ?? '', NORSK_DATO_FORMAT) + ' '}
+                </span>
+                med følgende årsak(er):
+                <span className={cls.element('bold')}>{' ' + hentReturÅrsaker(periode) + ' '}</span>
+                med forklaringen:
+                <span className={cls.element('bold')}>{' ' + periode.avslagsforklaring}</span>
+            </BodyShort>
         );
     };
 
@@ -89,13 +83,11 @@ const BeslutterTilskuddsPerioder: FunctionComponent<Props> = (props) => {
                         </tr>
                     </thead>
                     <tbody>
-                        {avtale.tilskuddPeriode.map((periode, index) => {
+                        {avtale.tilskuddPeriode.map((periode) => {
                             const gjeldende = periode.løpenummer === gjeldendeTilskuddsperiode?.løpenummer;
-                            const kanBesluttes = moment().isSameOrAfter(moment(periode.kanBesluttesFom));
                             return (
-                                <React.Fragment key={index}>
+                                <React.Fragment key={periode.id}>
                                     <tr
-                                        key={index}
                                         className={cls.element(
                                             'tilskuddsperiode-rad',
                                             settStylingForTabellrad(periode),
@@ -138,7 +130,7 @@ const BeslutterTilskuddsPerioder: FunctionComponent<Props> = (props) => {
                                                     {periode.status === 'UBEHANDLET' && (
                                                         <>
                                                             <Button
-                                                                disabled={!enhet || !kanBesluttes}
+                                                                disabled={!enhet || !periode.kanBehandles}
                                                                 onClick={() => {
                                                                     if (enhet) {
                                                                         setGodkjennModalÅpen(true);

diff --git a/src/mocking/arbeidstrening-avtale-mock.ts b/src/mocking/arbeidstrening-avtale-mock.ts
@@ -27,6 +27,7 @@ const tilskuddsPeriode: TilskuddsPeriode[] = [
         avslagsårsaker: new Set<Returårsaker>(),
         lonnstilskuddProsent: 60,
         kanBesluttesFom: '2021-01-01',
+        kanBehandles: true,
         aktiv: true,
     },
 ];

diff --git a/src/mocking/context-mock.ts b/src/mocking/context-mock.ts
@@ -46,6 +46,7 @@ const avtaleInnhold: Avtale | Avtaleinnhold = {
             avslagsårsaker: new Set<Returårsaker>(),
             lonnstilskuddProsent: 60,
             kanBesluttesFom: '2021-01-01',
+            kanBehandles: true,
             aktiv: true,
         },
     ],

diff --git a/src/stories/AvtaleStatuserVeileder.stories.tsx b/src/stories/AvtaleStatuserVeileder.stories.tsx
@@ -42,6 +42,7 @@ const erUfordelt: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -66,6 +67,7 @@ const erUfordelt: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -109,6 +111,7 @@ const annullert: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -133,6 +136,7 @@ const annullert: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -176,6 +180,7 @@ const avbrutt: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -200,6 +205,7 @@ const avbrutt: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -243,6 +249,7 @@ const påbegynt: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -267,6 +274,7 @@ const påbegynt: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -310,6 +318,7 @@ const manglerGodkjenningVeilederHarGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -334,6 +343,7 @@ const manglerGodkjenningVeilederHarGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -377,6 +387,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerHarGodkjentMenIkkeMentor: Avtale =
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2024-05-10',
@@ -402,6 +413,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerHarGodkjentMenIkkeMentor: Avtale =
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -445,6 +457,7 @@ const manglerGodkjenningDeltakerOgArbeidsgiverOgMentorHarGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -470,6 +483,7 @@ const manglerGodkjenningDeltakerOgArbeidsgiverOgMentorHarGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -513,6 +527,7 @@ const manglerGodkjenningDeltakerHarGodkjentMenIkkeArbeidsgiverOgMentor: Avtale =
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -538,6 +553,7 @@ const manglerGodkjenningDeltakerHarGodkjentMenIkkeArbeidsgiverOgMentor: Avtale =
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -581,6 +597,7 @@ const manglerGodkjenningDeltakerHarGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -605,6 +622,7 @@ const manglerGodkjenningDeltakerHarGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -648,6 +666,7 @@ const manglerGodkjenningArbeidsgiverHarGodkjentMenIkkeDeltakerogMentor: Avtale =
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '',
@@ -673,6 +692,7 @@ const manglerGodkjenningArbeidsgiverHarGodkjentMenIkkeDeltakerogMentor: Avtale =
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -716,6 +736,7 @@ const manglerGodkjenningArbeidsgiverHarGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '',
@@ -740,6 +761,7 @@ const manglerGodkjenningArbeidsgiverHarGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -783,6 +805,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerOgMentorHarIkkeGodkjent: Avtale =
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '',
@@ -808,6 +831,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerOgMentorHarIkkeGodkjent: Avtale =
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -851,6 +875,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerHarIkkeGodkjent: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '',
@@ -875,6 +900,7 @@ const manglerGodkjenningArbeidsgiverOgDeltakerHarIkkeGodkjent: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -918,6 +944,7 @@ const klarForOppstartUteforArena: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -942,6 +969,7 @@ const klarForOppstartUteforArena: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -985,6 +1013,7 @@ const klarForOppstartArena: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -1009,6 +1038,7 @@ const klarForOppstartArena: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -1052,6 +1082,7 @@ const gjennomføres: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -1076,6 +1107,7 @@ const gjennomføres: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,
@@ -1119,6 +1151,7 @@ const avsluttet: Avtale = {
             refusjonStatus: undefined,
             aktiv: true,
             kanBesluttesFom: '-999999999-01-01',
+            kanBehandles: true,
         },
     ],
     godkjentAvDeltaker: '2021-08-01',
@@ -1143,6 +1176,7 @@ const avsluttet: Avtale = {
         refusjonStatus: undefined,
         aktiv: true,
         kanBesluttesFom: '-999999999-01-01',
+        kanBehandles: true,
     },
     avtaleInngått: '2021-08-01',
     erAnnullertEllerAvbrutt: true,

diff --git a/src/types/avtale.ts b/src/types/avtale.ts
@@ -295,6 +295,7 @@ export interface TilskuddsPeriode {
     status: TilskuddPeriodeStatus;
     lonnstilskuddProsent: number;
     kanBesluttesFom: string;
+    kanBehandles: boolean;
     aktiv: boolean;
     enhet?: string;
     enhetsnavn?: string;