Updated: Comment #0
Problem/Motivation
Some schema is missing for the committed #1833022: Only display interface language detection options to customize more granularity
Proposed resolution
core/modules/system/config/schema/system.schema.yml should have a schema snippet for system.language.types
Remaining tasks
TBD
for hints on adding and reviewing, please read #1910624: [META] Introduce and complete configuration schemas in all of core
User interface changes
No.
API changes
No.
Related Issues
Comment | File | Size | Author |
---|---|---|---|
#10 | language-types.png | 220.84 KB | YesCT |
#8 | drupal-config-schema-language-types-2033971-8.patch | 803 bytes | e0ipso |
#3 | drupal-config-schema-language-types-2033971-3.patch | 801 bytes | e0ipso |
#3 | Configuration_inspector___D8_Dev-2.png | 17.7 KB | e0ipso |
#3 | Tree_of_configuration_data_for_system.language.types___D8_Dev.png | 55.56 KB | e0ipso |
Comments
Comment #1
Gábor HojtsyComment #2
Gábor HojtsyRemove unrelated tags.
Comment #3
e0ipsoAdded config schema to
core/modules/system/config/schema/system.schema.yml
.See config_inspector screenshots:
Comment #4
e0ipsoComment #5
e0ipsoComment #6
e0ipsoSorry for the noise, I seem to be unable to get all options at once.
Comment #7
Gábor HojtsyIn the label you say customized while the key says configurable. Let's say that on the label too? Otherwise looks good.
Comment #8
e0ipsoChanged label.
Comment #9
Gábor HojtsyLooks good!
Comment #10
YesCT CreditAttribution: YesCT commentedI agree. I checked it per #1910624: [META] Introduce and complete configuration schemas in all of core
Comment #11
alexpottCommitted a142acc and pushed to 8.x. Thanks!
Comment #12
Gábor HojtsyYay, thanks!