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

Make "Learn more" link visible on Redesigned Homepage and link it to About Page #7793

Conversation

belunatic
Copy link
Member

@belunatic belunatic commented Dec 4, 2024

Fixes #7689

What changes did you make?

  • I made changes to the file _includes/home-page/hero.html
  • I deleted a comment
  • I made the Learn more link visible by uncommenting it

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

  • Because the author wanted the Learn more link to be visible and link to the About US page on the redesigned homepage so that visitors can read more information about Hack for LA.

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

FireShot Capture 080 - Hack for LA - localhost

Visuals after changes are applied

FireShot Capture 079 - Hack for LA - localhost

Copy link

github-actions bot commented Dec 4, 2024

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 belunatic-make-learn-more-link-visible-on-redesign-homepage feature-homepage-launch
git pull https://github.com/belunatic/website.git make-learn-more-link-visible-on-redesign-homepage

@github-actions github-actions bot added role: front end Tasks for front end developers P-Feature: About Us https://www.hackforla.org/about/ P-Feature: Home page https://www.hackforla.org/ Complexity: Small Take this type of issues after the successful merge of your second good first issue size: 0.25pt Can be done in 0.5 to 1.5 hours labels Dec 4, 2024
@codyyjxn codyyjxn self-requested a review December 5, 2024 04:24
@siyunfeng
Copy link
Member

@codyyjxn , please provide your Review ETA and Availability when assigning yourself

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.

Things that went well:

-The file change is correct with uncomment code.
-The title is concise and the description is also clear.

Some suggestions

  • You need to change the P: HfLA Website: Project Board : Prioritized backlog to In Progress

@belunatic
Copy link
Member Author

@codyyjxn Thank you for that I went ahead and changed the status.

@minkang3 minkang3 assigned minkang3 and unassigned minkang3 Dec 9, 2024
@minkang3 minkang3 self-requested a review December 9, 2024 21:15
@minkang3
Copy link
Member

minkang3 commented Dec 9, 2024

Eta

Within the hour

Availability

5-9 pm

Copy link
Member

@minkang3 minkang3 left a comment

Choose a reason for hiding this comment

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

What looks good

  • What changes you made look good
  • Reason for changes look good
  • Branch name looks good
  • Code changes looks good
  • Visual changes looks good

Everything looks good to me.

@mchait18 mchait18 self-requested a review December 10, 2024 16:08
@mchait18
Copy link
Member

Review ETA: 12:30 PM 12/10/24
Availability: 9-3 EST

Copy link
Member

@mchait18 mchait18 left a comment

Choose a reason for hiding this comment

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

Thank you for working on this issue, @belunatic !
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 be removed is accurate
The CodeQL Alerts are checked (check box above)
The screenshots are clear

Great job!

Copy link
Member

@DrAcula27 DrAcula27 left a comment

Choose a reason for hiding this comment

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

@belunatic Great job on your Small Issue! Your code passed the ABC's, including:

  • PR is made from the correct branch
  • PR links to the correct issue:
  • The correct file _includes/home-page/hero.html was edited
  • No extra edits were made
  • Commit is in the correct feature branch
  • Appearance of the website looks good in Chrome and Firefox
    Approved! 👍

@FamousHero FamousHero merged commit 9bb311e into hackforla:feature-homepage-launch Dec 13, 2024
6 checks passed
@belunatic belunatic deleted the make-learn-more-link-visible-on-redesign-homepage branch December 17, 2024 20:49
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: About Us https://www.hackforla.org/about/ P-Feature: Home page https://www.hackforla.org/ role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours
Projects
Development

Successfully merging this pull request may close these issues.

7 participants