Skip to content

Updating github handle for Dennis Chatkhan#6781

Merged
jphamtv merged 1 commit intohackforla:gh-pagesfrom
vanessasinam:update-handle-6181
May 2, 2024
Merged

Updating github handle for Dennis Chatkhan#6781
jphamtv merged 1 commit intohackforla:gh-pagesfrom
vanessasinam:update-handle-6181

Conversation

@vanessasinam
Copy link
Member

@vanessasinam vanessasinam commented Apr 29, 2024

Fixes #6181

What changes did you make?

  • Replaced "- name: Dennis Chatkhan" with "- name: Dennis Chatkhan 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)

Visuals before changes are applied Screenshot 2024-04-29 at 3 09 26 PM
Visuals after changes are applied Screenshot 2024-04-29 at 3 13 51 PM

@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-6181 gh-pages
git pull https://github.com/vanessasinam/website.git update-handle-6181

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

https://github.com/vanessasinam/website/blob/update-handle-6181/CONTRIBUTING.md  

@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) status: To Update! No update has been provided size: 0.25pt Can be done in 0.5 to 1.5 hours labels Apr 29, 2024
@elisetvy elisetvy self-requested a review April 29, 2024 20:45
Copy link
Member

@elisetvy elisetvy left a comment

Choose a reason for hiding this comment

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

Hi @vanessasinam,

  1. The pull request is done with the correct branch
  2. The issue is properly linked
  3. Your summary of changes is clear
  4. Your code change correctly uses spaces instead of tabs to indent
  5. The appearance of the project webpage is unchanged

Please check off the Action Items when working on issues to indicate you have completed them, then I will be happy to approve!

Thanks for your contribution!

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!
Review ETA: 2 PM 4/30/24
Availability: 5-8 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.

  1. The PR is done with the correct branch
  2. The issue is linked correctly
  3. spaces were used rather than tabs
  4. The summary is clear
  5. The code change is clean
  6. The visual aspect of the project webpage hasn't changed
    Nice job!

@vanessasinam vanessasinam removed the status: To Update! No update has been provided label May 2, 2024
@vanessasinam
Copy link
Member Author

Hi @vanessasinam,

  1. The pull request is done with the correct branch
  2. The issue is properly linked
  3. Your summary of changes is clear
  4. Your code change correctly uses spaces instead of tabs to indent
  5. The appearance of the project webpage is unchanged

Please check off the Action Items when working on issues to indicate you have completed them, then I will be happy to approve!

Thanks for your contribution!

Hi @elisetvy I have updated the Action items!

@jphamtv
Copy link
Member

jphamtv commented May 2, 2024

Nice work on this @vanessasinam. For future issues, if there are no visual changes to the website, you can replace the "Visual changes before/after changes are applied" sections with a comment like "No visual changes to website".

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 Dennis Chatkhan in civic-tech-index.md

4 participants