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.

Sign up for GitHub

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

Jump to bottom

Libraries Modernisation: Directory Dataset Picker #18638

Open
wants to merge 18 commits into
base: dev
Choose a base branch
from
Open

Libraries Modernisation: Directory Dataset Picker #18638

wants to merge 18 commits into from

Conversation

itisAliRH
Copy link
Member

@itisAliRH itisAliRH commented Aug 2, 2024 •
edited
Loading

This PR is a follow-up on #18518 and replaces the old dataset picker from user and import directories and path to the library folder. Also, remove the jstree and related files (related #18451)

Libraries.Modernisation.Directory.Dataset.Picker.webm

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. Open a library in Data Libraries
    2. Import file or folder from Dataset > User directory or from Dataset > from import directory Dataset > from path as admin
    3. Select files or folders or past paths to enter

License

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

@github-actions github-actions bot added this to the 24.2 milestone Aug 2, 2024
@itisAliRH itisAliRH force-pushed the libraries-modernization-1-directory-dataset-picker branch 2 times, most recently from 11cf6e3 to f169c1f Compare August 2, 2024 13:41
@itisAliRH itisAliRH marked this pull request as draft August 2, 2024 13:48
@itisAliRH itisAliRH force-pushed the libraries-modernization-1-directory-dataset-picker branch from f169c1f to 3e5bea1 Compare September 16, 2024 11:57
@itisAliRH
🛠️: refactor FontAwesome icon import in FolderTopBar and `FormDrill…
cfaf9be 
…downOption`
@itisAliRH
🛠️: refactor remote files fetching logic in DirectoryDatasetPicker
2cdbbb7
@itisAliRH
✨: use progress bar on importing datasets from directories
b7d9240
@itisAliRH
🛠️: unified dataset addition logic in FolderTopBar
b4e7605
@itisAliRH
🛠️: enhanced DirectoryDatasetPicker by adding a scroll to the items…
caa4d0e 
… list and making the import button dynamic to reflect the number of selected items being import
@itisAliRH
🛠️: use IconDefinition for leafIcon and branchIcon in FormDrill…
d31a760 
…downOption`
@itisAliRH
✨: modified SingleItemSelectors to utilize flexible trackBy and `…
7f622f2 
…label` prop, revised selected item type, and update its usage
@itisAliRH
🛠️: replaced Multiselect with SingleItemSelector, revised data fe…
938e5c9 
…tching logic, improved types, and adjusted ui in `DirectoryDatasetPicker`
@itisAliRH
✨: add no-options alert to FormDrilldown and import missing bootstr…
d74a025 
…ap components
@itisAliRH
🛠️: DirectoryDatasetPicker shows error on fetching options and fix …
bcb9a54 
…errors when there is no option available
@itisAliRH itisAliRH marked this pull request as ready for review September 20, 2024 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area/API area/testing/selenium area/testing area/UI-UX
Projects
None yet
Milestone
24.2
Development

Successfully merging this pull request may close these issues.
1 participant
@itisAliRH