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

Adds suggestion for drupal/transliterate_filenames #761

Merged
merged 4 commits into from
Feb 13, 2020

Conversation

dannylamb
Copy link

JIRA Ticket: Resolves Islandora/documentation#1309 and Islandora/documentation#1366

What does this Pull Request do?

Pulls in drupal/transliterate_filenames to sanitize file names on upload.

What's new?

A new dependency, that's it. Requires no config or special installation steps.

Interested parties

@Islandora/8-x-committers

@dannylamb dannylamb changed the title Update composer.json Install drupal/transliterate_filenames Feb 13, 2020
@seth-shaw-unlv
Copy link

This looks good. The module itself is relatively simple and could probably be used as a template for an alternate version if a repository wants more control over renaming.

That said, could we not include in islandora.info.yml? That way if a repository wants to use an alternative they don't have to modify islandora's info file? We could still update the playbook to install and enable this module by default.

@dannylamb
Copy link
Author

I'm fine with that. Keep it in the composer.json but enable using Ansible.

@dannylamb dannylamb changed the title Install drupal/transliterate_filenames Adds suggestion for drupal/transliterate_filenames Feb 13, 2020
Copy link

@seth-shaw-unlv seth-shaw-unlv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Checked the composer section of the travis log and saw the suggest statement as expected. 👍

@seth-shaw-unlv seth-shaw-unlv merged commit c83c273 into 8.x-1.x Feb 13, 2020
@seth-shaw-unlv seth-shaw-unlv deleted the issue-1309 branch February 13, 2020 19:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants