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 text module.
Schema in place
Schema not yet in place
text.settings.yml
Comment | File | Size | Author |
---|---|---|---|
#5 | text_patch_4.PNG | 21.31 KB | rachit_gupta |
#4 | 1919210-text-schema-4.patch | 459 bytes | rahuldolas123 |
#1 | 1919210-text-schema-1.patch | 360 bytes | vijaycs85 |
Comments
Comment #1
vijaycs85Adding schema file
Comment #2
YesCT CreditAttribution: YesCT commentedfor people coming in to test and review, see: http://drupal.org/node/1910624#steps-to-test in the issue summary for #1910624: [META] Introduce and complete configuration schemas in all of core
Those are draft instructions, please suggest improvements to them.
Comment #3
dags CreditAttribution: dags commentedComment #4
rahuldolas123 CreditAttribution: rahuldolas123 commentedAdded code styles and labels.
Comment #5
rachit_gupta CreditAttribution: rachit_gupta commentedHi
I have tested the above patch with config inspector module and the patch works.
Attaching the screenshot for the same.
Comment #6
vijaycs85Thanks @rahuldolas123 and @rachit_gupta. Setting it RTBC.
Comment #7
dags CreditAttribution: dags commentedPatch in #4 looks right. Unassigning from myself.
Comment #8
webchickCommitted and pushed to 8.x. Thanks!