Просмотр исходного кода

Catch cron expression parsing error and display different message

jnfrati 4 лет назад
Родитель
Сommit
802e5be729
1 измененных файлов с 12 добавлено и 5 удалено
  1. 12 5
      dashboard/src/main/home/cluster-dashboard/expanded-chart/ExpandedJobChart.tsx

+ 12 - 5
dashboard/src/main/home/cluster-dashboard/expanded-chart/ExpandedJobChart.tsx

@@ -152,6 +152,17 @@ export const ExpandedJobChartFC: React.FC<{
       timeStyle: "long",
       timeStyle: "long",
     });
     });
 
 
+    let runDescription = "";
+
+    try {
+      runDescription = `Runs ${CronPrettifier.toString(
+        chart?.config?.schedule.value
+      ).toLowerCase()} UTC`;
+    } catch (error) {
+      runDescription =
+        "An unexpected error happened while trying to parse the cron expression.";
+    }
+
     if (currentTab === "jobs") {
     if (currentTab === "jobs") {
       return (
       return (
         <TabWrapper>
         <TabWrapper>
@@ -186,11 +197,7 @@ export const ExpandedJobChartFC: React.FC<{
           {chart?.config?.schedule?.enabled ? (
           {chart?.config?.schedule?.enabled ? (
             <RunsDescription>
             <RunsDescription>
               <i className="material-icons">access_time</i>
               <i className="material-icons">access_time</i>
-              Runs{" "}
-              {CronPrettifier.toString(
-                chart?.config?.schedule.value
-              ).toLowerCase()}{" "}
-              UTC
+              {runDescription}
               <Dot
               <Dot
                 style={{
                 style={{
                   color: "#ffffff88",
                   color: "#ffffff88",