Skip to content

Allow uploading to collections. #3696

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

Merged
merged 1 commit into from
Mar 29, 2017

Conversation

jmchilton
Copy link
Member

@jmchilton jmchilton commented Mar 2, 2017

This builds on collection builder enhancements from open PR #3806.

This adds a new upload tab for "Collections".

  • This has a new option to select "Collection Type" in footer.
  • Collection upload geared toward homogeneous datasets - must specify genome and file format for all files.
  • Add a new button "Build" after all individual datasets have finished their upload to the server.
  • Clicking "Build" launches the appropriate collection builder for selected collection type.
  • The original datasets of the uploads will be hidden by default after the collection is built (though this can be disabled by unchecking an option in the collection builder form).

This includes Selenium web tests for all three collection types as well as for hiding/not hiding the original datasets and the file extension selector.

image
image
image
image
image

@martenson
Copy link
Member

martenson commented Mar 2, 2017

Just a note: to test this you need to rebuild client as this PR does not include the built scripts and css. (might be an oversight @jmchilton ?)

@jmchilton
Copy link
Member Author

@martenson It is intentional - rebuilding the client makes rebasing difficult - but you are correct I should have pointed this out.

@jmchilton jmchilton force-pushed the upload_collection branch 2 times, most recently from 7e9496c to 7d438e4 Compare March 6, 2017 14:44
@jmchilton jmchilton force-pushed the upload_collection branch 2 times, most recently from 6bfc2b8 to 0d6f076 Compare March 24, 2017 14:12
@jmchilton jmchilton changed the title WIP: Upload to collections. Allow uploading to collections. Mar 24, 2017
@jmchilton jmchilton removed the request for review from guerler March 24, 2017 14:57
- New upload tab for "Collections".
  - This has a new option to select "Collection Type" in footer.
  - Collection upload geared toward homogeneous datasets - must specify genome and file format for all files.
  - New button "Build" after all individual datasets have finished their upload to the server.
  - Clicking "Build" launches the appropriate collection builder for selected collection type.
  - By default, uploading a collection hides the original uploaded datasets (can be unchecked).

Implement tests for all uploaded collection types.
@martenson martenson mentioned this pull request Mar 28, 2017
Copy link
Member

@martenson martenson left a comment

Choose a reason for hiding this comment

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

It is fully functional and fairly intuitive.

The UX would benefit from the de-atomization of the upload requests (thus allowing the build button to be available almost instantly) discussed in #1090

@martenson martenson merged commit 777db98 into galaxyproject:dev Mar 29, 2017
@martenson
Copy link
Member

martenson commented Mar 29, 2017

thanks @jmchilton - big step forward in improving collection user experience

@jmchilton
Copy link
Member Author

Yay - thanks for the detailed review and merge @martenson!

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.

2 participants