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

Changed content field to content-type and removed type field in credits/404 yml file #3158

Merged
merged 1 commit into from
May 20, 2022
Merged

Changed content field to content-type and removed type field in credits/404 yml file #3158

merged 1 commit into from
May 20, 2022

Conversation

scorbz9
Copy link
Member

@scorbz9 scorbz9 commented May 18, 2022

Fixes #2633

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

  • Updated content field to read content-type: image
  • Deleted type field from file
  • This will make the code easier to understand for other developers and less redundant

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

No visual updates 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-update-credits-yml-template-2642 gh-pages
git pull https://github.com/scorbz9/website.git update-credits-yml-template-2642

@github-actions github-actions bot added Collaborative Work Feature: Wiki role: front end Tasks for front end developers size: 1pt Can be done in 4-6 hours Complexity: Medium good first issue Good for newcomers P-Feature: Credit https://www.hackforla.org/credits/ size: 0.5pt Can be done in 3 hours or less and removed Complexity: Medium Collaborative Work Feature: Wiki size: 1pt Can be done in 4-6 hours labels May 18, 2022
@answebdev answebdev self-requested a review May 18, 2022 23:02
@answebdev
Copy link
Member

ETA: 5/18/2022 (end of day)
Availability: 2 hours

Copy link
Member

@answebdev answebdev left a comment

Choose a reason for hiding this comment

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

@scorbz9 Thank you for submitting your PR. For this issue, content: Image needed to be changed to content-type: image (line 4), and type: icon needed to be removed (line 11) for the file _data/internal/credits/404.yml, which I can see that you did. There is a linked issue, and you have also added the PR to the project board. Great job!

@harshitasao harshitasao self-requested a review May 19, 2022 08:00
@harshitasao
Copy link
Member

Availability: 10pm-11pm IST
Review ETA: 19/05/2022 (End of the day)

@tamara-snyder tamara-snyder self-requested a review May 20, 2022 02:37
@tamara-snyder
Copy link
Member

Availability: 1 hour
Review ETA: Thursday, May 19

@tamara-snyder
Copy link
Member

Hi @scorbz9, great work on this! I can see that you replaced content: Image with content-type: image and removed the redundant type: icon line. Well done!

@tamara-snyder tamara-snyder merged commit 46956d1 into hackforla:gh-pages May 20, 2022
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: 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 404.yml credits file
4 participants