Skip to content

Commit

Permalink
fix: build log should always show log (#2840)
Browse files Browse the repository at this point in the history
  • Loading branch information
Andrewnt219 committed Feb 5, 2022
1 parent 1cda631 commit da1041d
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions src/api/status/public/js/build-log/build-header.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,9 @@ function renderSender(sender) {
buildSenderImg.src = sender.avatar_url;
}

function renderSha(compare, after) {
function renderSha(compare, sha) {
buildGitSHA.href = compare;
buildGitSHA.innerText = after.substring(0, 7);
buildGitSHA.innerText = sha.substring(0, 7);
}

function renderLoadingAnimation(code) {
Expand All @@ -47,8 +47,8 @@ function renderLoadingAnimation(code) {
}
}

function renderBuildInfo({ isCurrent, githubData, startedDate, stoppedDate, code }) {
const { sender, after, compare } = githubData;
function renderBuildInfo({ isCurrent, githubData, startedDate, stoppedDate, code, sha }) {
const { sender, compare } = githubData;

if (buildHeaderInfo.hidden) {
buildHeaderInfo.hidden = false;
Expand All @@ -60,7 +60,7 @@ function renderBuildInfo({ isCurrent, githubData, startedDate, stoppedDate, code

buildHeaderTitle.innerHTML = '';
renderSender(sender);
renderSha(compare, after);
renderSha(compare, sha);
renderBuildTimeInfo(startedDate, stoppedDate);
renderLoadingAnimation(code);
}
Expand Down

0 comments on commit da1041d

Please sign in to comment.