Skip to content

merging content and type to content-type in building.yml#3168

Merged
Jaretzbalba merged 1 commit intohackforla:gh-pagesfrom
mmogri:update-content-bldg-yml-2797
May 24, 2022
Merged

merging content and type to content-type in building.yml#3168
Jaretzbalba merged 1 commit intohackforla:gh-pagesfrom
mmogri:update-content-bldg-yml-2797

Conversation

@mmogri
Copy link
Member

@mmogri mmogri commented May 23, 2022

Fixes #2797

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

  • Changed line 4 for the content field from content: icon to content-type: image
  • Removed line 11 for the type field type: icon

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

Visuals before changes are applied

image

Visuals after changes are applied

image

@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 mmogri-update-content-bldg-yml-2797 gh-pages
git pull https://github.com/mmogri/website.git update-content-bldg-yml-2797

@github-actions github-actions bot added good first issue Good for newcomers P-Feature: Credit https://www.hackforla.org/credits/ 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 labels May 23, 2022
@Jaretzbalba Jaretzbalba self-requested a review May 23, 2022 15:21
@Jaretzbalba
Copy link
Member

ETA: End of the day on 5/23
Availability: 2 hours

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.

@mmogri The changes on the building.yml file look great! Thank you for working on this issue!

@kathrynsilvaconway kathrynsilvaconway self-requested a review May 24, 2022 00:50
@kathrynsilvaconway
Copy link
Member

ETA: End of the day on 5/23
Availability: 1 hours

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.

@mmogri Congrats on completing your first issue with Hack for LA. I see that you have changed the content field in line 4 to read 'content-type: image' and you have deleted line 11 from the building yml file. Nice work.

@Jaretzbalba Jaretzbalba merged commit d52b254 into hackforla:gh-pages May 24, 2022
@KBWells77 KBWells77 self-requested a review May 24, 2022 02:03
@KBWells77
Copy link
Member

ETA: End of day on 5/23
Availability: 1 hour

Copy link
Member

@KBWells77 KBWells77 left a comment

Choose a reason for hiding this comment

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

Looks great @mmogri , great job fixing the error.

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: Credit https://www.hackforla.org/credits/ 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

Development

Successfully merging this pull request may close these issues.

Edit content field and remove type field from building.yml credits file

4 participants