Skip to content

Remove Ren Demeis-Ortiz from project profile on hfla-site page#8633

Open
castillios wants to merge 1 commit into
hackforla:gh-pagesfrom
castillios:update-project-profile-8450
Open

Remove Ren Demeis-Ortiz from project profile on hfla-site page#8633
castillios wants to merge 1 commit into
hackforla:gh-pagesfrom
castillios:update-project-profile-8450

Conversation

@castillios
Copy link
Copy Markdown
Member

@castillios castillios commented May 10, 2026

Fixes #8450

What changes did you make?

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

  • In order to keep information accurate and up to date for all Hack for LA site visitors

CodeQL Alerts

After the PR has been submitted and the resulting GitHub actions/checks have been completed, developers should check the PR for CodeQL alert annotations.

Check the PR's comments. If present on your PR, the CodeQL alert looks similar as shown

Screenshot 2024-10-28 154514

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

  • 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)
Instructions for resolving CodeQL alerts

If CodeQL alert/annotations appear, refer to How to Resolve CodeQL alerts.

In general, CodeQL alerts should be resolved prior to PR reviews and merging

Screenshots of Proposed Changes To The Website (if any, please do not include screenshots of code changes)

Visuals before changes are applied Before
Visuals after changes are applied After

@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 castillios-update-project-profile-8450 gh-pages
git pull https://github.com/castillios/website.git update-project-profile-8450

@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) time sensitive Needs to be worked on by a particular timeframe size: 0.25pt Can be done in 0.5 to 1.5 hours labels May 10, 2026
@viet456 viet456 self-requested a review May 10, 2026 06:19
@viet456
Copy link
Copy Markdown
Member

viet456 commented May 10, 2026

Review ETA: 11:30PM PST 5/9/26

Copy link
Copy Markdown
Member

@viet456 viet456 left a comment

Choose a reason for hiding this comment

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

Hi Castillios,
The changes look good from here. Screenshots of changes provided, code checks passed, branch name, and progress checkboxes ticked off.
Thanks for your contributions

@mocodesss mocodesss self-requested a review May 11, 2026 23:27
@mocodesss
Copy link
Copy Markdown
Member

ETA: 5/12 EOD
Availability: Tuesday am, Tuesday evening, Thursday am, Weekend at various times

@nathanjkim-codes
Copy link
Copy Markdown
Member

Review ETA: 5/11 6PM
Availability: Monday, Thursday 4:00 - 6:00 PST

Copy link
Copy Markdown
Member

@nathanjkim-codes nathanjkim-codes left a comment

Choose a reason for hiding this comment

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

I tested the changes locally and confirmed that the profile was removed correctly from the Website project page.

The linked issue and branch setup look correct, and the requested updates appear to be completed properly.

Well done Castillios!. Thank you for your contribution!

@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 May 12, 2026
Copy link
Copy Markdown
Member

@mocodesss mocodesss left a comment

Choose a reason for hiding this comment

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

Tested locally and it's looking great. Thanks for the before and after pictures. This was organized and easy follow. Nice job!

PR checklist:
Done correctly:

  • The merge from and to branches have been correctly set.
  • The issue has been correctly linked.
  • In the issue, you've correctly assigned yourself.
  • The CodeQL alerts have been correctly checked off.
  • In the issue, you've correctly set the project status.
  • Visuals before/after changes are applied.
  • In the issue, action items have correctly been checked off.

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 time sensitive Needs to be worked on by a particular timeframe

Projects

Status: PRs ✅ waiting for merge team

Development

Successfully merging this pull request may close these issues.

Update Project Profile: Website- Remove Ren Demeis-Ortiz

4 participants