Skip to content

Commit

Permalink
fix(3227): Add feature flag for download all artifacts (#1202)
Browse files Browse the repository at this point in the history
  • Loading branch information
tkyi authored Oct 23, 2024
1 parent ea6d4ca commit f10e9b1
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 9 deletions.
1 change: 1 addition & 0 deletions app/components/artifact-preview/component.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ export default Component.extend({
router: service(),
iframeUrl: '',
iframeId: '',
allowDownloadDir: ENV.APP.DOWNLOAD_ARTIFACT_DIR,

init() {
this._super(...arguments);
Expand Down
20 changes: 11 additions & 9 deletions app/components/artifact-preview/template.hbs
Original file line number Diff line number Diff line change
@@ -1,13 +1,15 @@
<div>
{{#if (eq this.selectedArtifact "/")}}
<BsButton
@type="primary"
@icon="glyphicon glyphicon-download"
@onClick={{action "downloadAll"}}
title="Download all artifacts as ZIP file"
>
Download All
</BsButton>
{{#if this.allowDownloadDir}}
{{#if (eq this.selectedArtifact "/")}}
<BsButton
@type="primary"
@icon="glyphicon glyphicon-download"
@onClick={{action "downloadAll"}}
title="Download all artifacts as ZIP file"
>
Download All
</BsButton>
{{/if}}
{{else}}
<BsButton
@type="primary"
Expand Down
1 change: 1 addition & 0 deletions config/environment.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,7 @@ module.exports = environment => {
RELEASE_VERSION: 'stable',
DOWNTIME_JOBS: true,
SHOW_AVATAR: true,
DOWNLOAD_ARTIFACT_DIR: true,
FEEDBACK_HOSTNAME: '',
FEEDBACK_SCRIPT: '',
FEEDBACK_CONFIG: ''
Expand Down

0 comments on commit f10e9b1

Please sign in to comment.