Follow up of #1754246: Languages should be configuration entities

Part of #1910624: [META] Introduce and complete configuration schemas in all of core See that for hints on reviewing.

Create configuration schema for language config entities to work like other other config entities.

CommentFileSizeAuthor
#2 language.entity.en_.png43.98 KBvijaycs85
#1 2031185-config-schema-language-1.patch909 bytesvijaycs85
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented
Status: Active » Needs review
FileSize
2031185-config-schema-language-1.patch909 bytes

Intial patch...

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented
FileSize
language.entity.en_.png43.98 KB

Here is config_inspector screenshot...

rahul.shinde’s picture

rahul.shinde
he/him
Primary language Marathi
Location Pune
CreditAttribution: rahul.shinde commented
Status: Needs review » Reviewed & tested by the community

Patch applied seamlessly. Created new language and the config seems to what exactly written. Tried n tested as per #2.

YesCT’s picture

YesCT CreditAttribution: YesCT commented
Issue tags: +RTBC July 1

This issue was RTBC and passing tests on July 1, the beginning of API freeze.

YesCT’s picture

YesCT CreditAttribution: YesCT commented
Issue tags: +Configuration schema

tagging.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented

Committed 359aabe and pushed to 8.x. Thanks!

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed

Yay, thanks!

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

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

added link to schema meta.