Other Relevant Links (Google Groups discussion, related pull requests,
Release pull requests, etc.)
What does this Pull Request do?
Fleshes out config schema.
What's new?
A number of new config schemas being defined. Later exports may end up changing the types of values stored in exported YAML; for example: If something is now defined as a boolean, it should now be exported as a true or false whereas before it would've likely been the integers 1 or 0... shouldn't really have any effect, just might cause a bit of noise in diffs as it happens.
Does this change add any new dependencies? No.
Does this change require any other modifications to be made to the repository
(i.e. Regeneration activity, etc.)? No.
Could this change impact execution of existing code? Unlikely.
How should this be tested?
Install and enable the config_inspector module
Either:
Run:
drush config:inspect --only-error --detail
or:
Looks at the reports at /admin/reports/config-inspector
to note the number of config errors present from the plugins defined in Islandora.
Grab the code
Clear cache
Re-run/Look at the config_inspector output, and see that the number of errors should be reduced.
Documentation Status
Does this change existing behaviour that's currently documented?
Does this change require new pages or sections of documentation?
Who does this need to be documented for?
Associated documentation pull request(s): or documentation issue
Additional Notes:
Any additional information that you think would be helpful when reviewing this
PR.
Interested parties
Tag (@ mention) interested parties or, if unsure, @Islandora/8-x-committers
GitHub Issue: https://github.com/Islandora/documentation/issues/2164
What does this Pull Request do?
Fleshes out config schema.
What's new?
A number of new config schemas being defined. Later exports may end up changing the types of values stored in exported YAML; for example: If something is now defined as a
boolean
, it should now be exported as atrue
orfalse
whereas before it would've likely been the integers1
or0
... shouldn't really have any effect, just might cause a bit of noise in diffs as it happens.How should this be tested?
config_inspector
moduleEither:
or:
/admin/reports/config-inspector
to note the number of config errors present from the plugins defined in Islandora.
Documentation Status
Additional Notes:
Any additional information that you think would be helpful when reviewing this PR.
Interested parties
Tag (@ mention) interested parties or, if unsure, @Islandora/8-x-committers