After 2903432 was addressed, I found that I had mistakenly forced the config type to 'config' - which in the case of other types of dependencies was causing the configuration system to prevent configs from loading due to missing dependencies. Obviously, we should use the correct type. The following patch should address the issue.

CommentFileSizeAuthor
#2 jsonapi_extras_config_dependency-2913958-2.patch588 byteslogickal

Comments

logickal created an issue. See original summary.

logickal’s picture

logickal commented
StatusFileSize
new jsonapi_extras_config_dependency-2913958-2.patch588 bytes
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Status: Active » Needs review

Ahhh. Sorry for missing that in the review. We need so much some testing here.

  • e0ipso committed 6f5d000 on 8.x-1.x authored by logickal 
    fix(Configuration): Fix the config dependency (#2913958 by logickal)
e0ipso’s picture

e0ipso
Primary language Catalan
Location Can Picafort
commented
Status: Needs review » Fixed

Thanks!

Status: Fixed » Closed (fixed)

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