Skip to content

Conversation

@colinthebomb1
Copy link
Contributor

@colinthebomb1 colinthebomb1 commented Sep 21, 2025

Description

Fixes the incorrect JSON structure and formatting when downloading workflows from the hub's workflow list. Previously, the downloaded JSON had operators and other workflow properties incorrectly nested inside a content object, which prevented the workflow from being re-uploaded and loaded properly. Workflow metadata was also removed from the download to keep the JSON consistent with workflow downloads using the "export workflow" button within an individual workflow canvas.

Incorrect JSON Download Button:

Screenshot_2025-09-18_131519

Changes:

Edited the downloadWorkflow function in download.service.ts to correctly format the JSON file and bring workflow properties nested inside the content object up a level, as well as remove metadata from the download.

Issues

Closes #3750

@chenlica
Copy link
Contributor

@mengw15 Please review it. After that, ask @aglinxinyuan to review it.

Copy link
Contributor

@mengw15 mengw15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe upload a screenshot of the frontend page might make the pr clearer (letting people know which export has issue)

Copy link
Contributor

@mengw15 mengw15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

left one comment

Copy link
Contributor

@mengw15 mengw15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@aglinxinyuan aglinxinyuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@aglinxinyuan aglinxinyuan enabled auto-merge (squash) September 24, 2025 01:56
@aglinxinyuan aglinxinyuan merged commit 5b1bba1 into apache:main Sep 29, 2025
17 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Downloading workflow from the hub produces incorrect JSON

4 participants