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

Update Expunge Assist with new description, add tools, wiki and overview #7446

Conversation

FamousHero
Copy link
Member

Fixes #7379

What changes did you make?

  • Add Expunge-Assist-One-Sheet.pdf to pdfs folder to link in _projects/expunge-assist.md
  • Update description section in _projects/expunge-assist.md
  • Add Overview and Wiki entry within links section in _projects/expunge-assist.md

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

  • To keep the website up to date with the projects current tools, description, and relevant links.

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

Visuals before changes are applied

Project_Page_Before

Project_Description_Before

Project_Links_Before

Project Tools_before

Visuals after changes are applied

Project_Page_After

Project_Description_After

Project_Links_After

Project Tools_After

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 FamousHero-update-project-profile-expunge-assist-7379 gh-pages
git pull https://github.com/FamousHero/H4LAwebsite.git update-project-profile-expunge-assist-7379

@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 Sep 14, 2024
@aadilahmed aadilahmed self-requested a review September 14, 2024 05:17
@aadilahmed
Copy link
Member

Review ETA: 9/13/24
Availability: 9-5 PM Monday-Friday

Copy link
Member

@aadilahmed aadilahmed left a comment

Choose a reason for hiding this comment

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

You've correctly edited the Expunge Assist project description, added the list of tools to the project card and detail page, and the Overview and Wiki links lead to the correct documents. Great job @FamousHero !

@codyyjxn codyyjxn self-requested a review September 14, 2024 21:38
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.

@FamousHero Great Job! The pr looks good.

  • PR is made from the correct branch
  • The correct file _projects/expunge-assist.md was edited
  • No extra edits were made

@t-will-gillis t-will-gillis merged commit 1686b9a into hackforla:gh-pages Sep 15, 2024
6 checks passed
@FamousHero FamousHero deleted the update-project-profile-expunge-assist-7379 branch September 26, 2024 21:30
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: Expunge Assist with new description, add tools, wiki and overview links
4 participants