We have to convert the module to use short array as per new coding standard

CommentFileSizeAuthor
#6 2876839-2.patch2.85 KBshaktik
#5 broken link.PNG10.25 KBpankajsachdeva
#2 2876839-2.patch2.85 KBPavan B S
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Pavan B S created an issue. See original summary.

Pavan B S’s picture

Pavan B S CreditAttribution: Pavan B S at Valuebound commented
Assigned: Pavan B S » Unassigned
Status: Active » Needs review
FileSize
2876839-2.patch2.85 KB

Applying the patch, please review

pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented

Hi Pavan,

I manually check your patch and its looking good and also with new D8 coding standardsd.

pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
Status: Needs review » Reviewed & tested by the community
pankajsachdeva’s picture

pankajsachdeva CreditAttribution: pankajsachdeva as a volunteer and at ]init[ AG commented
FileSize
broken link.PNG10.25 KB

Attaching screenshot

shaktik’s picture

shaktik
he/him
Primary language Hindi
Location New Delhi
CreditAttribution: shaktik as a volunteer commented
Assigned: Unassigned » shaktik
Status: Reviewed & tested by the community » Fixed
FileSize
2876839-2.patch2.85 KB

Converted the module to use short array as per new coding standard.

Status: Fixed » Closed (fixed)

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