-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Conversation
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 ?) |
@martenson It is intentional - rebuilding the client makes rebasing difficult - but you are correct I should have pointed this out. |
7e9496c
to
7d438e4
Compare
6bfc2b8
to
0d6f076
Compare
0d6f076
to
b524a88
Compare
- 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.
b524a88
to
606c329
Compare
There was a problem hiding this 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
thanks @jmchilton - big step forward in improving collection user experience |
Yay - thanks for the detailed review and merge @martenson! |
This builds on collection builder enhancements from open PR #3806.
This adds a new upload tab for "Collections".
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.