Skip to content

added new variable github-handle to Eric Cho in 311-data.md#5768

Merged
kwangric merged 1 commit intohackforla:gh-pagesfrom
kevin31yu:add-github-handle-to-eric-cho-in-311-data-5713
Oct 25, 2023
Merged

added new variable github-handle to Eric Cho in 311-data.md#5768
kwangric merged 1 commit intohackforla:gh-pagesfrom
kevin31yu:add-github-handle-to-eric-cho-in-311-data-5713

Conversation

@kevin31yu
Copy link
Member

@kevin31yu kevin31yu commented Oct 23, 2023

Fixes #5713

What changes did you make?

  • added variable github-handle to Eric Cho and checked with Docker at localhost:4000/

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

  • The issue aimed to replace variables github and picture with new variable github-handle

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

Visuals before changes are applied

before

Visuals after changes are applied

after

@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 kevin31yu-add-github-handle-to-eric-cho-in-311-data-5713 gh-pages
git pull https://github.com/kevin31yu/website.git add-github-handle-to-eric-cho-in-311-data-5713

@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 Oct 23, 2023
@kevin31yu kevin31yu self-assigned this Oct 23, 2023
@KyleA99 KyleA99 self-requested a review October 24, 2023 01:15
@KyleA99
Copy link
Member

KyleA99 commented Oct 24, 2023

Review ETA: EOD PM 10/23/23
Availability: 5-8 PM M - F

@DakuwoN
Copy link
Member

DakuwoN commented Oct 24, 2023

Hack for LA Hacktorfest?? :D

@pdimaano pdimaano self-requested a review October 24, 2023 03:30
@pdimaano
Copy link
Contributor

ETA: 10/26/23
Availability: Weekday evenings

Copy link
Member

@KyleA99 KyleA99 left a comment

Choose a reason for hiding this comment

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

Looks great @kevin31yu code is properly changed and I do not see any issues in your screenshots or when I open locally on docker. Nice work. Approved from me.

Copy link
Contributor

@pdimaano pdimaano left a comment

Choose a reason for hiding this comment

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

@kevin31yu Good job: The branches are correct and the issue is linked. The webpage is visually unchanged on my system on desktop, mobile and tablet. The correct file was updated with the appropriate information. Also, for future pull requests, note that the contribution wiki states that no screenshots are necessary if there are no visual changes to the website. You can simply remove the section and write "No visual changes to the website." Otherwise, well done!

Copy link
Member

@blulady blulady left a comment

Choose a reason for hiding this comment

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

This branch is coming from the correct branch and going into gh-pages. There is a linked issue, it is correct and addressed in the PR by adding line 10 _projects. The code looks clean, only one commit and only one file changed.
Good job!!

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 new variable github-handle to member Eric Cho in 311-data.md

6 participants