From a972c048d581962cbb77e7390cc7475dc0102f44 Mon Sep 17 00:00:00 2001
From: Perttu Salonen <perttu.salonen@gmail.com>
Date: Mon, 17 Apr 2023 11:32:52 +0300
Subject: [PATCH] Handle null `job.latestRun` before `runStateColor` (#2467)

* Handle null `job.latestRun` before `runStateColor`

Signed-off-by: Perttu Salonen <perttu.salonen@gmail.com>

* generate default color with `NEW` `runState`

Signed-off-by: Perttu Salonen <perttu.salonen@gmail.com>

* added changelog description

Signed-off-by: Perttu Salonen <perttu.salonen@gmail.com>

---------

Signed-off-by: Perttu Salonen <perttu.salonen@gmail.com>
---
 CHANGELOG.md                 | 4 ++++
 web/src/routes/jobs/Jobs.tsx | 4 ++--
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index a52f96e6d7..eb6fcb1bb1 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## [Unreleased](https://github.com/MarquezProject/marquez/compare/0.32.0...HEAD)
 
+### Fixed
+
+* UI: better handling of null job latestRun for Jobs page [#2467](https://github.com/MarquezProject/marquez/pull/2467) [@perttus](https://github.com/perttus)
+
 ### Added
 
 * Support `inputFacets` and `outputFacets` from Openlineage specificatio [`#2417`](https://github.com/MarquezProject/marquez/pull/2417) [@pawel-big-lebowski]( https://github.com/pawel-big-lebowski)  
diff --git a/web/src/routes/jobs/Jobs.tsx b/web/src/routes/jobs/Jobs.tsx
index 74704d3bab..e97ffcfed3 100644
--- a/web/src/routes/jobs/Jobs.tsx
+++ b/web/src/routes/jobs/Jobs.tsx
@@ -118,8 +118,8 @@ class Jobs extends React.Component<JobsProps> {
                           </TableCell>
                           <TableCell key={i18next.t('jobs_route.latest_run_col')} align='left'>
                             <MqStatus
-                              color={runStateColor(job.latestRun.state)}
-                              label={job.latestRun.state}
+                              color={job.latestRun && runStateColor(job.latestRun.state || 'NEW')}
+                              label={job.latestRun && job.latestRun.state ? job.latestRun.state : 'N/A'}
                             />
                           </TableCell>
                         </TableRow>