Skip to content

Conversation

@hoanhua14
Copy link
Member

@hoanhua14 hoanhua14 commented Mar 3, 2025

Fixes #7868

What changes did you make?

  • Remove from leadership
    Name: Manju Chawla - UX Reseacher

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

  • To update project profile of Civic Tech Jobs

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 Screenshot 2025-03-02 at 7 01 00 PM

@github-actions
Copy link

github-actions bot commented Mar 3, 2025

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 hoanhua14-update-project-profile-7868 gh-pages
git pull https://github.com/hoanhua14/hackforlawebsite.git update-project-profile-7868

@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 size: 0.5pt Can be done in 3 hours or less labels Mar 3, 2025
@rogerioduenas rogerioduenas self-requested a review March 3, 2025 06:21
@rogerioduenas
Copy link
Member

Availability: Monday - Friday, 10:00 AM to 4:00 PM JST
ETA: Friday by 4:00 PM JST

Copy link
Member

@rogerioduenas rogerioduenas left a comment

Choose a reason for hiding this comment

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

Hi @hoanhua14 !

Great job with this pull request!

  • The branching was done correctly.
  • Issue number was listed.
  • The PR title is descriptive of the changes.
  • Changes were made correctly in the code.
  • Changes appear correctly on the site - Manju Chawla was Removed from the Civic Tech Jobs page.
  • Relevant images were included to show visual changes.
  • The PR request clearly states what was updated.

Again, great work and thanks for taking the time to contribute to the website!

@jphamtv
Copy link
Member

jphamtv commented Mar 7, 2025

Hi @hoanhua14 - I noticed you've assigned yourself this issue from the 'New Issue Approval' column. I'm closing this PR and unassigning you from the issue because items in this column still need to be reviewed, prepared, and approved before they're ready to be worked on.

Please only self-assign and work on issues from the Prioritized backlog column in the Project Board. Please reach out on Slack if you have any questions. Thanks!

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.5pt Can be done in 3 hours or less 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: [Civic Tech Jobs] Update Project Profile: [Manju Chawla - UX Researcher]

3 participants