Skip to content

Conversation

@a-asaad
Copy link
Collaborator

@a-asaad a-asaad commented Jun 16, 2025

Motivation and Context

We currently support drag-n-drop for importing flows from JSON files, however it is sometimes more convenient to directly copy-paste a block of JSON text.

Details

Adds an Import Flow JSON option to the OptionsMenu in the header. Clicking the new option opens a text editor modal that can be used for entering arbitrary JSON text.

Screenshots (if appropriate)

json-modal1
json-modal-err

Checklist

  • The PR follows the branch and commit styles outlined in the CONTRIBUTING doc
  • Linting and formatting checks pass locally with the new changes

try {
e.target && importFlow(e.target.result as string)
} catch (e) {
// TODO: Display an error pop-up on failed import
Copy link
Collaborator

Choose a reason for hiding this comment

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

Will this be covered under a different issue?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, I created #7 to cover this task and added a link to the TODO comment.

@a-asaad a-asaad merged commit 69c3b20 into main Jun 16, 2025
1 check passed
@a-asaad a-asaad deleted the import-json-text branch June 16, 2025 23:32
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.

3 participants