Skip to content

Commit 0f96962

Browse files
export workflow history using correct url (#543)
* add origin to export workflow history * update test case for export url to include origin --------- Co-authored-by: Assem Hafez <[email protected]>
1 parent df73ace commit 0f96962

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

client/containers/workflow-history/component.vue

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,6 +88,7 @@ export default {
8888
'workflowHistoryEventHighlightList',
8989
'workflowHistoryEventHighlightListEnabled',
9090
'workflowId',
91+
'origin',
9192
],
9293
created() {
9394
this.onResizeWindow = debounce(() => {
@@ -399,7 +400,7 @@ export default {
399400
>
400401
<a
401402
class="export"
402-
:href="baseAPIURL + '/export'"
403+
:href="origin + baseAPIURL + '/export'"
403404
:download="exportFilename"
404405
>Export</a
405406
>

client/containers/workflow-history/connector.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,11 @@
2121

2222
import { connect } from 'vuex-connect';
2323
import { WORKFLOW_EXECUTION_PENDING_TASK_COUNT } from '../workflow/getter-types';
24+
import { DOMAIN_CROSS_ORIGIN } from '../domain/getter-types';
2425

2526
const gettersToProps = {
2627
pendingTaskCount: WORKFLOW_EXECUTION_PENDING_TASK_COUNT,
28+
origin: DOMAIN_CROSS_ORIGIN,
2729
};
2830

2931
const stateToProps = {

client/test/workflow.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -577,7 +577,7 @@ describe('Workflow', () => {
577577

578578
exportEl.should.have.attr(
579579
'href',
580-
'/api/domains/ci-test/workflows/email-daily-summaries/emailRun1/export'
580+
'http://localhost:8090/api/domains/ci-test/workflows/email-daily-summaries/emailRun1/export'
581581
);
582582
exportEl.should.have.attr(
583583
'download',

0 commit comments

Comments
 (0)