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.

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

Export tool : add space sanitizing #18455

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

Conversation

Delphine-L
Copy link
Contributor

@Delphine-L Delphine-L commented Jun 28, 2024

I need to be able to sanitize the blank spaces in a file name before exporting it. Also, the sanitization of "/" was not working when providing a name for the dataset.
This PR adds the space sanitization and fix the "/" sanitization.

How to test the changes?

  • I've included appropriate automated tests.
  • This is a refactoring of components with existing test coverage.
  • Instructions for manual testing are as follows:
    1. Create a file or a collection with spaces in the name
    2. Export using the "Sanitize spaces" option
    3. Verify the exported file contain underscores instead of spaces

License

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

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.

1 participant