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

Removed Farah Khan from Home Unite Us Project #7588

Conversation

siyunfeng
Copy link
Member

Fixes #7477

What changes did you make?

  • deleted Farah Khan's info from the leadership section in _projects/home-unite-us.md

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

  • to ensure the project leadership info is up-to-date

Screenshots of Proposed Changes To The Website (if any, please do not include screenshots of code changes)

Visuals before changes are applied
Visuals after changes are applied

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 siyunfeng-remove-farah-khan-from-home-unite-us-7477 gh-pages
git pull https://github.com/siyunfeng/hackforla-website.git remove-farah-khan-from-home-unite-us-7477

@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) time sensitive Needs to be worked on by a particular timeframe size: 0.25pt Can be done in 0.5 to 1.5 hours labels Oct 13, 2024
@floydferrer floydferrer self-requested a review October 13, 2024 22:22
@floydferrer
Copy link
Member

Review ETA: 10/14
Availability: 5-9PM Sunday, 9AM-12PM Monday

Copy link
Member

@buneeIsSlo buneeIsSlo left a comment

Choose a reason for hiding this comment

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

Hi! @siyunfeng Great work

  • The branches from and to are setup correctly
  • Original issue is linked
  • There is a clear and concise description of what was done and why
  • Screenshots of before and after the changes are included

Thanks for working on this issue!

Copy link
Member

@floydferrer floydferrer left a comment

Choose a reason for hiding this comment

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

Great job @siyunfeng!

  • The branches from and to are setup correctly
  • Original issue is linked
  • There is a clear and concise description of what was done and why
  • Screenshots of before and after the changes are included

Thanks for working on this issue!

@daras-cu daras-cu merged commit b59439a into hackforla:gh-pages Oct 15, 2024
7 checks passed
@siyunfeng siyunfeng deleted the remove-farah-khan-from-home-unite-us-7477 branch October 15, 2024 18:30
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 time sensitive Needs to be worked on by a particular timeframe
Projects
Development

Successfully merging this pull request may close these issues.

Update Project Profile: Home Unite Us Remove Farah Khan
4 participants