-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Strip galaxy filename annotation on upload #18561
base: dev
Are you sure you want to change the base?
Strip galaxy filename annotation on upload #18561
Conversation
The forced confirmation dialogue isn't ideal here.
|
Used our existing strip_galaxy_filename_annotation_on_upload.mp4This is still a dialog that though can be dismissed easily; if we still think this is still potentially in the user's way, we could try something else. |
@ahmedhamidawan I like how the message box appropriately pops up within the confirmDialog component. TY for that one! |
78062a7
to
ca6c38d
Compare
I do. imho the stripping should just happen, it is hard to imagine a situation where users don't want to do it. Additionally, handling this through extra modal for multiple files at once would be confusing at least. |
We could just make it automatically stripped and see if anyone complains? Or, maybe strip automatically and put the original name into the file description? Sort of like a log of provenance? I would like this a lot. Maybe something like: Uploaded as [original file name] on DATE-timestamp or make it really log-like Universal-DATE action original-filename 2024000000 Upload Galaxy-filename[whatever].bam I think all actions on a dataset under Edit Attr should be logged there but .. not asking for that, just this :) and leaves room to log more label/metadata actions in later. And -- operations on multiple datasets could have a batch action "simplify dataset names" and this could be one of the actions. I'll stop now! |
- Uses a helper isGalaxyFile() to check for Galaxy file syntax. - Handles the rename gracefully with a confirmation pop-up.
ca6c38d
to
a731902
Compare
Removed any confirmation dialogs and enforced rename if it is potentially a Galaxy file: @jennaj I do like the option you've mentioned of adding the fact that it was renamed to the file's metadata |
Issue #17642
How to test the changes?
(Select all options that apply)
License