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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Gábor Hojtsy’s picture

Issue tags: +sprint
Gábor Hojtsy’s picture

Remove unrelated tags.

e0ipso’s picture

Added config schema to core/modules/system/config/schema/system.schema.yml.

See config_inspector screenshots:

Configuration_inspector___D8_Dev-2.png

Tree_of_configuration_data_for_system.language.types___D8_Dev.png

e0ipso’s picture

Status: Active » Needs review
e0ipso’s picture

Assigned: Unassigned » e0ipso
Status: Needs review » Active
e0ipso’s picture

Status: Active » Needs review

Sorry for the noise, I seem to be unable to get all options at once.

Gábor Hojtsy’s picture

Status: Needs review » Needs work

In the label you say customized while the key says configurable. Let's say that on the label too? Otherwise looks good.

e0ipso’s picture

Status: Needs work » Needs review
FileSize
803 bytes

Changed label.

Gábor Hojtsy’s picture

Status: Needs review » Reviewed & tested by the community

Looks good!

YesCT’s picture

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed a142acc and pushed to 8.x. Thanks!

Gábor Hojtsy’s picture

Issue tags: -sprint

Yay, thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.