CommentFileSizeAuthor
#5 fix_comment_in-2903531-5.patch557 bytesyogeshmpawar
#4 fix_comment_in-2903531-4.patch688 bytesyogeshmpawar
#2 fix_comment_in-2903531-2.patch500 bytesborisson_
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

borisson_ created an issue. See original summary.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ as a volunteer and at Dazzle commented
Status: Active » Needs review
FileSize
fix_comment_in-2903531-2.patch500 bytes
yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location 🇮🇳 Pune(MH)
CreditAttribution: yogeshmpawar at QED42 commented
Assigned: Unassigned » yogeshmpawar

Checking the patch

yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location 🇮🇳 Pune(MH)
CreditAttribution: yogeshmpawar at QED42 commented
FileSize
fix_comment_in-2903531-4.patch688 bytes

Updated patch after removing the coding standards issues from code.

yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location 🇮🇳 Pune(MH)
CreditAttribution: yogeshmpawar at QED42 commented
FileSize
fix_comment_in-2903531-5.patch557 bytes

Updated patch after removing the coding standards issues from code & previous patch have some issues to apply.

  • Yogesh Pawar committed f11b434 on 8.x-1.x 
    Issue #2903531 by Yogesh Pawar, borisson_: Fix comment in...
yogeshmpawar’s picture

yogeshmpawar
He/Him/His
Primary language English
Location 🇮🇳 Pune(MH)
CreditAttribution: yogeshmpawar at QED42 commented
Assigned: yogeshmpawar » Unassigned
Status: Needs review » Fixed

Committed & Pushed to 8.x-1.x branch.
Thanks @borisson_

Status: Fixed » Closed (fixed)

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