search

hackforla / website

Hack for LA's website
https://www.hackforla.org
GNU General Public License v2.0
325 stars 778 forks source link

Removed api-actionnetwork Script from default-markdown.html and JS file from assets/js #7719

Closed floydferrer closed 1 week ago

floydferrer commented 1 week ago
Fixes #7091 ### What changes did you make? - Removed api-actionnetwork script tag from _layouts/default-markdown.html - Removed api-actionnetwork.js file from assets/js directory ### Why did you make the changes (we will use this info to test)? - The api-actionnetwork script is no longer being used and therefore all references to api-actionnetwork were removed

CodeQL Alerts

After the PR has been submitted and the resulting GitHub actions/checks have been completed, developers should check the PR for CodeQL alert annotations.
Check the PR's comments. If present on your PR, the CodeQL alert looks similar as shown ![Screenshot 2024-10-28 154514](https://github.com/user-attachments/assets/ea66c586-c14c-45fd-8705-1c116224e704)
Please let us know that you have checked for CodeQL alerts. **Please do not dismiss alerts.** - [x] I have checked this PR for CodeQL alerts and none were found. - [ ] I found CodeQL alert(s), and (select one): - [ ] I have resolved the CodeQL alert(s) as noted - [ ] I believe the CodeQL alert(s) is a false positive (Merge Team will evaluate) - [ ] I have followed the Instructions below, but I am still stuck (Merge Team will evaluate)
Instructions for resolving CodeQL alerts If CodeQL alert/annotations appear, refer to [How to Resolve CodeQL alerts](https://github.com/hackforla/website/issues/6463#issuecomment-2002573270). In general, CodeQL alerts should be resolved prior to PR reviews and merging
### Screenshots of Proposed Changes To The Website (if any, please do not include screenshots of code changes) - No visual changes to the website
github-actions[bot] commented 1 week ago

Want to review this pull request? Take a look at this documentation for a step by step guide!

From your project repository, check out a new branch and test the changes.

git checkout -b floydferrer-remove-api-actionnetwork.js-from-md-template-7091 gh-pages
git pull https://github.com/floydferrer/website.git remove-api-actionnetwork.js-from-md-template-7091
siyunfeng commented 1 week ago

Review ETA: 11/15/2024 Friday Availability: 6-10 PM Weekdays (EDT)