New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update wins.html file to use liquid in this file #5258

Open

ronaldpaek wants to merge 3 commits into hackforla:gh-pages from ronaldpaek:refactor-liquid-js-1442

Open

Update wins.html file to use liquid in this file #5258

Refactor code to remove nodes after parsing data

GitHub Advanced Security / CodeQL succeeded Aug 20, 2023 in 3s

5 new alerts including 1 medium severity security vulnerability

New alerts in code changed by this pull request

Security Alerts:

1 medium

Other Alerts:

2 warnings
2 notes

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 481 in assets/js/wins.js

Code scanning / CodeQL

Semicolon insertion Note

Avoid automated semicolon insertion (96% of all statements in

the enclosing function

have an explicit semicolon).

Check warning on line 289 in assets/js/wins.js

Code scanning / CodeQL

DOM text reinterpreted as HTML Medium

DOM text

is reinterpreted as HTML without escaping meta-characters.

Check notice on line 199 in assets/js/wins.js

Code scanning / CodeQL

Semicolon insertion Note

Avoid automated semicolon insertion (93% of all statements in

the enclosing function

have an explicit semicolon).

Check warning on line 489 in assets/js/wins.js

Code scanning / CodeQL

Expression has no effect Warning

This expression has no effect.

Check warning on line 489 in assets/js/wins.js

Code scanning / CodeQL

Expression has no effect Warning

This expression has no effect.