CommentFileSizeAuthor
#2 3117947-2.patch2.92 KBswatichouhan012
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

swatichouhan012 created an issue. See original summary.

swatichouhan012’s picture

swatichouhan012 CreditAttribution: swatichouhan012 at Valuebound for Valuebound commented
Assigned: swatichouhan012 » Unassigned
Status: Active » Needs review
FileSize
3117947-2.patch2.92 KB

Kindly review patch

martin107’s picture

martin107 CreditAttribution: martin107 as a volunteer commented
Status: Needs review » Reviewed & tested by the community

a) After a visual scan of the patch - All the changes make sense to me.
b) Looking at the test result, no additional coding standard error are introduced.
c) The patch applies against 8.x-4.x

I think the module is improved by these changes.

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir at MD Systems GmbH commented
Status: Reviewed & tested by the community » Fixed

Thanks.

Status: Fixed » Closed (fixed)

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