Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

empty node row renders the "save" code invalid due to comma #11

Open
sfirke opened this issue Jun 7, 2018 · 1 comment
Open

empty node row renders the "save" code invalid due to comma #11

sfirke opened this issue Jun 7, 2018 · 1 comment
Labels

Comments

@sfirke
Copy link

sfirke commented Jun 7, 2018

This web app is wonderful and I've used it extensively this week. Thanks for creating and maintaining this.

I stumbled into a problem that took a while to pin down. Say you click "add new link" but then realize you don't need it. There's no way to get rid of that blank box row, but it will add a comma to the end of the list of links, e.g.,

...},{"source":13,"target":14,"value":12},]}

Which will cause the subsequent loading of that exported code to silently fail.

Could the export check for blank rows in the link list and ignore them?

@csaladenes
Copy link
Owner

Hey @sfirke - will take a look at this in a bit!

@csaladenes csaladenes added the bug label Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants