Commit b560d954 authored by Jose Vargas's avatar Jose Vargas

Fix specs and change utility class

parent 7d6d46e6
...@@ -48,7 +48,7 @@ export default { ...@@ -48,7 +48,7 @@ export default {
return this.job.status && this.job.recoverable ? 'primary' : 'secondary'; return this.job.status && this.job.recoverable ? 'primary' : 'secondary';
}, },
hasArtifact() { hasArtifact() {
return Boolean(this.job?.artifact?.browse_path); return !isEmpty(this.job.artifact);
}, },
hasTriggers() { hasTriggers() {
return !isEmpty(this.job.trigger); return !isEmpty(this.job.trigger);
...@@ -103,7 +103,6 @@ export default { ...@@ -103,7 +103,6 @@ export default {
category="tertiary" category="tertiary"
class="gl-md-display-none gl-ml-2" class="gl-md-display-none gl-ml-2"
icon="chevron-double-lg-right" icon="chevron-double-lg-right"
data-testid="sidebar-build-toggle"
@click="toggleSidebar" @click="toggleSidebar"
/> />
</div> </div>
...@@ -114,7 +113,7 @@ export default { ...@@ -114,7 +113,7 @@ export default {
category="secondary" category="secondary"
variant="confirm" variant="confirm"
:href="job.new_issue_path" :href="job.new_issue_path"
class="float-left mr-2" class="float-left gl-mr-2"
data-testid="job-new-issue" data-testid="job-new-issue"
>{{ $options.i18n.newIssue }} >{{ $options.i18n.newIssue }}
</gl-button> </gl-button>
......
---
title: fix artifact section on the job sidebar always showing up
merge_request: 55685
author:
type: other
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment