CommentFileSizeAuthor
#6 cleanly_.png18.21 KBguilhermevp
#4 3121938-4.patch373 byteschintan.vyas
#3 3121938-3.patch438 bytesprabha1997
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

prabha1997 created an issue. See original summary.

prabha1997’s picture

prabha1997 CreditAttribution: prabha1997 at Valuebound for Valuebound commented
Assigned: Unassigned » prabha1997

I am working in this issue

prabha1997’s picture

prabha1997 CreditAttribution: prabha1997 at Valuebound for Valuebound commented
Assigned: prabha1997 » Unassigned
Status: Needs work » Needs review
FileSize
3121938-3.patch438 bytes

Kindly review patch

chintan.vyas’s picture

chintan.vyas CreditAttribution: chintan.vyas as a volunteer commented
FileSize
3121938-4.patch373 bytes

The patch looks good however adding configure link can be tracked in separate issue so removing it and updating the patch.

chintan.vyas’s picture

chintan.vyas CreditAttribution: chintan.vyas as a volunteer commented
Category: Bug report » Feature request
guilhermevp’s picture

guilhermevp CreditAttribution: guilhermevp at CI&T commented
Status: Needs review » Reviewed & tested by the community
FileSize
cleanly_.png18.21 KB

Patch applies cleanly and work as intended.

1