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

make sure to add trailing slashes, they are important #128

Merged
merged 1 commit into from
Aug 8, 2019

Conversation

Natkeeran
Copy link
Contributor

@Natkeeran Natkeeran commented Aug 1, 2019

Thank you to Danny for troublshotting this issue.

What does this Pull Request do?

This PR resolves an issue introduced with this PR: https://github.com/Islandora-Devops/claw-playbook/commits/dev.
It adds the trailing slash required by the flysystem config.

How should this be tested?

  • Get the PR and Vagrant up
  • Sanity checks

Additional Notes:

  • We need to better understand where to add the trailing slash and where not to!

Interested parties

Tag (@ mention) interested parties or, if unsure, @Islandora-Devops/committers

@dannylamb May want to have a quick look at this one as the current playbook is broken. Thank you.

@dannylamb
Copy link
Member

vagrant up

Copy link
Member

@dannylamb dannylamb left a comment

Choose a reason for hiding this comment

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

:shipit: All good @Natkeeran. Provisions fine and passes basic sanity tests.

@dannylamb dannylamb merged commit 8ea32be into Islandora-Devops:dev Aug 8, 2019
whikloj pushed a commit to whikloj/claw-playbook that referenced this pull request Dec 18, 2019
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