Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated profile section from Anjola Jaiyeola to Kelly Chuang #6711

Merged
merged 1 commit into from
Apr 25, 2024

Conversation

Cloid
Copy link
Member

@Cloid Cloid commented Apr 23, 2024

Fixes #6632

What changes did you make?

  • Updated profile section from Anjola Jaiyeola to Kelly Chuang by removing Anjola's section and adding it with the specified Kelly section.

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

  • To make sure sections are up to date so visitors can contact volunteers if needed.
  • Descriptions are already included in issue.

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

Visuals before changes are applied Screenshot 2024-04-22 at 5 16 18 PM
Visuals after changes are applied Screenshot 2024-04-22 at 5 30 58 PM

@Cloid Cloid added role: front end Tasks for front end developers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) time sensitive Needs to be worked on by a particular timeframe Complexity: Small Take this type of issues after the successful merge of your second good first issue size: 0.25pt Can be done in 0.5 to 1.5 hours labels Apr 23, 2024
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 Cloid-update-profile-pic-6632 gh-pages
git pull https://github.com/Cloid/hackforla_website.git update-profile-pic-6632

Note that CONTRIBUTING.md cannot previewed locally; rather it should be previewed at this URL:

https://github.com/Cloid/website/blob/update-profile-pic-6632/CONTRIBUTING.md  

@tony1ee tony1ee self-requested a review April 23, 2024 01:50
@tony1ee
Copy link
Member

tony1ee commented Apr 23, 2024

Review ETA: EOD 04/22/2024
Availability: 7-10 PM 04/22/2024

Copy link
Member

@tony1ee tony1ee left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution @Cloid

  • correctly linked issue
  • applied changes at the right places
  • visual changes check out on desktop, mobile and tablet view on Chrome

Keep up the good work, approved!

@pdimaano pdimaano self-requested a review April 23, 2024 16:43
@pdimaano
Copy link
Member

ETA: 4/26/24
Availability: Weekday evenings

Copy link
Member

@Thinking-Panda Thinking-Panda left a comment

Choose a reason for hiding this comment

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

@Cloid - Thank you for working on this issue. Merge branches are setup correctly, code changes are as mentioned in the linked issue and clean. Website works well on my local machine. Well done!

Copy link
Member

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

@Cloid Good job: The branches are correct and the issue is linked. The changes are visible on my system on desktop, mobile and tablet. The correct file was changed with the requested information. Well done!

@jphamtv jphamtv merged commit b659859 into hackforla:gh-pages Apr 25, 2024
9 of 10 checks passed
@Cloid Cloid deleted the update-profile-pic-6632 branch April 26, 2024 00:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours time sensitive Needs to be worked on by a particular timeframe
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update Project Profile: Hack for LA Site (Remove Anjola Jaiyeola, Add Kelly Chuang)
5 participants