Skip to content

Edit Content Field and Remove Type Field From yml Credit Files - #2863#3798

Merged
jdingeman merged 2 commits intohackforla:gh-pagesfrom
Adaakal:edit-content-field-remove-type-field-2863
Jan 11, 2023
Merged

Edit Content Field and Remove Type Field From yml Credit Files - #2863#3798
jdingeman merged 2 commits intohackforla:gh-pagesfrom
Adaakal:edit-content-field-remove-type-field-2863

Conversation

@Adaakal
Copy link
Member

@Adaakal Adaakal commented Jan 6, 2023

Fixes #2863

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

For the file _data/internal/credits/star.yml:

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

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

Details Edited a .yml file. No visual changes to the website.

@github-actions
Copy link

github-actions bot commented Jan 6, 2023

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 Adaakal-edit-content-field-remove-type-field-2863 gh-pages
git pull https://github.com/Adaakal/website.git edit-content-field-remove-type-field-2863

@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 status: Updated No blockers and update is ready for review labels Jan 6, 2023
@gstemmann
Copy link
Member

Review ETA: 6 PM 1/12/23
Availability: 5-8 PM Monday

@jdingeman
Copy link
Member

Review ETA: EOD January 8, 2023
Availability: 4 hrs

Copy link
Member

@jdingeman jdingeman left a comment

Choose a reason for hiding this comment

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

Hi Adaugo, nice job. I would go ahead and remove the space from line 11 all together so it's uniform with the other files like this one.

@gstemmann
Copy link
Member

Hi Adaugo, nice job. I would go ahead and remove the space from line 11 all together so it's uniform with the other files like this one.

Yes, awesome job with your first good issue. When you take out line 11 we can approve the PR :)

@Adaakal Adaakal requested a review from jdingeman January 10, 2023 17:14
@Adaakal
Copy link
Member Author

Adaakal commented Jan 10, 2023

@gstemmann @jdingeman thank you for the reviews. I've removed the spaces.

Copy link
Member

@jdingeman jdingeman left a comment

Choose a reason for hiding this comment

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

Nice job!

@jdingeman jdingeman merged commit b58bd14 into hackforla:gh-pages Jan 11, 2023
@Adaakal
Copy link
Member Author

Adaakal commented Jan 18, 2023

Thank you!

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 status: Updated No blockers and update is ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

3 participants