Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Rachel Ichniowski Profile to Expunge Assist Leadership #7747

Conversation

nmotamedi
Copy link
Member

@nmotamedi nmotamedi commented Nov 20, 2024

Fixes #7702

What changes did you make?

  • Added Name / Role / Links / Picture for Rachel Ichniowski to the expunge-assist.md

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.

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

Visuals before changes are applied Screen Shot 2024-11-19 at 10 03 29 PM
Visuals after changes are applied Screen Shot 2024-11-19 at 10 02 46 PM

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

Copy link

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 nmotamedi-expunge-assist-add-rachel-ichniowski-7702 gh-pages
git pull https://github.com/nmotamedi/website.git expunge-assist-add-rachel-ichniowski-7702

@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 Nov 20, 2024
@mchait18 mchait18 self-requested a review November 20, 2024 16:59
@mchait18
Copy link
Member

Review ETA: 12:30 PM 11/20/24
Availability: 9-3 PM EST M-Th

@codyjohnsontx codyjohnsontx self-requested a review November 20, 2024 17:29
@codyjohnsontx
Copy link
Member

Review ETA: 12:00 PM 11/20/24
Availability: On Call Today: 10am-6 PM CST, 9pm-12midnight CST 11/20

mchait18
mchait18 previously approved these changes Nov 20, 2024
@mchait18
Copy link
Member

Hi @nmotamedi, thanks for taking this issue!

  • the PR contains the correct branch
  • issue is correctly linked
  • changes are verified across browsers (firefox, edge, chrome)
  • code changes are accurate and concise
  • no other notable issues
    Great job!

@codyjohnsontx
Copy link
Member

Thank you for working on this issue, @nmotamedi !

You're doing these right:
The branch you made the change on is correct
The linked issue (original issue) number is included
The content required to add is accurate
The before / after screenshot is helpful to display the change

Changes you need to make:
-Address the questions from HackforLABot in the original issue ( comment your Availability and ETA)
-The CodeQL Alerts are checked (check box above)
-Fix the unnecessary changes (double quotation marks(") instead of the original single quotation marks('), and the spacing on some of the lines) and submit your PR again (It could be caused by the formatting tool in your IDE, such as prettier)
-You can re-request me to review your PR once you make the changes. Feel free to reach out if you have any questions.

@codyjohnsontx
Copy link
Member

Review ETA: 3:00 PM 11/20/24
Availability: On Call Today: 10am-6 PM CST, 9pm-12midnight CST 11/20

codyjohnsontx
codyjohnsontx previously approved these changes Nov 20, 2024
Copy link
Member

@codyjohnsontx codyjohnsontx left a comment

Choose a reason for hiding this comment

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

Nice Work @nmotamedi!
Excellent job fixing the quotation marks and spacing issues in the file
Proper into and from branches
Clear and simple what and why
Correctly added the team members info

Thanks for the PR!

Copy link
Member

@codyyjxn codyyjxn left a comment

Choose a reason for hiding this comment

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

@nmotamedi thank you for taking on this issue.

Some suggestions

  • You need to add this question Why did you make the changes (we will use this info to test)?
  • I believe you have this section twice please remove one Screenshots of Proposed Changes To The Website (if any, please do not include screenshots of code changes)

@nmotamedi
Copy link
Member Author

Thank you, @codyyjxn. I have made the updates requested.

@nmotamedi nmotamedi requested a review from codyyjxn November 21, 2024 17:17
@belunatic belunatic self-requested a review November 21, 2024 19:15
@belunatic
Copy link
Member

Review ETA: 10 AM 11/22/24
Availability: 4-5 PM Monday

belunatic
belunatic previously approved these changes Nov 21, 2024
Copy link
Member

@belunatic belunatic left a comment

Choose a reason for hiding this comment

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

@nmotamedi great job and thank you for fixing the previous issues

  • The issue name looks great
  • the before and after images a re spot on
  • issue number is great

Suggestions

  • I like that you dded the file name you changed, but it will be great if you also added the folder name i.e: _project/expung_assist, to locate it faster

Besides that keep up the good work!

@rachelcorawood rachelcorawood self-requested a review November 21, 2024 22:22
@rachelcorawood
Copy link
Member

Review ETA: 7PM 11/21/24
Availability: M-F noon-3pm

rachelcorawood
rachelcorawood previously approved these changes Nov 22, 2024
Copy link
Member

@rachelcorawood rachelcorawood left a comment

Choose a reason for hiding this comment

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

You linked the issue, understand the linked issues, changes can be viewed, and your code looks clean and efficient with no bugs. Great job!

@jphamtv
Copy link
Member

jphamtv commented Nov 22, 2024

Hi @nmotamedi, there's a conflict in _projects/expunge-assist.md from a recent merge that needs resolving. Once you resolve it, we'll get this merged in. Check out How to Resolve Merge Conflicts for help.

Copy link
Member

@codyyjxn codyyjxn left a comment

Choose a reason for hiding this comment

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

Hey @nmotamedi great job on fixing these issues.
Keep it up!

@dvernon5
Copy link
Member

Review ETA: 7:00 PM 11/26/2024
Availability: 9AM - 6PM Tuesday & Wednesday of this week.

Copy link
Member

@dvernon5 dvernon5 left a comment

Choose a reason for hiding this comment

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

Hello @nmotamedi

Branch name looks good.
The code changes looks suburb.
This issue is linked correctly.
CodeQL alerts are good.
The content added is correct.
Before and After photo look stunning

Your PR is absolutely perfect. Great job!

@t-will-gillis t-will-gillis merged commit 59bdbdd into hackforla:gh-pages Nov 27, 2024
3 checks passed
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
Development

Successfully merging this pull request may close these issues.

Update Project Profile: Expunge Assist Add Rachel Ichniowski
9 participants