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

[20.09] Fix Workflow node input terminal not updating #10626

Merged
merged 1 commit into from Nov 5, 2020

Conversation

@mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Nov 4, 2020

when a new input is selected.
This is easier to describe with a gif:
switch labels

Fixes #10622

This fixes a silent error when hitting
`this.outputTerminals[name].$el.remove();`

This seems to not be necessary, but it is super weird that there is no
error logged. I just found it by stepping through this line by line in
the debugger.

This fixes galaxyproject#10622
@mvdbeek mvdbeek added this to the 20.09 milestone Nov 5, 2020
@jmchilton jmchilton requested a review from guerler Nov 5, 2020
@mvdbeek mvdbeek force-pushed the workflow_editor_fixes branch from 7aa8868 to 6e3e91b Nov 5, 2020
guerler
guerler approved these changes Nov 5, 2020
@guerler guerler merged commit 844c4e5 into galaxyproject:release_20.09 Nov 5, 2020
15 of 19 checks passed
Loading
@guerler
Copy link
Contributor

@guerler guerler commented Nov 5, 2020

@mvdbeek Thank you for fixing this!

Loading

@nsoranzo nsoranzo deleted the workflow_editor_fixes branch Nov 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants