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

[17.01] Fix setting UUID on steps when copying steps from another workflow. #3856

Merged

Conversation

Projects
None yet
3 participants
@jmchilton
Copy link
Member

commented Mar 31, 2017

It seems like the client is correctly setting the UUID to None when cloning the step, but the backend is treating the None as a UUID because it was (incorrectly IMO) making a distinction between uuid being absent and being set to None.

Fixes #3845.

Fix setting UUID on steps when copying steps from another workflow.
It seems like the client is correctly setting the UUID to None, but the backend is treating the None as a UUID because it was (incorrectly IMO) making a distinction between ``uuid`` being absent and being set to ``None``.

Fixes #3845.
@dannon

This comment has been minimized.

Copy link
Member

commented Mar 31, 2017

Good catch! +1

@nsoranzo nsoranzo merged commit 268aac2 into galaxyproject:release_17.01 Apr 3, 2017

4 of 5 checks passed

integration test Test scheduled.
Details
api test Build finished. 257 tests run, 0 skipped, 0 failed.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
framework test Build finished. 136 tests run, 0 skipped, 0 failed.
Details
toolshed test Build finished. 580 tests run, 0 skipped, 0 failed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.