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

Fix post job actions for Database Operation tools, implement change_datatype for collection output #7480

Merged
merged 7 commits into from
Mar 11, 2019

Conversation

mvdbeek
Copy link
Member

@mvdbeek mvdbeek commented Mar 7, 2019

The rename action works, the hide action fails.
Building this to fix #7473.

@mvdbeek mvdbeek force-pushed the fix_build_list_pjas branch 2 times, most recently from aae9020 to 670f190 Compare March 7, 2019 14:28
@dannon
Copy link
Member

dannon commented Mar 7, 2019

@mvdbeek Thanks for this -- I should have added tests for these years ago, would have saved some pain!

The rename action works, the hide action fails.
Building this to fix
galaxyproject#7473.
@galaxybot galaxybot added this to the 19.05 milestone Mar 7, 2019
@mvdbeek
Copy link
Member Author

mvdbeek commented Mar 7, 2019

Looks like it's working, but I think I'll want to test all of the possible PJAs.

`change_datatype` wasn't part of gxformat2, so this also requires
a new release of gxformat2 with galaxyproject/gxformat2#7
This fixes dataset being added to histories when the source
HDA is visible.
@mvdbeek
Copy link
Member Author

mvdbeek commented Mar 8, 2019

This will also fix #3947.

@mvdbeek mvdbeek changed the title Test PJA actions Fix post job actions for Database Operation tools, implement change_datatype for collection output Mar 11, 2019
@mvdbeek mvdbeek marked this pull request as ready for review March 11, 2019 17:28
@jmchilton
Copy link
Member

I assume we should backport this to at least 19.01?

@jmchilton jmchilton merged commit cc62d06 into galaxyproject:dev Mar 11, 2019
@mvdbeek
Copy link
Member Author

mvdbeek commented Mar 11, 2019

Yep, I wouldn't go further back than 19.01, it is more of a new feature than a bug I'd say.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Workflow collection operation "Build list": no hide; no delete intermediate data
4 participants