Skip to content

Edit content field and remove type 2846#3617

Merged
blulady merged 3 commits intohackforla:gh-pagesfrom
gstemmann:edit-content-field-and-remove-type-2846
Oct 14, 2022
Merged

Edit content field and remove type 2846#3617
blulady merged 3 commits intohackforla:gh-pagesfrom
gstemmann:edit-content-field-and-remove-type-2846

Conversation

@gstemmann
Copy link
Member

@gstemmann gstemmann commented Oct 13, 2022

Fixes #2846

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

  • Changed content field key: value pair in _data/internal/credits/locations.yml
  • Removed type field from _data/internal/credits/locations.yml
    Changes were made to reduce redundancy

Screenshots of Proposed Changes Of The Website

No visual changes to the website or photos to attach

@gstemmann gstemmann self-assigned this Oct 13, 2022
@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 gstemmann-edit-content-field-and-remove-type-2846 gh-pages
git pull https://github.com/gstemmann/website.git edit-content-field-and-remove-type-2846

@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 Oct 13, 2022
@gstemmann gstemmann removed their assignment Oct 13, 2022
@MattPereira
Copy link
Contributor

MattPereira commented Oct 13, 2022

Review ETA: 4PM 10/13/22
Availability: 6 hours

Copy link
Contributor

@MattPereira MattPereira left a comment

Choose a reason for hiding this comment

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

Phenomenal work George! I love how you improved the code by removing redundancy. 😎

@jyaymie jyaymie self-requested a review October 13, 2022 23:06
@jyaymie
Copy link
Member

jyaymie commented Oct 13, 2022

Review ETA: 5pm PST 10/13/22
Availability: 1 hour

@jyaymie
Copy link
Member

jyaymie commented Oct 13, 2022

Great work! I see all the changes that needed to be made. I did notice that in your PR, the listed file path is different from that in the issue. I believe the end of the path should be locations.yml instead of meetup.yml.

@gstemmann
Copy link
Member Author

Great work! I see all the changes that needed to be made. I did notice that in your PR, the listed file path is different from that in the issue. I believe the end of the path should be locations.yml instead of meetup.yml.

Thank you so much for catching this! I will change this now.

Copy link
Member

@blulady blulady left a comment

Choose a reason for hiding this comment

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

This branch is coming from the correct branch and going into gh-pages. There is a linked issue, it is correct and addressed in the PR by changing line four to content-type: image and deleting line 11. The code looks clean and there is only one file changed (even though there are multiple commits).
Good job!!

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 locations.yml credits file

4 participants