From 330ddb3311c44e44d2fb9105dded3908308687aa Mon Sep 17 00:00:00 2001
From: James Ritchie <james@jamesritchie.co.uk>
Date: Fri, 3 Nov 2023 14:36:33 +0000
Subject: [PATCH] =?UTF-8?q?NoTask=20message=20only=20displays=20if=20the?=
 =?UTF-8?q?=20run=20has=20status=20=E2=80=9CSUCCESS=E2=80=9D?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../webapp/app/components/run/RunOverview.tsx | 21 ++++++++++---------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/apps/webapp/app/components/run/RunOverview.tsx b/apps/webapp/app/components/run/RunOverview.tsx
index f33aa1668..130ecfd0d 100644
--- a/apps/webapp/app/components/run/RunOverview.tsx
+++ b/apps/webapp/app/components/run/RunOverview.tsx
@@ -159,16 +159,17 @@ export function RunOverview({ run, trigger, showRerun, paths }: RunOverviewProps
         <div className="grid h-full grid-cols-2 gap-2">
           <div className="flex flex-col gap-6 overflow-y-auto py-4 pl-4 pr-2 scrollbar-thin scrollbar-track-transparent scrollbar-thumb-slate-700">
             <div>
-              {(run.tasks.length === 0 || run.tasks.every((t) => t.noop)) && (
-                <Callout
-                  variant={"warning"}
-                  to="https://trigger.dev/docs/documentation/concepts/tasks"
-                  className="mb-4"
-                >
-                  This Run completed but it did not use any Tasks – this can cause unpredictable
-                  results. Read the docs to view the solution.
-                </Callout>
-              )}
+              {run.status === "SUCCESS" &&
+                (run.tasks.length === 0 || run.tasks.every((t) => t.noop)) && (
+                  <Callout
+                    variant={"warning"}
+                    to="https://trigger.dev/docs/documentation/concepts/tasks"
+                    className="mb-4"
+                  >
+                    This Run completed but it did not use any Tasks – this can cause unpredictable
+                    results. Read the docs to view the solution.
+                  </Callout>
+                )}
               <Header2 className="mb-2">Trigger</Header2>
               <RunPanel
                 selected={selectedId === "trigger"}