Skip to content

Removed unused alt-hero from _projects/access-the-data.md#3169

Merged
Jaretzbalba merged 1 commit intohackforla:gh-pagesfrom
scorbz9:remove-alt-hero-field-2920
May 24, 2022
Merged

Removed unused alt-hero from _projects/access-the-data.md#3169
Jaretzbalba merged 1 commit intohackforla:gh-pagesfrom
scorbz9:remove-alt-hero-field-2920

Conversation

@scorbz9
Copy link
Member

@scorbz9 scorbz9 commented May 23, 2022

Fixes #2920

What changes did you make and why did you make them ?

  • Removed the unused alt-hero field from _projects/access-the-data.md (line 11)
  • This will make the website load faster, as well as make the code easier to understand and take less time to review

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

No visual changes made to website.

@github-actions
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 scorbz9-remove-alt-hero-field-2920 gh-pages
git pull https://github.com/scorbz9/website.git remove-alt-hero-field-2920

@github-actions github-actions bot added 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 Size: Good second issue labels May 23, 2022
@kathrynsilvaconway kathrynsilvaconway self-requested a review May 24, 2022 01:55
@kathrynsilvaconway
Copy link
Member

ETA: End of day 5/23
Availability: One Hour

@Jaretzbalba Jaretzbalba self-requested a review May 24, 2022 02:06
@Jaretzbalba
Copy link
Member

ETA: End of day 5/23
Availability: One Hour

Copy link
Member

@Jaretzbalba Jaretzbalba left a comment

Choose a reason for hiding this comment

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

@scorbz9 Great work on this issue! I see that the the alt-hero property was deleted and everything looks good.

Copy link
Member

@kathrynsilvaconway kathrynsilvaconway left a comment

Choose a reason for hiding this comment

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

@scorbz9 Nice work on this issue. I see you have deleted line 11 from the access-the-data md file. Everything looks good!

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.5pt Can be done in 3 hours or less

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove unused alt-hero field from access-the-data.md project file

4 participants