Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xiaohua Guan created an issue. See original summary.

Xiaohua Guan’s picture

Issue summary: View changes
Xiaohua Guan’s picture

Status: Active » Needs review
FileSize
26.69 KB

@yas

Please review the patch file. Thanks.

Xiaohua Guan’s picture

Assigned: Unassigned » Xiaohua Guan
yas’s picture

@xiaohua-guan

Thank you for adding the feature. It looks good to me.

@baldwinlouie

What do you think?

baldwinlouie’s picture

Status: Needs review » Reviewed & tested by the community

@yas, @xiaohua-guan

Patch looks good to me!

yas’s picture

@baldwinlouie

Thank you for your review. I'll merge the patch to 8.x-1.x and 8.x-2.x and close this issue as Fixed.

  • yas committed 22f6c18 on 8.x-1.x authored by Xiaohua Guan
    Issue #3171496 by Xiaohua Guan, yas, baldwinlouie: Notify unused EIP(s)...

  • yas committed 24090d4 on 8.x-2.x authored by Xiaohua Guan
    Issue #3171496 by Xiaohua Guan, yas, baldwinlouie: Notify unused EIP(s)...

  • yas committed 1b593ec on 3.x authored by Xiaohua Guan
    Issue #3171496 by Xiaohua Guan, yas, baldwinlouie: Notify unused EIP(s)...
yas’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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