From a93b554f8b9a004dc8f28a1a479ff6d9537b3290 Mon Sep 17 00:00:00 2001
From: Son Yen Nguyen <nyenson1997@gmail.com>
Date: Wed, 17 Jan 2024 00:22:35 +0700
Subject: [PATCH] Feat: Update cron parser human readable string (#847)

* Update cron parser human readable string

* Create hot-cobras-flow.md

---------

Co-authored-by: Matt Aitken <matt@mattaitken.com>
---
 .changeset/hot-cobras-flow.md                  |  5 +++++
 apps/webapp/app/components/jobs/JobsTable.tsx  | 10 +++++-----
 packages/trigger-sdk/src/triggers/scheduled.ts |  6 +++++-
 3 files changed, 15 insertions(+), 6 deletions(-)
 create mode 100644 .changeset/hot-cobras-flow.md

diff --git a/.changeset/hot-cobras-flow.md b/.changeset/hot-cobras-flow.md
new file mode 100644
index 0000000000..0da799c89a
--- /dev/null
+++ b/.changeset/hot-cobras-flow.md
@@ -0,0 +1,5 @@
+---
+"@trigger.dev/sdk": patch
+---
+
+Make it clear that schedules are UTC by appending "UTC" to the end.
diff --git a/apps/webapp/app/components/jobs/JobsTable.tsx b/apps/webapp/app/components/jobs/JobsTable.tsx
index f785fba03a..0b52ef6f13 100644
--- a/apps/webapp/app/components/jobs/JobsTable.tsx
+++ b/apps/webapp/app/components/jobs/JobsTable.tsx
@@ -49,13 +49,13 @@ export function JobsTable({ jobs, noResultsText }: { jobs: ProjectJob[]; noResul
               <TableRow key={job.id} className="group">
                 <TableCell to={path}>
                   <span className="flex items-center gap-2">
-                    <NamedIcon name={job.event.icon} className="h-8 w-8" />
+                    <NamedIcon name={job.event.icon} className="w-8 h-8" />
                     <LabelValueStack
                       label={job.title}
                       value={
                         job.dynamic ? (
                           <span className="flex items-center gap-0.5">
-                            <NamedIcon name="dynamic" className="h-4 w-4" />{" "}
+                            <NamedIcon name="dynamic" className="w-4 h-4" />{" "}
                             <span className="uppercase">Dynamic:</span> {job.event.title}
                           </span>
                         ) : (
@@ -75,9 +75,9 @@ export function JobsTable({ jobs, noResultsText }: { jobs: ProjectJob[]; noResul
                       key={integration.key}
                       button={
                         <div className="relative">
-                          <NamedIcon name={integration.icon} className="h-6 w-6" />
+                          <NamedIcon name={integration.icon} className="w-6 h-6" />
                           {integration.setupStatus === "MISSING_FIELDS" && (
-                            <NamedIcon name="error" className="absolute -left-1 -top-1 h-4 w-4" />
+                            <NamedIcon name="error" className="absolute w-4 h-4 -left-1 -top-1" />
                           )}
                         </div>
                       }
@@ -99,7 +99,7 @@ export function JobsTable({ jobs, noResultsText }: { jobs: ProjectJob[]; noResul
                   {job.properties && (
                     <SimpleTooltip
                       button={
-                        <div className="flex max-w-[200px] items-start justify-start gap-5 truncate">
+                        <div className="flex max-w-[300px] items-start justify-start gap-5 truncate">
                           {job.properties.map((property, index) => (
                             <LabelValueStack
                               key={index}
diff --git a/packages/trigger-sdk/src/triggers/scheduled.ts b/packages/trigger-sdk/src/triggers/scheduled.ts
index 5ca886de0e..fcfd840873 100644
--- a/packages/trigger-sdk/src/triggers/scheduled.ts
+++ b/packages/trigger-sdk/src/triggers/scheduled.ts
@@ -84,9 +84,13 @@ export class CronTrigger implements Trigger<ScheduledEventSpecification> {
   constructor(private options: CronOptions) {}
 
   get event() {
+    /**
+     * We need to concat `(UTC)` string at the end of the human readable string to avoid confusion 
+     * with execution time/last run of a job in the UI dashboard which is displayed in local time.
+     */
     const humanReadable = cronstrue.toString(this.options.cron, {
       throwExceptionOnParseError: false,
-    });
+    }).concat(" (UTC)");
 
     return {
       name: "trigger.scheduled",