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

Basebox issue-28 #125

Merged
merged 2 commits into from
Jul 25, 2017
Merged

Conversation

lutaylor
Copy link
Contributor

@lutaylor lutaylor commented Jun 1, 2017

JIRA Ticket: Islandora-Labs/islandora_vagrant_base_box#28 (no JIRA just issue in queue)

What does this Pull Request do?

Updates variables to match PR in basebox Islandora-Labs/islandora_vagrant_base_box#29

What's new?

New Solr, FITS, and Solr home.

Updated variables that are reflected in Islandora-Labs/islandora_vagrant_base_box#29

How should this be tested?

Make sure you test this with Islandora-Labs/islandora_vagrant_base_box#29.

Additional Notes:

Interested parties

@DonRichards
@Natkeeran

@ruebot
Copy link
Contributor

ruebot commented Jun 1, 2017

FITS 1.1.1 came out May 30th.

@jordandukart
Copy link

jordandukart commented Jun 1, 2017

Thanks for pointing that out @ruebot and @lutaylor for all of his gracious hard work :)!

@lutaylor
Copy link
Contributor Author

lutaylor commented Jun 1, 2017

Updated to FITS 1.1.1 as per @ruebot comment.

@DonRichards
Copy link
Member

Testing the base box currently. As soon as that passes and the PR is merged I'll test this one.

@lutaylor
Copy link
Contributor Author

Having any luck?

@DonRichards
Copy link
Member

OK, looks good.

Copy link
Member

@DonRichards DonRichards left a comment

Choose a reason for hiding this comment

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

Works and looks good. Thanks for the PR

@DonRichards DonRichards merged commit f0667e1 into Islandora-Labs:master Jul 25, 2017
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.

4 participants