diff options
| author | Felix Edel <felix.edel@bmw.de> | 2021-10-20 09:38:32 +0200 |
|---|---|---|
| committer | Felix Edel <felix.edel@bmw.de> | 2021-10-20 09:38:32 +0200 |
| commit | fa8e1a7c95b038cfda93b6507813b27c438759a2 (patch) | |
| tree | 453aee9fe9375b3d72a7fac4f757671c2be42f7e /web/src/containers/charts/GanttChart.jsx | |
| parent | 55242af7cbb7a81aab1a16760b114c876083d7fa (diff) | |
| download | zuul-fa8e1a7c95b038cfda93b6507813b27c438759a2.tar.gz | |
UI: Fix gant chart on buildset page for missing timestamps
Currently, the "Builds Timeline" gant chart on the buildset result page
is broken when at least one build doesn't provide the required time
information (e.g. SKIPPED builds).
index.js:1 Warning: Received NaN for the `x1` attribute. If this is
expected, cast the value to a string.
in line (created by Line)
in Line
in LineSegment
in g
in g
in VictoryAxis (created by ChartAxis)
in ChartAxis (at GanttChart.jsx:88)
...
To fix this, we fill the missing timestamp information with 0 values.
Change-Id: I3ec6a36a3cfcf818127b827543fd7b8ecf5d785a
Diffstat (limited to 'web/src/containers/charts/GanttChart.jsx')
| -rw-r--r-- | web/src/containers/charts/GanttChart.jsx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/web/src/containers/charts/GanttChart.jsx b/web/src/containers/charts/GanttChart.jsx index dfbf3b6aa..5ac065fce 100644 --- a/web/src/containers/charts/GanttChart.jsx +++ b/web/src/containers/charts/GanttChart.jsx @@ -43,8 +43,8 @@ function BuildsetGanttChart(props) { const data = sortedByStartTime.map((build) => { return { x: build.job_name, - y0: (moment.utc(build.start_time).tz(timezone) - origin) / 1000, - y: (moment.utc(build.end_time).tz(timezone) - origin) / 1000, + y0: build.start_time ? (moment.utc(build.start_time).tz(timezone) - origin) / 1000 : 0, + y: build.end_time ? (moment.utc(build.end_time).tz(timezone) - origin) / 1000 : 0, result: build.result, started: moment.utc(build.start_time).tz(timezone).format('YYYY-MM-DD HH:mm:ss'), ended: moment.utc(build.end_time).tz(timezone).format('YYYY-MM-DD HH:mm:ss'), |