Skip to content

Add github-handle for Bonnie Wolfe in engage.md#6912

Merged
jphamtv merged 1 commit intohackforla:gh-pagesfrom
jchue:add-github-handle-bonnie-wolfe-engage-6729
May 30, 2024
Merged

Add github-handle for Bonnie Wolfe in engage.md#6912
jphamtv merged 1 commit intohackforla:gh-pagesfrom
jchue:add-github-handle-bonnie-wolfe-engage-6729

Conversation

@jchue
Copy link
Member

@jchue jchue commented May 30, 2024

Fixes #6729

What changes did you make?

  • Added github-handle property to Bonnie Wolfe in the engage.md project file

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

  • To reduce redundancy in the project file in the future

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

Adding empty property to the YAML. No visual changes to the 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 jchue-add-github-handle-bonnie-wolfe-engage-6729 gh-pages
git pull https://github.com/jchue/hackforla-website.git add-github-handle-bonnie-wolfe-engage-6729

@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 30, 2024
@ino-iosdev ino-iosdev self-requested a review May 30, 2024 04:05
@ino-iosdev
Copy link
Member

Review ETA: 11:59PM 5/29/24
Availability: Flexible except for Wed. afternoon till late June

Copy link
Member

@ino-iosdev ino-iosdev left a comment

Choose a reason for hiding this comment

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

Hi @jchue , It's such a blessing to have you as part of the team here!
 

  • Naming was appropriate
  • Direction between branches "from" and "to" are clear
  • The changes and why were clear
  • PR issue link was clear
  • No visual changes to the website
     
    Great job, thank you for contributing!

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.

Hi @jchue - Excellent effort on this pull request.

  • You created the pull request using the correct branch.
  • The issue is properly linked to the pull request.
  • Your summary of changes is clear and concise.
  • You correctly used spaces instead of tabs to indent.

Thanks for your contribution to resolving this issue!

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

None yet

Development

Successfully merging this pull request may close these issues.

Add github-handle for Bonnie Wolfe in engage.md

3 participants