Skip to content

Vueify Data Uploader - #16472

Merged
jmchilton merged 131 commits into
galaxyproject:devfrom
guerler:uploader_default_mixin
Aug 14, 2023
Merged

Vueify Data Uploader#16472
jmchilton merged 131 commits into
galaxyproject:devfrom
guerler:uploader_default_mixin

Conversation

@guerler

@guerler guerler commented Jul 27, 2023

Copy link
Copy Markdown
Contributor

ref: #16407. This PR rewrites the Uploader Interface by removing Backbone and Mixins, and converting all modules to Vue's composition API. Additionally this PR removes the usage of jQuery, and replaces the Select2-field with vue-multiselect. The appearance of the user interface has only been minimally altered, except for the RulesInput builder which has been brought into alignment with other upload types. This new uploader is functionally equivalent to the existing uploader, however FTP files are selected using the Remote Files dialog, instead of the popover table. TypeScript will be applied in a follow-up.

How to test the changes?

(Select all options that apply)

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. [add testing steps and prerequisites here if you didn't write automated tests covering all your changes]

License

  • I agree to license these and all my past contributions to the core galaxy codebase under the MIT license.

@guerler guerler added kind/enhancement area/UI-UX kind/refactoring cleanup or refactoring of existing code, no functional changes labels Jul 27, 2023
@guerler guerler added this to the 23.2 milestone Jul 27, 2023
@guerler
guerler force-pushed the uploader_default_mixin branch 7 times, most recently from 178cda9 to 698442b Compare August 2, 2023 06:17
@guerler
guerler force-pushed the uploader_default_mixin branch 19 times, most recently from 700f993 to cc91fbf Compare August 7, 2023 18:01
@guerler
guerler force-pushed the uploader_default_mixin branch 2 times, most recently from 94e6427 to f1d2bba Compare August 12, 2023 06:53
@guerler
guerler force-pushed the uploader_default_mixin branch from f1d2bba to ecf862e Compare August 12, 2023 07:01
@guerler
guerler force-pushed the uploader_default_mixin branch from 9b104a4 to 7f656d8 Compare August 13, 2023 06:52
@jmchilton
jmchilton merged commit 3a91143 into galaxyproject:dev Aug 14, 2023
@jmchilton

Copy link
Copy Markdown
Member

Really nice, thanks so much!

@jmchilton

Copy link
Copy Markdown
Member

The clipping on the selecting a datatype for all uploads seems a little off.

Screenshot 2023-08-14 at 11 29 04 AM

@guerler
guerler deleted the uploader_default_mixin branch August 14, 2023 16:46
@guerler

guerler commented Aug 14, 2023

Copy link
Copy Markdown
Contributor Author

Thanks a lot for the review @jmchilton. I can follow-up with the selection area visibility.

@jdavcs jdavcs mentioned this pull request Jan 10, 2024
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/UI-UX kind/enhancement kind/refactoring cleanup or refactoring of existing code, no functional changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants