add github-handle variable for Gian Reyes Dionisio

[williamzhang73]

Fixes #6937

What changes did you make?

-add github-handle: for Gian Reyes Dionisio

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

-add Github handle variable for Gian Reyes Dinnisio to reduce redundancy in the project file.

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

-No visual change on the website.

[github-actions]

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 williamzhang73-add-github-handle-6937 gh-pages
git pull https://github.com/williamzhang73/website.git add-github-handle-6937

[dcotelessa]

Hi williamzhang,

Thanks for the PR. We need a few updates:

• In the PR description, we need to replace "Fixes #replace_this_text_with_the_issue_number" with the number of the ticket, linking the task to the ticket. This is described here
• Also, we can remove the "Screenshots" portion of the PR description as there are no visual changes.
• The github-handle is applied and left blank , as the task is described.

Look forward to the update, thanks!

[williamzhang73]

Thanks @dcotelessa,

1. The issue number was updated.
2. The "screenshots" part of the PR description was removed.
   please review it again.

[dcotelessa]

Thanks for the updates.
The description looks good and the passing all tests.
Great job!

[andyphancode]

Review ETA: 6/11/2024 EOD
Availability: Throughout week

[andyphancode]

Looks good!

• Correctly merging from collaborator's branch into hackforla:gh-pages
• Issue is correctly fixed (github-handle: added)

Approved.

[ino-iosdev]

@williamzhang73

• Issue number was properly linked
• Branch naming is good (just a side note for future reference, no need to make change on this one: branch naming is more clear if you could include the name of the person, e.g. add-github-handle-for-bryon-heart-6730)

Changes required:

1. Remove 6937 from the title (notice how it becomes unclear since it stacks with your PR number)

2. Use syntax highlight when referencing changes made so that it's clear to read, e.g. below:

3. Revise the Why did you make the changes answer: In general, it's a good practice to write details in your own words based on the request from the issue, e.g. Added Github-handle for Bryon Heart #6917

4. Please include this at the end if no visual change is made to the website:

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

-  No visual change on the website. 

After making all these changes, don't forget to click on something like this on the top right of your page (see below) to request a review of your changes:

[ino-iosdev]

Looks good!

• Correctly merging from collaborator's branch into hackforla:gh-pages

• Issue is correctly fixed (github-handle: added)

Approved.

@andyphancode Just noticed that your view mentioned merging, since this is a pushing only, and the merge team will be handling merging once it's approved overall, I would kindly suggest to say correctly pushing instead to avoid confusing. Hope this make sense, cheers!

[del9ra]

ETA: 06/13/24
Availability: 1-5 PM monday-friday

[del9ra]

HI @williamzhang73 Well done! Good job!

• The branching was done correctly
• No need to mention issue number in the PR title. Refer to this guide https://github.com/hackforla/website/blob/gh-pages/CONTRIBUTING.md
• Issue number was listed
• Not all pull requests will have significant changes to our website. Please just add the following explanation for no images like:
  ### Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)
  No visual changes to the website.
  Please refer to the Contributing Guide: https://github.com/hackforla/website/blob/gh-pages/CONTRIBUTING.md
• Changes were made correctly in the code
• Please provide a more detailed explanation of why the changes are being made
  Thank you! Just make these changes, and I'll be happy to approve it.

[jphamtv]

Hi @williamzhang73 - Please update the "What changes did you make?" summary according to @ino-iosdev's feedback:

2. Use syntax highlight when referencing changes made so that it's clear to read, e.g. below:

Tip: Remove the underscores (_), and use backticks (`) instead

Once updated, ping @ino-iosdev to review and approve the changes. This PR is ready to be merged and is only waiting for this final update. Thanks!

The requested changes have been made

[ino-iosdev]

Thank you for the changes!