Handle null job.latestRun before runStateColor (MarquezProject#2467)

* Handle null `job.latestRun` before `runStateColor` Signed-off-by: Perttu Salonen <[email protected]> * generate default color with `NEW` `runState` Signed-off-by: Perttu Salonen <[email protected]> * added changelog description Signed-off-by: Perttu Salonen <[email protected]> --------- Signed-off-by: Perttu Salonen <[email protected]> Signed-off-by: Xavier-Cliquennois <[email protected]>
Xavier-Cliquennois · Jul 26, 2023 · 947c203 · 947c203
1 parent 2748c6d
commit 947c203
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git 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
@@ -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>