From bc820f17b016718d4ce10c90054f6f6290abf84e Mon Sep 17 00:00:00 2001
From: Nicolas Meienberger <github@thisprops.com>
Date: Wed, 18 Mar 2026 19:55:48 +0100
Subject: [PATCH] refactor: improve frontend code organization

---
 .../modules/backups/components/schedule-summary.tsx    | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/app/client/modules/backups/components/schedule-summary.tsx b/app/client/modules/backups/components/schedule-summary.tsx
index f3353fc7..089b6c5f 100644
--- a/app/client/modules/backups/components/schedule-summary.tsx
+++ b/app/client/modules/backups/components/schedule-summary.tsx
@@ -91,11 +91,6 @@ export const ScheduleSummary = (props: Props) => {
 		handleStopBackup();
 	};
 
-	const backupDetails =
-		schedule.lastBackupStatus === "warning"
-			? (schedule.lastBackupError ?? "Last backup completed with warnings. Check your container logs for more details.")
-			: schedule.lastBackupError;
-
 	return (
 		<div className="space-y-4">
 			<Card className="@container">
@@ -205,7 +200,7 @@ export const ScheduleSummary = (props: Props) => {
 						</p>
 					</div>
 
-					{backupDetails && (schedule.lastBackupStatus === "warning" || schedule.lastBackupStatus === "error") && (
+					{(schedule.lastBackupStatus === "warning" || schedule.lastBackupStatus === "error") && (
 						<div className="@medium:col-span-2 @wide:col-span-4">
 							<Collapsible
 								className={cn("border border-border/50 rounded-lg overflow-hidden", {
@@ -234,7 +229,8 @@ export const ScheduleSummary = (props: Props) => {
 												"text-red-600": schedule.lastBackupStatus === "error",
 											})}
 										>
-											{backupDetails}
+											{schedule.lastBackupError ??
+												"No additional details available. check your container logs for more information."}
 										</p>
 									</div>
 								</CollapsibleContent>