Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Tests were failing for #3073177: Add tests for field_group 3.x because they didn't set the region.
When creating a new field group, there's no default region set, not even a key in in the configuration.
Let's at least set one, defaulting to content, and double check also if DS is enabled.
Comment | File | Size | Author |
---|---|---|---|
#6 | 3086050-6.patch | 6.2 KB | swentel |
#4 | 3086050-4.patch | 5.93 KB | swentel |
Comments
Comment #2
swentel CreditAttribution: swentel at eps & kaas for Dropsolid commentedComment #3
swentel CreditAttribution: swentel at eps & kaas for Dropsolid commentedRaising to critical. Check the entity display and there's no region key indeed. While you will probably start configuring after, it's a good idea to at least set one.
Comment #4
swentel CreditAttribution: swentel at eps & kaas for Dropsolid commentedFailing test.
Took the liberty to fix the docs at the same time too.
Comment #6
swentel CreditAttribution: swentel at eps & kaas for Dropsolid commentedWent for the easy fix: set the region to 'hidden'. You're going to reconfigure the display anyway afterwards. This makes sure though the configuration is valid.
Comment #8
swentel CreditAttribution: swentel at eps & kaas for Dropsolid commentedComment #9
swentel CreditAttribution: swentel at eps & kaas for Dropsolid commentedI mean fixed :)