As of Drupal 8 all JS needs to follow our JavaScript coding standards. For this we use ESLint to check most of our standards.

Here is the summary of running ESLint on this module JS:
✖ 8 problems (8 errors, ZZZ warnings)

CommentFileSizeAuthor
#2 account-sentinel-eslinterror-2.patch856 byteskpoornima
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kpoornima created an issue. See original summary.

kpoornima’s picture

kpoornima CreditAttribution: kpoornima as a volunteer and at Melity commented
FileSize
account-sentinel-eslinterror-2.patch856 bytes

this patch has the cleaned up code.

kpoornima’s picture

kpoornima CreditAttribution: kpoornima as a volunteer and at Melity commented
Status: Active » Needs review

banviktor’s picture

banviktor CreditAttribution: banviktor at CARD.com commented

Thank you @kpoornima!

banviktor’s picture

banviktor CreditAttribution: banviktor at CARD.com commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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