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

Installation for Issue 887 #106

Merged
merged 8 commits into from
May 21, 2019
Merged

Installation for Issue 887 #106

merged 8 commits into from
May 21, 2019

Conversation

dannylamb
Copy link
Member

This will install everything you need to test the changes for Islandora/documentation#887.

See the issue for testing instructions.

@whikloj
Copy link
Member

whikloj commented May 17, 2019

vagrant up

@@ -1,5 +1,7 @@
---

crayfish_version_tag: dev-format_jsonld_strip
Copy link
Member

Choose a reason for hiding this comment

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

The dev- here caused this to fail for me. Not sure why.

Copy link
Contributor

Choose a reason for hiding this comment

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

This particular line failed for me because the ansible-role-crayfish still referenced the main Crayfish repo in the install task instead of @dannylamb's fork. Once I updated my local copy of the install task it worked fine.

Copy link
Member

@whikloj whikloj left a comment

Choose a reason for hiding this comment

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

Other than the issue above this works for me. By default you get URIs without the ?_format=jsonld but by switching the JsonLD setting and one in Milliner you can add items with that suffix.
👍

@dannylamb
Copy link
Member Author

@seth-shaw-unlv @whikloj This is ready to go now that all applicable PRs have been merged.

@seth-shaw-unlv
Copy link
Contributor

Doing a final spin-up before approval and merge.

@seth-shaw-unlv
Copy link
Contributor

Doing a final spin-up before approval and merge.

It would help if I cleaned out my external roles first... 🤦‍♂️

Copy link
Contributor

@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.

:shipit:

@seth-shaw-unlv seth-shaw-unlv merged commit 0036464 into master May 21, 2019
@seth-shaw-unlv seth-shaw-unlv deleted the issue-887 branch May 21, 2019 19:05
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.

3 participants