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

Rename controlled access terms default configuration: Issue 1132 #112

Merged
merged 3 commits into from May 30, 2019

Conversation

Projects
None yet
2 participants
@seth-shaw-unlv
Copy link
Contributor

commented May 29, 2019

GitHub Issue: Islandora-CLAW/CLAW/issues/1132

What does this Pull Request do?

Replaces 'controlled_access_terms_default_configuration' with 'controlled_access_terms_defaults'.

What's new?

  • Renames 'controlled_access_terms_default_configuration' to 'controlled_access_terms_defaults'.
  • Does this change require documentation to be updated? No
  • Does this change add any new dependencies? No
  • Does this change require any other modifications to be made to the repository (ie. Regeneration activity, etc.)? No
  • Could this change impact execution of existing code? No.

How should this be tested?

  • Apply the PR
  • vagrant up
  • Ensure everything still works.

Additional Notes:

Once this is tested and both Islandora-CLAW/controlled_access_terms/pull/29 and Islandora-CLAW/islandora_defaults/pull/1 are merged we need to revert the islandora/islandora_defaults:dev-issue-1132 in drupal.yml back to islandora/islandora_defaults:dev-8.x-1.x before merging.

Interested parties

@dannylamb, @Islandora-CLAW/committers

@dannylamb

This comment has been minimized.

Copy link
Contributor

commented May 30, 2019

@seth-shaw-unlv Works just fine.

@seth-shaw-unlv

This comment has been minimized.

Copy link
Contributor Author

commented May 30, 2019

@dannylamb you can now merge Islandora-CLAW/islandora_defaults/pull/1 and this one.

@dannylamb dannylamb merged commit c96adef into Islandora-Devops:master May 30, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.