From 9299d2a76f15db4eec72f8a5845c62e12b6022bd Mon Sep 17 00:00:00 2001
From: Daniel Roth <daniel_roth@hotmail.co.uk>
Date: Tue, 3 Mar 2026 11:16:05 +0000
Subject: [PATCH] Only show button if no scenario selected

---
 .../(portfolio)/reporting/RecommendationsOptions.tsx      | 8 --------
 .../[slug]/(portfolio)/reporting/ReportingClientArea.tsx  | 3 +--
 2 files changed, 1 insertion(+), 10 deletions(-)

diff --git a/src/app/portfolio/[slug]/(portfolio)/reporting/RecommendationsOptions.tsx b/src/app/portfolio/[slug]/(portfolio)/reporting/RecommendationsOptions.tsx
index 2180caa..67ea5f6 100644
--- a/src/app/portfolio/[slug]/(portfolio)/reporting/RecommendationsOptions.tsx
+++ b/src/app/portfolio/[slug]/(portfolio)/reporting/RecommendationsOptions.tsx
@@ -30,7 +30,6 @@ import { CSS } from "@dnd-kit/utilities";
 import { ScenarioSummary } from "./types";
 
 export interface RecommendationsOptionsProps {
-  onApply: (value: CategorisationTriggerRequest) => void | Promise<void>;
   disabled?: boolean;
   scenarios: ScenarioSummary[]
   portfolioId: number
@@ -111,7 +110,6 @@ function SortableScenarioItem({
 }
 
 export function RecommendationsOptions({
-  onApply,
   disabled = false,
   scenarios,
   portfolioId
@@ -182,12 +180,6 @@ export function RecommendationsOptions({
       body: JSON.stringify(payload)
     });
 
-    await onApply(payload.scenarios_to_consider ? payload : {
-      portfolio_id: 123,
-      scenarios_to_consider: null,
-      scenario_priority_order: null,
-    });
-
     setIsApplying(false);
     setOpen(false);
   };
diff --git a/src/app/portfolio/[slug]/(portfolio)/reporting/ReportingClientArea.tsx b/src/app/portfolio/[slug]/(portfolio)/reporting/ReportingClientArea.tsx
index e8b2ad4..f22ffa7 100644
--- a/src/app/portfolio/[slug]/(portfolio)/reporting/ReportingClientArea.tsx
+++ b/src/app/portfolio/[slug]/(portfolio)/reporting/ReportingClientArea.tsx
@@ -267,9 +267,8 @@ export function ReportingClientArea({
             </button>
           </div>
         )}
-        {
+        { !selectedScenarioId &&
           <RecommendationsOptions
-            onApply={() => {console.log('Generat Recommendations')}}
             disabled={scenarioBusy}
             scenarios={scenarios}
             portfolioId={portfolioId}