fix: only show list of instalments if application is in payment

City-of-Helsinki · Nov 12, 2024 · 1ed681a · 1ed681a
1 parent eb0ac9b
commit 1ed681a
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 10 deletions.
diff --git a/frontend/benefit/handler/src/components/applicationList/HandlerIndex.tsx b/frontend/benefit/handler/src/components/applicationList/HandlerIndex.tsx
@@ -3,6 +3,7 @@ import {
   APPLICATION_LIST_TABS,
 } from 'benefit/handler/constants';
 import FrontPageProvider from 'benefit/handler/context/FrontPageProvider';
+import { Application } from 'benefit/handler/types/application';
 import { APPLICATION_STATUSES, BATCH_STATUSES } from 'benefit-shared/constants';
 import { ApplicationListItemData } from 'benefit-shared/types/application';
 import { LoadingSpinner, Tabs } from 'hds-react';
@@ -34,11 +35,20 @@ const isBatchStatusHandlingComplete = (batchStatus: BATCH_STATUSES): boolean =>
     BATCH_STATUSES.REJECTED_BY_TALPA,
   ].includes(batchStatus);
 
-const isAcceptedOrRejected = (status: APPLICATION_STATUSES): boolean =>
+export const isAcceptedOrRejected = (status: APPLICATION_STATUSES): boolean =>
   [APPLICATION_STATUSES.ACCEPTED, APPLICATION_STATUSES.REJECTED].includes(
     status
   );
 
+export const isInPayment = (
+  application: ApplicationListItemData | Application
+): boolean =>
+  [APPLICATION_STATUSES.ACCEPTED].includes(application.status) &&
+  !isString(application.batch) &&
+  [BATCH_STATUSES.DECIDED_ACCEPTED, BATCH_STATUSES.REJECTED_BY_TALPA].includes(
+    application?.batch?.status
+  );
+
 const HandlerIndex: React.FC<ApplicationListProps> = ({
   layoutBackgroundColor,
   list = [],
@@ -57,14 +67,6 @@ const HandlerIndex: React.FC<ApplicationListProps> = ({
       | 'instalments'
   ): string => t(`${translationBase}.${headingStatus}`);
 
-  const isInPayment = (application: ApplicationListItemData): boolean =>
-    [APPLICATION_STATUSES.ACCEPTED].includes(application.status) &&
-    !isString(application.batch) &&
-    [
-      BATCH_STATUSES.DECIDED_ACCEPTED,
-      BATCH_STATUSES.REJECTED_BY_TALPA,
-    ].includes(application?.batch?.status);
-
   const getTabCountPending = (): number =>
     list.filter(
       (app: ApplicationListItemData) =>

diff --git a/...it/handler/src/components/applicationReview/handlingView/DecisionCalculationAccordion.tsx b/...it/handler/src/components/applicationReview/handlingView/DecisionCalculationAccordion.tsx
@@ -27,6 +27,7 @@ import { formatFloatToCurrency } from 'shared/utils/string.utils';
 import { useTheme } from 'styled-components';
 
 import { renderInstalmentTagPerStatus } from '../../applicationList/ApplicationListForInstalments';
+import { isInPayment } from '../../applicationList/HandlerIndex';
 import {
   $CalculatorTableHeader,
   $CalculatorTableRow,
@@ -158,7 +159,7 @@ const DecisionCalculationAccordion: React.FC<Props> = ({ data }) => {
           </$GridCell>
         </Accordion>
       </$DecisionCalculatorAccordion>
-      {data.pendingInstalment && (
+      {data.pendingInstalment && isInPayment(data) && (
         <$DecisionCalculatorAccordion>
           <$DecisionCalculatorAccordionIconContainer aria-hidden="true">
             <IconBagCogwheel />