This is a sub-issue of #1910624: [META] Introduce and complete configuration schemas in all of core.
Problem/motivation
#1866610: Introduce Kwalify-inspired schema format for configuration introduced the idea of config schema. The changelog leads to (hopefully extensive) documentation on the format at http://drupal.org/node/1905070. While there are little cleanups planned for the format overall, the current format is a result of months of back and forths, so it should be perfectly fine to apply it more widely to core.
Proposed solution
Create a configuration schema for openid module.
Schema in place
Schema not yet in place
openid.settings.yml
Comment | File | Size | Author |
---|---|---|---|
#13 | 1919188-openid-schema.png | 24.53 KB | sandipmkhairnar |
#10 | 1919188-openid-schema-10.patch | 472 bytes | rvilar |
#8 | 1919188-openid-schema-8.patch | 473 bytes | rvilar |
#6 | 1919188-openid-schema-6.patch | 474 bytes | rvilar |
#4 | 1919188-openid-schema-4.patch | 651 bytes | charmi |
Comments
Comment #1
vijaycs85Adding schema file...
Comment #3
vijaycs85#1: 1919188-openid-schema-1.patch queued for re-testing.
Comment #4
charmi CreditAttribution: charmi commentedAdded code and styles in openid module.
Comment #6
rvilarAttached a new patch to fix the test
Comment #7
vijaycs85Thanks @rvilar, here is some minor label changes:
Can we change "Openid text settings" to "Openid settings"
and "proxy resolver for XRI" for this?
Comment #8
rvilarNew patch with label modifications
Comment #9
vijaycs85Can we remove ':' here please?
Also thinking 'Openid settings' might need to be 'OpenID settings'?, but we can leave that to @YesCT or @Gabor.
Comment #10
rvilarNew patch
Comment #11
vijaycs85for some reasons, I don't see fields in config_inspector.
Comment #12
rvilarI see them without any problems.
Comment #13
sandipmkhairnar CreditAttribution: sandipmkhairnar commentedI have reviewed the patche and its working fine.
Comment #14
pfrenssenAssigning for review.
Comment #15
pfrenssenI did not see the openid schema either in the configuration inspector, until I enabled the OpenID module :D
Looks good, RTBC!
Comment #16
webchickCommitted and pushed to 8.x. Thanks!