Skip to content

Changed content to content-type: image and removed type: icon#3659

Merged
arpitapandya merged 1 commit intohackforla:gh-pagesfrom
h3nry-m:update-content-type-field-2857
Oct 26, 2022
Merged

Changed content to content-type: image and removed type: icon#3659
arpitapandya merged 1 commit intohackforla:gh-pagesfrom
h3nry-m:update-content-type-field-2857

Conversation

@h3nry-m
Copy link
Member

@h3nry-m h3nry-m commented Oct 25, 2022

Fixes #2857

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

  • Changed the content-type to image
  • Removed type: icon

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

Changed types, did not actually alter any images.

@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 h3nry-m-update-content-type-field-2857 gh-pages
git pull https://github.com/h3nry-m/website.git update-content-type-field-2857

@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 25, 2022
@jyaymie jyaymie self-requested a review October 25, 2022 22:09
@jyaymie
Copy link
Member

jyaymie commented Oct 25, 2022

Review ETA: 10/25/22 @ 4:00pm
Availability: 1 hour

Copy link
Member

@jyaymie jyaymie left a comment

Choose a reason for hiding this comment

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

Excellent work, @h3nry-m! The PR was done on the correct branch and all the requested changes from the linked issue were made. This is a super small thing, but in the next PR, I'd include a brief reason for whatever changes you make.

@MattPereira MattPereira self-requested a review October 25, 2022 23:59
@MattPereira
Copy link
Contributor

Review ETA: 10/25/22 @ 5:30pm
Availability: 1 hour

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.

Great job @h3nry-m ! 👍

You linked the correct issue in the PR and removed the redundant code making it easier to understand! Overall solid work, but I do agree with @jyaymie that it is good to get in the habit of remembering to provide the rationale behind your changes.

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

4 participants