Skip to content

Update Project Profile: Website- add Mugdh Chauhan to Merge Team #7939#8062

Merged
LRenDO merged 1 commit intohackforla:gh-pagesfrom
githelsui:update-leadership-7939
Apr 16, 2025
Merged

Update Project Profile: Website- add Mugdh Chauhan to Merge Team #7939#8062
LRenDO merged 1 commit intohackforla:gh-pagesfrom
githelsui:update-leadership-7939

Conversation

@githelsui
Copy link
Copy Markdown
Member

Fixes #7939

What changes did you make?

  • Update leadership variable to include new Merge Team member on HackForLA website

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

  • To work on my first issue
  • To update the leadership information on the website with a new Merge Team Member

Screenshots of Proposed Changes To The Website

Visuals before changes are applied

image

Visuals after changes are applied

image

@githelsui githelsui added good first issue Good for newcomers 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) size: 0.25pt Can be done in 0.5 to 1.5 hours labels Apr 13, 2025
@githelsui githelsui self-assigned this Apr 13, 2025
@github-actions
Copy link
Copy Markdown

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 githelsui-update-leadership-7939 gh-pages
git pull https://github.com/githelsui/website.git update-leadership-7939

Copy link
Copy Markdown
Member

@santiseccovidal santiseccovidal left a comment

Choose a reason for hiding this comment

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

Hi @githelsui everything looks great!
Branches are correct, well done with screenshots and code changes are correct.
Great job working on your first issue!

Copy link
Copy Markdown
Member

@owaisjunedi owaisjunedi left a comment

Choose a reason for hiding this comment

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

Things Done Well

  • The issue has the correct 'commit into': "hackforla:gh-pages".
  • The linked issue is present: 'Fixes #7939 '.
  • The changes made are applicable and clean.
  • Changes were viewable in browser.
  • Screenshots provided are clear & correct

Great job completing your issue, Githelsui! @githelsui

@github-project-automation github-project-automation Bot moved this from PR Needs review to PRs ✅ waiting for merge team in P: HfLA Website: Project Board Apr 16, 2025
Copy link
Copy Markdown
Member

@RodgerLugo RodgerLugo left a comment

Choose a reason for hiding this comment

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

Hey,
I tested locally and confirmed that Mugdh Chauhan appears correctly on the team page. The image, role, and link all look good.
Nice work!

@LRenDO LRenDO merged commit 12f5aa1 into hackforla:gh-pages Apr 16, 2025
5 of 7 checks passed
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: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours

Projects

Development

Successfully merging this pull request may close these issues.

Update Project Profile: Website- add Mugdh Chauhan to Merge Team

5 participants