Skip to content

Update expunge-assist.md with two UX Research Co-leads - #8356

Merged
t-will-gillis merged 13 commits into
hackforla:gh-pagesfrom
kdaca19xx:update-expunge-assist-ux-research-coleads-8350
Oct 6, 2025
Merged

t-will-gillis merged 13 commits into
hackforla:gh-pagesfrom
kdaca19xx:update-expunge-assist-ux-research-coleads-8350

Conversation

@kdaca19xx

@kdaca19xx kdaca19xx commented Oct 1, 2025 •

Copy link
Copy Markdown
Member

Fixes #8350

What changes did you make?

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

  • We need to keep project information up to date so that visitors to the website can find accurate information.

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 8350-before
Visuals after changes are applied 8350-after

@github-actions

github-actions Bot commented Oct 1, 2025

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 kdaca19xx-update-expunge-assist-ux-research-coleads-8350 gh-pages
git pull https://github.com/kdaca19xx/website.git update-expunge-assist-ux-research-coleads-8350

@github-actions github-actions Bot added 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 Complexity: Small Take this type of issues after the successful merge of your second good first issue Ready for Prioritization size: 0.25pt Can be done in 0.5 to 1.5 hours labels Oct 1, 2025
@myronchen-git
myronchen-git self-requested a review October 2, 2025 01:56
@myronchen-git

Copy link
Copy Markdown
Member

Availability: 4 - 7 PM, 10/2/2025
ETA: 11 PM 10/2/2025

@santi-jose

Copy link
Copy Markdown
Member

Availability: 5-8PM Monday - Friday
ETA: 6PM 10/3/2025

@santi-jose
santi-jose self-requested a review October 2, 2025 05:40

@myronchen-git myronchen-git left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Good job on completing this small complexity issue.


Done Correctly

  • The merge from and to branches have been correctly set.
  • The issue has been correctly linked.
  • The pull request title is named properly.
  • The whats and whys in the pull request description is well written.
  • The CodeQL alerts have been correctly checked off.
  • Screenshots accurately reflect before and after changes.
  • Code changes have been correctly made.
  • In the issue, you've properly checked off action items and gave your availability and ETA.

Other Notes

  • In the issue, Will changed the status from "In progress" to "New Issue Approval". You might want to clarify this with Will before moving on.

@santi-jose santi-jose left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hi @kdaca19xx , good job working on this issue!

Things Done Well

  • Pull request is done in the correct branch
  • Issue is linked properly
  • Images of changes are included in pull request
  • Changes made are applicable to the issue
  • Changes preserve the integrity of the website
  • Source code was changed according to issue
  • What & Why sections of pull request are answered with sufficient detail
  • CodeQL Alerts were checked

Changes to be Made

  • None

I approve this PR!

@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 Oct 3, 2025
@t-will-gillis
t-will-gillis merged commit e88957d into hackforla:gh-pages Oct 6, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complexity: Small Take this type of issues after the successful merge of your second good first issue 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

Development

Successfully merging this pull request may close these issues.

Update Project Profile: Expunge Assist Add Namitha Murali and Prakriti Dumaru

4 participants