The config Lightning profiles /config/install contains config that depends on block_content (field.field.block_content.basic.body.yml) and bartik (most of the block.block.* config). That should be moved into the /config/optional directory so that extending profiles can exclude block_content if they want and/or not install Bartik.

CommentFileSizeAuthor
#6 2899017-6.patch6.4 KBbalsama
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

balsama created an issue. See original summary.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Active » Needs review

Pull request created: https://github.com/acquia/lightning/pull/419

balsama’s picture

balsama
he/him
Primary language English
Location boston
CreditAttribution: balsama at Acquia commented

Part of the problem we were having that inspired this issue was that we were excluding block_content. block_content has a dependency on block, so neither Lightning (nor the sub-profile we were building) had a dependency on block - which we still need.

So, in addition to moving the config mentioned above, sub-profiles that exclude block_content will need to ADD a dependency on block (assuming they still want block).

  • balsama committed abfe5cf on 8.x-2.x authored by phenaproxima 
    Issue #2899017 by phenaproxima: Remove hard dependency on block_content...
balsama’s picture

balsama
he/him
Primary language English
Location boston
CreditAttribution: balsama at Acquia commented
Status: Needs review » Fixed
balsama’s picture

balsama
he/him
Primary language English
Location boston
CreditAttribution: balsama at Acquia commented
FileSize
2899017-6.patch6.4 KB

Patch attached.

Status: Fixed » Closed (fixed)

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