CommentFileSizeAuthor
#3 After Patch.png66.42 KBzalak.addweb
#3 Before Patch.png78.65 KBzalak.addweb
#2 Configure-link-missing-on-module-page-for-ShURLy-Analytics-2918644.patch417 bytesanup.singh
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

anup.singh created an issue. See original summary.

anup.singh’s picture

anup.singh CreditAttribution: anup.singh as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commented
Status: Active » Needs review
FileSize
Configure-link-missing-on-module-page-for-ShURLy-Analytics-2918644.patch417 bytes

Attaching a patch, please review

zalak.addweb’s picture

zalak.addweb CreditAttribution: zalak.addweb at AddWeb Solution Pvt. Ltd. commented
Status: Needs review » Reviewed & tested by the community
FileSize
Before Patch.png78.65 KB
After Patch.png66.42 KB

@anup.singh, Thanks for the patch, it works well as required. I checked it over simplytest.me. PFA

anup.singh’s picture

anup.singh CreditAttribution: anup.singh as a volunteer and at Faichi Solutions Pvt Ltd for Faichi Solutions Pvt Ltd commented
Assigned: anup.singh » Unassigned
fengtan’s picture

fengtan
Location Montreal, Canada
CreditAttribution: fengtan as a volunteer and at McGill University commented

+1, patch looks good

  • Jibus committed 77e79ca on 7.x-1.x authored by anup.singh 
    Issue #2918644 by anup.singh, prafull.addweb: Configure link missing on...
Jibus’s picture

Jibus CreditAttribution: Jibus commented
Status: Reviewed & tested by the community » Fixed

Thanks ! Commited against 7.x-1.x

Status: Fixed » Closed (fixed)

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