Now fields and instances are config entities #1735118: Convert Field API to CMI

Let's move them to config as #1943468: Move Tags vocabulary to standard profile config

Files: 
CommentFileSizeAuthor
#8 1970206-config-fields-8.patch10.01 KBswentel
PASSED: [[SimpleTest]]: [MySQL] 54,475 pass(es). View
#8 interdiff.txt4.18 KBswentel
#1 1970206-config-fields-1.patch8.99 KBandypost
PASSED: [[SimpleTest]]: [MySQL] 54,309 pass(es). View

Comments

andypost’s picture

Status: Active » Needs review
FileSize
8.99 KB
PASSED: [[SimpleTest]]: [MySQL] 54,309 pass(es). View

Let's see what bot say

chx’s picture

Status: Needs review » Reviewed & tested by the community

Superb CMI demo.

andypost’s picture

Issue tags: +Needs manual testing

Not sure we have tests for standard profile - so needs confirmation that standard profile installs fine

xjm’s picture

Status: Reviewed & tested by the community » Needs review

We also need to answer the "UUIDs in default config" question in #1969800: Add UUIDs to default configuration for this.

andypost’s picture

Issue tags: -Needs manual testing

Tested manually standard install and it works!
Actually UUIDs currently is hard requirement for field instances...

xjm’s picture

Actually UUIDs currently is hard requirement for field instances...

UUIDs are a hard requirement for active field instances. UUIDs are added automatically to config entities supplied as default config at installation. See the other issue, where various people have expressed concerns about shipping UUIDs in default config.

swentel’s picture

Hmm tricky one :) Reference to uuid is the safest imo, however, we do allow in the FieldInstance class to accept an incoming field_name as well, so maybe change that for now and see what the outcome is of the other discussion.

swentel’s picture

FileSize
4.18 KB
10.01 KB
PASSED: [[SimpleTest]]: [MySQL] 54,475 pass(es). View

This should work - cleaned up also in tests of field api and I could install standard profile too.

andypost’s picture

Status: Needs review » Reviewed & tested by the community

Let's get it in

alexpott’s picture

Committed 0cc0335 and pushed to 8.x. Thanks!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed
Gábor Hojtsy’s picture

Issue tags: +D8MI, +language-config

Woot, this will properly make them translatable as part of the configuration of the site. Thanks all!

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