Skip to content

Add github-handle placeholder for Sarah Port#8648

Merged
dvernon5 merged 1 commit into
hackforla:gh-pagesfrom
joshjyu:sarah-port-GH-handle-7402
May 21, 2026
Merged

Add github-handle placeholder for Sarah Port#8648
dvernon5 merged 1 commit into
hackforla:gh-pagesfrom
joshjyu:sarah-port-GH-handle-7402

Conversation

@joshjyu
Copy link
Copy Markdown
Member

@joshjyu joshjyu commented May 20, 2026

Fixes #7402

What changes did you make?

  • Added a github-handle placeholder for Sarah Port in hellogov.md

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

  • To reduce the redundancy of the github and picture variables as github-handle will replace them eventually.

CodeQL Alerts

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)

@github-actions
Copy link
Copy Markdown

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 joshjyu-sarah-port-GH-handle-7402 gh-pages
git pull https://github.com/joshjyu/website.git sarah-port-GH-handle-7402

@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 May 20, 2026
@sushma110396 sushma110396 self-requested a review May 20, 2026 16:11
@sushma110396
Copy link
Copy Markdown
Member

Review ETA: 1PM 05/20/26
Availability: Weekdays after 6PM PST

Copy link
Copy Markdown
Member

@sushma110396 sushma110396 left a comment

Choose a reason for hiding this comment

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

I have verified the changes locally. Looks good to me!
All the required steps have been followed correctly.

  • Pull request has the correct branch name
  • There is a linked issue
  • File change looks good.
  • No visual changes to the website

PR approved.

@nathanjkim-codes
Copy link
Copy Markdown
Member

ETA: Wednesday EOD

Availability:

  • Wednesday, Friday: 6:00 PM - 9:00 PM PST

Copy link
Copy Markdown
Member

@nathanjkim-codes nathanjkim-codes left a comment

Choose a reason for hiding this comment

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

I reviewed the PR and checked the changes locally.

  • Branch name follows the project naming convention
  • Linked issue is included correctly
  • The file update looks correct
  • No visual changes were introduced to the website

Everything looks good to me. PR approved.

@github-project-automation github-project-automation Bot moved this from PR Needs review to PRs ✅ waiting for merge team in P: HfLA Website: Project Board May 21, 2026
@dvernon5 dvernon5 merged commit c8c272d into hackforla:gh-pages May 21, 2026
16 checks passed
@dvernon5
Copy link
Copy Markdown
Member

@joshjyu Thank you for your contributions, Joshua, and excellent job on your issue. Your issue has been merged.

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 Sarah Port in hellogov.md

4 participants