Skip to content

Added github-handle for Olivia Chiong#6906

Merged
jphamtv merged 1 commit intohackforla:gh-pagesfrom
terrencejihoonjung:add-github-handle-6760
May 30, 2024
Merged

Added github-handle for Olivia Chiong#6906
jphamtv merged 1 commit intohackforla:gh-pagesfrom
terrencejihoonjung:add-github-handle-6760

Conversation

@terrencejihoonjung
Copy link
Member

@terrencejihoonjung terrencejihoonjung commented May 28, 2024

Fixes #6760

What changes did you make?

  • created a single variable "github-handle" under "name" variable" for Olivia Chiong

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

  • in the future github-handle will replace the github and picture variables, reducing redundancy in the project file

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

  • Added variable to civic-tech-structure.md file. 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 terrencejihoonjung-add-github-handle-6760 gh-pages
git pull https://github.com/terrencejihoonjung/website.git add-github-handle-6760

@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 28, 2024
@taylorbeee taylorbeee self-assigned this May 28, 2024
Copy link
Member

@taylorbeee taylorbeee left a comment

Choose a reason for hiding this comment

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

Hey @terrencejihoonjung good work!

  • The issue is properly linked
  • Changes were summarized and implemented correctly
  • No visual changes were made to the website

Great job! Thanks for your contribution!

@taylorbeee taylorbeee removed their assignment May 28, 2024
@vanessasinam
Copy link
Member

Review ETA: 5/30/24 EST
Availability: 1PM-5PM (Mon-Fri)

@LRenDO
Copy link
Member

LRenDO commented May 29, 2024

Hi @vanessasinam! Thanks for volunteering to review this issue! When you have a minute, please add your ETA and Availability.

Copy link
Member

@vanessasinam vanessasinam left a comment

Choose a reason for hiding this comment

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

Hi @terrencejihoonjung

The PR is done with the correct branch
The issue is linked properly
The code change is clean
The visual aspect of the webpage hasn't changed

Good work, thanks for your contribution!

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 Olivia Chiong in civic-tech-structure.md

5 participants