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

Pulling in changes for 8.7, setting config, and clearing cache afterwards #118

Merged
merged 5 commits into from
Jun 25, 2019

Conversation

dannylamb
Copy link
Member

GitHub Issues
Resolves Islandora/documentation#1110 and Islandora/documentation#1095 and Islandora/documentation#1144

Supersedes https://github.com/Islandora-Devops/claw-playbook/pull/108/files

Installs 8.7, clears the cache, and pulls in the latest Islandora module, which has a nice refactor to normalize how we're generating urls.

post-install.yml Outdated Show resolved Hide resolved
@dannylamb
Copy link
Member Author

Looks like Travis has been pacified on Islandora/documentation#144. This is good to test. I made content and media, and verified triplestore and fedora indexing. Should probably take a look at a translated piece of content just to be sure I don't introduce any regressions.

@seth-shaw-unlv
Copy link
Member

vagrant up 🚀

@seth-shaw-unlv
Copy link
Member

Something went wrong. The build completed successfully but for some reason our additional media fields, for example field_media_use and field_media_of, are no longer enabled on the form display by default.
Screen Shot 2019-06-21 at 8 17 53 AM

@seth-shaw-unlv
Copy link
Member

They are also disabled in the display settings. Fortunately everything works once I enable them.

post-install.yml Outdated
command: "{{ drush_path }} --root {{ drupal_core_path }} -y en islandora_defaults"
- name: Import feature
command: "{{ drush_path }} --root {{ drupal_core_path }} -y fim --bundle=islandora islandora_defaults"
Copy link
Member

Choose a reason for hiding this comment

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

Re: @seth-shaw-unlv's comments I also got the same behaviour. I re-added this "Import feature" and everything seemed to come up hunky-dorry.

@dannylamb
Copy link
Member Author

dannylamb commented Jun 23, 2019 via email

@whikloj
Copy link
Member

whikloj commented Jun 24, 2019

job != jaerb

@dannylamb
Copy link
Member Author

Alright, good to go here. Sorry about that.

@whikloj
Copy link
Member

whikloj commented Jun 25, 2019

vagrant up

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.

👍

@whikloj
Copy link
Member

whikloj commented Jun 25, 2019

@dannylamb I'm assuming I need to merge Islandora/documentation#144, then you need to update this PR, then I can merge this?

@dannylamb
Copy link
Member Author

@whikloj You got it.

@dannylamb
Copy link
Member Author

@whikloj Updated to point back to dev-8.x-1.x

@whikloj whikloj merged commit 284271e into Islandora-Devops:dev Jun 25, 2019
@seth-shaw-unlv seth-shaw-unlv mentioned this pull request Jun 26, 2019
whikloj pushed a commit to whikloj/claw-playbook that referenced this pull request Dec 18, 2019
…ards (Islandora-Devops#118)

* Pulling in changes for 8.7, setting config, and clearing cache afterwards

* Update post-install.yml

* Update post-install.yml

* Un-derping

* Update drupal.yml
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.

Drupal 8.7 is released
3 participants