Skip to content

Updating github handle for Eli Selkin in engage.md#6900

Merged
gaylem merged 2 commits intohackforla:gh-pagesfrom
vanessasinam:update-handle-6731
May 31, 2024
Merged

Updating github handle for Eli Selkin in engage.md#6900
gaylem merged 2 commits intohackforla:gh-pagesfrom
vanessasinam:update-handle-6731

Conversation

@vanessasinam
Copy link
Member

@vanessasinam vanessasinam commented May 27, 2024

Fixes #6731

What changes did you make?

  • Replaced "- name: Eli Selkin" with "- name: Eli Selkin github-handle: "

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

  • The plan is to create a single variable github-handle to hold the github handle for each member of the leadership team.

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

No visual changes to 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 vanessasinam-update-handle-6731 gh-pages
git pull https://github.com/vanessasinam/website.git update-handle-6731

@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 27, 2024
@ha-bach ha-bach self-requested a review May 27, 2024 22:19
ha-bach
ha-bach previously approved these changes May 27, 2024
Copy link
Member

@ha-bach ha-bach left a comment

Choose a reason for hiding this comment

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

Edit since I found the template for PR review and reviewing accordingly.

What works:

  • branching was done correctly
  • the issue number was listed
  • code edit looks correct and only necessary code is added
  • what and why in PR description are well-stated

To review:

  • PR title is not descriptive. I would describe what you changed rather than listing the issue being fixed
  • Good work on listing "No visual changes to website." under "Visuals before/after changes are applied", but the collapsible bullet points could be removed as well, leaving just "No visual changes to website." under "Screenshots of Proposed Changes..."

Overall great work! Thank you for contributing.

@ha-bach ha-bach dismissed their stale review May 27, 2024 23:12

found potential issue that needed to be review

@Thinking-Panda Thinking-Panda requested a review from del9ra May 29, 2024 02:29
@LRenDO
Copy link
Member

LRenDO commented May 29, 2024

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

@del9ra
Copy link
Member

del9ra commented May 30, 2024

Review ETA: 10 PM 5/30/24
Availability: 1-3 PM Friday

Copy link
Member

@del9ra del9ra left a comment

Choose a reason for hiding this comment

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

Hey @vanessasinam,

  • The branching looks good.
  • The issue is linked properly.
  • I would make the title more descriptive
  • Your explanation of the changes is clear.
  • The project's webpage looks the same.
    Nice work, and thanks for contributing to the website!

@jphamtv
Copy link
Member

jphamtv commented May 30, 2024

Hey @vanessasinam, @muninnhugin provided feedback that needs to be addressed. Please make those changes and re-request a review from them by clicking on the circular arrows icon next to their name. Thanks!
Screenshot 2024-05-30 at 11 30 51

@vanessasinam vanessasinam requested a review from ha-bach May 30, 2024 18:34
@ha-bach
Copy link
Member

ha-bach commented May 30, 2024

@vanessasinam I previously reviewed this PR and edited it according to my understanding of how it should look: #6900 (review).
When you make the changes just let me know and I can approve the PR!

@vanessasinam vanessasinam changed the title Update handle 6731 Updating github handle for Eli Selkin in engage.md May 31, 2024
@vanessasinam
Copy link
Member Author

@vanessasinam I previously reviewed this PR and edited it according to my understanding of how it should look: #6900 (review). When you make the changes just let me know and I can approve the PR!

I've just made the changes!

Copy link
Member

@ha-bach ha-bach left a comment

Choose a reason for hiding this comment

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

The PR title is descriptive and the visual changes description is clean!

Thank you for adding the changes, and for contributing!!

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 Eli Selkin in engage.md

6 participants