vbatts/quay
Archived
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #2258 from iminoso/builds-null-check

Browse source 
Check for null data on builds table
This commit is contained in:
Ian Minoso 2016-12-22 15:52:44 -05:00 committed by GitHub
commit 82542619a3
1 changed files with 12 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
static/js/directives/components/pages/repo-page/build.tsx
View file

@ -26,12 +26,21 @@ export default class Build extends React.Component<any, any> {
); );
}); });
let buildId: string = element.id.split('-')[0]; let buildId: string = element.id.split('-')[0];
let phase: string = element.phase ? element.phase : 'Cannot retrieve phase';
let started: string = element.started ? element.started : 'Cannot retrieve start date';
let message: string;
if (element.trigger_metadata && element.trigger_metadata.commit_info && element.trigger_metadata.commit_info.message){
message = element.trigger_metadata.commit_info.message;
}
else {
message = 'Cannot retrieve message';
}
buildsTable.push( buildsTable.push(
<tr key={buildId}> <tr key={buildId}>
<td>{element.phase}</td> <td>{phase}</td>
<td>{buildId}</td> <td>{buildId}</td>
<td>{element.trigger_metadata.commit_info.message}</td> <td>{message}</td>
<td>{moment(element.started).format('l')}</td> <td>{moment(started).format('l')}</td>
<td>{tags}</td> <td>{tags}</td>
</tr> </tr>
) )