Skip to content

resolve codeQL alert3 6044#6362

Merged
jphamtv merged 1 commit intohackforla:gh-pagesfrom
duojet2ez:resolve-code-ql-alert-6044
Mar 1, 2024
Merged

resolve codeQL alert3 6044#6362
jphamtv merged 1 commit intohackforla:gh-pagesfrom
duojet2ez:resolve-code-ql-alert-6044

Conversation

@duojet2ez
Copy link
Member

@duojet2ez duojet2ez commented Feb 23, 2024

Fixes #6044

What changes did you make?

In _includes/current-guides.html I replaced this line:

<h3><a href="{{item.resource-url}}" target="_blank">{{ item.title }}</a></h3>

with this:

<h3><a href="{{item.resource-url}}" target="_blank" rel="noopener noreferrer">{{ item.title }}</a></h3>

Why did you make the changes (we will use this info to test)?

To resolve the alert "Potentially unsafe external link" which appears in the CodeQL alert 3

Screenshots of Proposed Changes

No visual changes to website

@github-actions
Copy link

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 duojet2ez-resolve-code-ql-alert-6044 gh-pages
git pull https://github.com/duojet2ez/website.git resolve-code-ql-alert-6044

Note that CONTRIBUTING.md cannot previewed locally; rather it should be previewed at this URL:

https://github.com/duojet2ez/website/blob/resolve-code-ql-alert-6044/CONTRIBUTING.md  

@github-actions github-actions bot added good first issue Good for newcomers role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers P-Feature: Toolkit https://www.hackforla.org/toolkit/ size: 0.25pt Can be done in 0.5 to 1.5 hours Feature: Code Alerts labels Feb 23, 2024
@roslynwythe
Copy link
Member

@duojet2ez Please complete the description of your work in the body of the PR. Thanks!

@duojet2ez
Copy link
Member Author

@roslynwythe Does this look ok now? I filled in the description of what I did and put in a code block. Thanks!!

@gaylem
Copy link
Member

gaylem commented Feb 28, 2024

Availability: 2/28 - 3/1 9AM-5PM CST
Review ETA: 2/28 10PM CST

@aidanwsimmons
Copy link
Member

Availability: weekdays afternoon/evening
ETA: EOD Wednesday

gaylem
gaylem previously requested changes Feb 28, 2024
Copy link
Member

@gaylem gaylem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @duojet2ez, the Contributing doc also says to keep the final section about screenshots but to put the following instead of images:

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

No visual changes to the website.

Can you add this as well? Otherwise your changes look good!

  • You linked the issue correctly in your PR description
  • Your branch is correct
  • No visible changes, but everything still looks good across mobile and desktop on Chrome, Firefox, and Safari

Thanks for your work on this! 😄

Copy link
Member

@aidanwsimmons aidanwsimmons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good to me!

  • issue linked correctly
  • branch looks good
  • changes made match the issue's action items

@duojet2ez
Copy link
Member Author

@gaylem done!

Copy link
Member

@jphamtv jphamtv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @duojet2ez, thanks for making the updates the reviewers have requested. All the changes look good. PR approved.

@jphamtv jphamtv dismissed gaylem’s stale review March 1, 2024 07:00

Review dismissed as all feedback has been addressed and we have the necessary approvals for merging. Big thanks for your help and understanding!

@jphamtv jphamtv merged commit 3b959b2 into hackforla:gh-pages Mar 1, 2024
@duojet2ez duojet2ez deleted the resolve-code-ql-alert-6044 branch September 12, 2024 17:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature: Code Alerts good first issue Good for newcomers P-Feature: Toolkit https://www.hackforla.org/toolkit/ role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resolve CodeQL alert 3 "Potentially unsafe external link"

5 participants