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 tracker module.
Schema in place
Schema not yet in place
tracker.settings.yml
Comment | File | Size | Author |
---|---|---|---|
#6 | 1919214-tracker-schema-6.patch | 474 bytes | vijaycs85 |
#3 | tracker_patch_2.PNG | 22.33 KB | rahuldolas123 |
#2 | 1919214-tracker-schema-2.patch | 502 bytes | ars- |
#1 | 1919214-tracker-schema-1.patch | 378 bytes | vijaycs85 |
Comments
Comment #1
vijaycs85Adding schema file...
Comment #2
ars- CreditAttribution: ars- commentedCode adding label
Comment #3
rahuldolas123 CreditAttribution: rahuldolas123 commentedHi
I have tested the above patch with config inspector module and the patch works.
Attaching the screenshot for the same.
Comment #4
vijaycs85Looks good to me...
Comment #5
aspilicious CreditAttribution: aspilicious commentedNeeds a newline in the end
Comment #6
vijaycs85Added new line...
Comment #8
vijaycs85#6: 1919214-tracker-schema-6.patch queued for re-testing.
Comment #9
pfrenssenI'm going to review this.
Comment #10
pfrenssenLooks good, RTBC!
Comment #11
webchickCommitted and pushed to 8.x. Thanks!
Though I have no idea why we still have tracker.module when Views is in core now. :D