screenshot-20191024a.png

CommentFileSizeAuthor
#6 screenshot-20191024a.png901.35 KByas
#4 3089815-4.patch11.96 KBXiaohua Guan
#2 3089815-2.patch11.68 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
3089815-2.patch11.68 KB
Xiaohua Guan’s picture

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

@yas

I uploaded a patch for your confirmation. Could you check whether the patch match your requirement?

Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented
FileSize
3089815-4.patch11.96 KB
Xiaohua Guan’s picture

Xiaohua Guan CreditAttribution: Xiaohua Guan commented

@yas

Please review the patch file. Thanks.

FYI, in the same time, I also fixed bugs related to heatmap.

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
Related issues: +#3084313: Composer could not parse version constraint / could not load package
FileSize
screenshot-20191024a.png901.35 KB

@xiaohua-guan

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

  • yas committed 226818e on 8.x-2.x authored by Xiaohua Guan 
    Issue #3089815 by Xiaohua Guan, yas: Refactor doughnut charts
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.