From 254ed5ed11aedd99bfaa934827f1765a7a16c289 Mon Sep 17 00:00:00 2001
From: CREDO23 <thierrybakera12@gmail.com>
Date: Fri, 13 Feb 2026 13:46:47 +0200
Subject: [PATCH] Unify SuccessCard for create and update tools to show only
 title and URL

---
 .../components/tool-ui/create-notion-page.tsx | 25 +++++++++----------
 .../components/tool-ui/update-notion-page.tsx | 18 -------------
 2 files changed, 12 insertions(+), 31 deletions(-)

diff --git a/surfsense_web/components/tool-ui/create-notion-page.tsx b/surfsense_web/components/tool-ui/create-notion-page.tsx
index 74004d894..e148c71ba 100644
--- a/surfsense_web/components/tool-ui/create-notion-page.tsx
+++ b/surfsense_web/components/tool-ui/create-notion-page.tsx
@@ -423,28 +423,27 @@ function SuccessCard({ result }: { result: SuccessResult }) {
 					<CheckIcon className="size-4 text-green-500" />
 				</div>
 				<div className="min-w-0 flex-1">
-					<p className=" text-[.8rem] text-muted-foreground">
+					<p className="text-[.8rem] text-muted-foreground">
 						{result.message || "Notion page created successfully"}
 					</p>
 				</div>
 			</div>
 
-			{/* Show details to verify the arguments were used */}
 			<div className="space-y-2 px-4 py-3 text-xs">
 				<div>
-					<span className="font-medium text-muted-foreground">Page ID: </span>
-					<span className="font-mono">{result.page_id}</span>
+					<span className="font-medium text-muted-foreground">Title: </span>
+					<span>{result.title}</span>
 				</div>
-				{result.content_length != null && (
+				{result.url && (
 					<div>
-						<span className="font-medium text-muted-foreground">Content: </span>
-						<span>{result.content_length} characters</span>
-					</div>
-				)}
-				{result.content_preview && (
-					<div>
-						<span className="font-medium text-muted-foreground">Preview: </span>
-						<span className="text-muted-foreground italic">{result.content_preview}</span>
+						<a
+							href={result.url}
+							target="_blank"
+							rel="noopener noreferrer"
+							className="text-primary hover:underline"
+						>
+							Open in Notion
+						</a>
 					</div>
 				)}
 			</div>
diff --git a/surfsense_web/components/tool-ui/update-notion-page.tsx b/surfsense_web/components/tool-ui/update-notion-page.tsx
index 01fe5308f..5bf5a45a9 100644
--- a/surfsense_web/components/tool-ui/update-notion-page.tsx
+++ b/surfsense_web/components/tool-ui/update-notion-page.tsx
@@ -393,19 +393,13 @@ function SuccessCard({ result }: { result: SuccessResult }) {
 				</div>
 			</div>
 
-			{/* Show details to verify the update */}
 			<div className="space-y-2 px-4 py-3 text-xs">
-				<div>
-					<span className="font-medium text-muted-foreground">Page ID: </span>
-					<span className="font-mono">{result.page_id}</span>
-				</div>
 				<div>
 					<span className="font-medium text-muted-foreground">Title: </span>
 					<span>{result.title}</span>
 				</div>
 				{result.url && (
 					<div>
-						<span className="font-medium text-muted-foreground">URL: </span>
 						<a
 							href={result.url}
 							target="_blank"
@@ -416,18 +410,6 @@ function SuccessCard({ result }: { result: SuccessResult }) {
 						</a>
 					</div>
 				)}
-				{result.content_length != null && (
-					<div>
-						<span className="font-medium text-muted-foreground">Content: </span>
-						<span>{result.content_length} characters</span>
-					</div>
-				)}
-				{result.content_preview && (
-					<div>
-						<span className="font-medium text-muted-foreground">Preview: </span>
-						<span className="text-muted-foreground italic">{result.content_preview}</span>
-					</div>
-				)}
 			</div>
 		</div>
 	);