There are some coding standard issues in this module.

CommentFileSizeAuthor
#4 instagramfeed_coding_standard-2908709-4.patch1.4 KBDeepthi kumari
#2 instagramfeed_coding_standard-2908709-2.patch3.52 KBjayesh_makwana
instagramfeed.TXT3.29 KBjayesh_makwana
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jayesh_makwana created an issue. See original summary.

jayesh_makwana’s picture

jayesh_makwana CreditAttribution: jayesh_makwana at cmsMinds commented
FileSize
instagramfeed_coding_standard-2908709-2.patch3.52 KB

We created a patch for solving coding standard issues for this module. Please check and apply it.

  • Elfsight committed 8848aa3 on 8.x-1.x 
    Issue #2908709 by jayesh_makwana : coding standards issue fix.
Deepthi kumari’s picture

Deepthi kumari CreditAttribution: Deepthi kumari as a volunteer and at TA Digital commented
Status: Active » Needs review
FileSize
instagramfeed_coding_standard-2908709-4.patch1.4 KB

Removed \Drupal calls and have used Dependency Injection. Please review.

Elfsight’s picture

Elfsight CreditAttribution: Elfsight commented
Status: Needs review » Fixed
Elfsight’s picture

Elfsight CreditAttribution: Elfsight commented
Status: Fixed » Closed (fixed)