Problem/Motivation

See https://www.drupal.org/node/3194517

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 3259173.patch727 bytesheddn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn created an issue. See original summary.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn at MTech, LLC, Tag1 Consulting commented
Status: Active » Needs review
FileSize
3259173.patch727 bytes
jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Thanks for the patch.
This part of DER is heavily under-tested. :(

  • jibran committed 40d0908 on 8.x-2.x authored by heddn 
    Issue #3259173 by heddn, jibran: Not marking service definitions as...

  • jibran committed 91a0b55 on 8.x-1.x authored by heddn 
    Issue #3259173 by heddn, jibran: Not marking service definitions as...
jibran’s picture

jibran
he/him
Primary language Urdu
Location Sydney, Australia
CreditAttribution: jibran as a volunteer commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed and pushed to 1.x and 2.x.

Status: Fixed » Closed (fixed)

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