website icon indicating copy to clipboard operation
website copied to clipboard

Update Github handle for Santiago Vidal

Open Colombianah opened this issue 1 month ago • 3 comments

Fixes #8423

What changes did you make?

  • Updated Github Handle from 'santisecco' to 'santiseccovidal'
  • Updated profile picture link
  • Verified YAML formatting

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

  • The contributor's Github handle had changed, so the previous link was no longer correct.
  • Updating the handle and profile picture ensures the website displays accurate and up-to-date contributor information.

CodeQL Alerts

Please let us know that you have checked for CodeQL alerts. Please do not dismiss alerts.

  • [x ] I have checked this PR for CodeQL alerts and none were found.
  • [ ] I found CodeQL alert(s), and (select one):
    • [ ] I have resolved the CodeQL alert(s) as noted
    • [ ] I believe the CodeQL alert(s) is a false positive (Merge Team will evaluate)
    • [ ] I have followed the Instructions below, but I am still stuck (Merge Team will evaluate)

Screenshots of Proposed Changes To The Website:

Before

Screenshot 2025-12-12 183205

After Screenshot 2025-12-12 183215

No visual changes to the website. This update modifies backend data only.

Colombianah avatar Dec 11 '25 22:12 Colombianah

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 Colombianah-update-Github-handle-8423 gh-pages
git pull https://github.com/Colombianah/website.git update-Github-handle-8423

github-actions[bot] avatar Dec 11 '25 22:12 github-actions[bot]

Review ETA: 12/12/2025 Availability: 2 - 5 PM

ldaws003 avatar Dec 12 '25 20:12 ldaws003

Hey, good job in removing the message. The screenshot doesn't reflect the changes. To see what the profile picture is before you can go to https://www.hackforla.org/projects/website, and to see what the website is after your changes you should run website on your computer, making sure that you're on your branch, and go to http://localhost:4000/projects/website. Also make sure to re-request review again to make sure that future reviewers get a notification for any changes to your PR.

ldaws003 avatar Dec 13 '25 00:12 ldaws003

Review ETA: EOD 12/17/2025 Availability: 4-7 PM Wednesday

gabbyjimenez avatar Dec 16 '25 19:12 gabbyjimenez

The screenshot has been updated as requested. Now waiting for a new PR review. Let me know if there's anything else you need.

Colombianah avatar Dec 17 '25 03:12 Colombianah

oh one thing, since you do have screenshots please remove this line No visual changes to the website. This update modifies backend data only. at the end of your PR comment.

ldaws003 avatar Dec 17 '25 04:12 ldaws003

Hi @Colombianah, everything looks correct, great job! The merge from and to branches have been correctly set. The issue has been correctly linked. Screenshots accurately reflect before and after changes. Code changes have been correctly made.

gabbyjimenez avatar Dec 18 '25 03:12 gabbyjimenez