• Add checkbox in each time on the listing page of Images (AMI)
  • Add the test code to validate whether this function is working or not
  • Use Drupal core bulk_form functionality
CommentFileSizeAuthor
#3 3060173-3.patch16.33 KBXiaohua Guan
#2 3060173-2.patch15.44 KBXiaohua Guan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Xiaohua Guan created an issue. See original summary.

Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
FileSize
3060173-2.patch15.44 KB
Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
FileSize
3060173-3.patch16.33 KB
Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
Status: Active » Needs review

@yas

Please review the patch file. Thanks.

yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

@xiaohua-guan

Thank you for providing the patch. I tested it, and it looks good to me. I'll merge the patch to 8.x-1.x and 8.x-2.x and close this issue as Fixed.

  • yas committed bb84aec on 8.x-1.x authored by Xiaohua Guan 
    Issue #3060173 by Xiaohua Guan, yas: Delete multiple AMI images at once

  • yas committed aa7dcd2 on 8.x-2.x authored by Xiaohua Guan 
    Issue #3060173 by Xiaohua Guan, yas: Delete multiple AMI images at once
yas’s picture

yas
🇯🇵 he/him
Primary language Japanese
Location California 🇺🇸
CreditAttribution: yas at DOCOMO Innovations, Inc. commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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