Skip to content

Conversation

@hoanhua14
Copy link
Member

@hoanhua14 hoanhua14 commented Feb 13, 2025

Fixes #7766

What changes did you make?

  • replace - name: Ethan Hopkins
  • with
- name: Ethan Hopkins

  github-handle:

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

  • We need to create a variable github-handle which will eventually be replaced to reduce repetition.

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

Please let us know that you have checked for CodeQL alerts. Please do not dismiss alerts.

  • 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.

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)

Visuals before changes are applied Screenshot 2025-02-19 at 11 49 54 AM Visuals after changes are applied Screenshot 2025-02-19 at 11 08 58 AM

@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 hoanhua14-add-github-handle-for-Ethan-Hopkins-7766 gh-pages
git pull https://github.com/hoanhua14/hackforlawebsite.git add-github-handle-for-Ethan-Hopkins-7766

@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: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) size: 0.25pt Can be done in 0.5 to 1.5 hours labels Feb 13, 2025
@gmgonzal gmgonzal self-requested a review February 14, 2025 18:38
@gmgonzal
Copy link
Member

Review ETA: 4p 2/14
Availability: 9p - 4p 2/14

@santiseccovidal santiseccovidal requested review from santiseccovidal and removed request for santiseccovidal February 16, 2025 17:57
@acterin acterin self-requested a review February 17, 2025 02:21
@acterin
Copy link
Member

acterin commented Feb 17, 2025

Review ETA: 9p 2/17
Availability: On-demand, ; You may reach out to me on Slack

@acterin
Copy link
Member

acterin commented Feb 17, 2025

Hi @hoanhua14,
On the source code, your changes look good.
Please add visuals of your changes before and after (even if there aren't any)
Great job and thank you!

@gmgonzal
Copy link
Member

Review ETA: 3p 2/17
Availability: 9p - 3p 2/17

Copy link
Member

@gmgonzal gmgonzal left a comment

Choose a reason for hiding this comment

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

Hi @hoanhua14, thank you for working on this issue. Your code changes look good but there are a few changes needed:

Things You Did Well:

  • Step 0: PR is done with the correct branch.
    • PR description looks good.
  • Step 1: PR links to the correct issue #7766.
  • Step 2: Changes appear to address the issue action items.
  • Step 3: Website page looks good on the browser.
  • Step 4: Source code changes look good.
  • Step 5: No other issues seen.

Things You Need to Change:

  • Check off the Action Items in the Issue page for #7906.
  • Mark that you have checked this PR for CodeQL alerts.
  • Remove the linked issue number from the title.
  • Add a before and after visual to confirm that appearance of the webpage is unchanged.

I will approve this PR once the above changes are made.

@hoanhua14 hoanhua14 changed the title Add GitHub handle for Ethan Hopkins - #7766 Add GitHub handle for Ethan Hopkins Feb 19, 2025
@hoanhua14
Copy link
Member Author

Hi @gmgonzal ,
Thank you for the detailed feedback. I made some changes and it should be ready to be reviewed again. Please let me know if anything else needs be to fixed. Thank you.

@hoanhua14 hoanhua14 requested a review from gmgonzal February 19, 2025 21:15
Copy link
Member

@gmgonzal gmgonzal left a comment

Choose a reason for hiding this comment

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

Hi @hoanhua14, thank you for making the requested changes. The changes look good! I would recommend you to check off the action items in the linked issue (#7766) to help you and the team keep track of progress made.

That said, well done, I have approved the PR :)

@LRenDO LRenDO merged commit f65650c into hackforla:gh-pages Feb 20, 2025
17 checks passed
@hoanhua14
Copy link
Member Author

Hi @gmgonzal, I misunderstood that part but the action items are checked now (except for the one for merge team). Thank you for the guidance.

@hoanhua14 hoanhua14 deleted the add-github-handle-for-Ethan-Hopkins-7766 branch February 20, 2025 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue Good for newcomers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) 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

Development

Successfully merging this pull request may close these issues.

Add github-handle for Ethan Hopkins in new-schools-today.md

4 participants