This is a sub-issue of #1910624: [META] Introduce and complete configuration schemas in all of core

We need some schema with translatable properties for the tests in config_translation.

CommentFileSizeAuthor
#3 config-schema-config_test-2098661-3.patch873 byteswebflo
#1 config-schema-config_test-2098661-1.patch675 byteswebflo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

webflo’s picture

webflo CreditAttribution: webflo commented
Status: Active » Needs review
FileSize
config-schema-config_test-2098661-1.patch675 bytes
Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Needs review » Reviewed & tested by the community
Issue tags: +blocker

Yay! Looks good. This is a blocker for #2098675: Add missing tests for config_translation.mapper_list and config_translation.entity_list which includes test coverage for this :) And that is a blocker for #1952394: Add configuration translation user interface module in core.

webflo’s picture

webflo CreditAttribution: webflo commented
Status: Reviewed & tested by the community » Needs review
FileSize
config-schema-config_test-2098661-3.patch873 bytes

Let's add the other properties of this entity.

Gábor Hojtsy’s picture

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

Thanks for the update. Reviewed in person.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed to 8.x. Thanks!

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