Skip to content

Updated content field and removed type field in internal/rocket.yml#3728

Merged
arpitapandya merged 2 commits intohackforla:gh-pagesfrom
wendybarrios:update-rocket.yml-credits-2873
Nov 17, 2022
Merged

Updated content field and removed type field in internal/rocket.yml#3728
arpitapandya merged 2 commits intohackforla:gh-pagesfrom
wendybarrios:update-rocket.yml-credits-2873

Conversation

@wendybarrios
Copy link
Contributor

@wendybarrios wendybarrios commented Nov 17, 2022

Fixes #2873

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

  • Updated "content: icon" field to "content-type: image"
  • Removed type field

Screenshots of Proposed Changes Of The Website

No visual changes to the 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 wendybarrios-update-rocket.yml-credits-2873 gh-pages
git pull https://github.com/wendybarrios/website.git update-rocket.yml-credits-2873

@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 Nov 17, 2022
@angelenelm angelenelm self-requested a review November 17, 2022 02:38
@angelenelm
Copy link
Member

Review ETA: 7PM PST 11/16/2022
Availability: 10AM - 5PM PST M-F

angelenelm
angelenelm previously approved these changes Nov 17, 2022
Copy link
Member

@angelenelm angelenelm left a comment

Choose a reason for hiding this comment

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

The requested changes to rocket.yml in the issue are addressed correctly, and are done on the correct branch. Good job @wendybarrios :)

@Adastros Adastros self-requested a review November 17, 2022 05:03
@Adastros
Copy link
Member

Review ETA: 11PM 16NOV22
Availability: 9 - 11 PM

Copy link
Member

@Adastros Adastros left a comment

Choose a reason for hiding this comment

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

Good job @wendybarrios! You completed the action items successfully!

One thing I would like to note is that line 11 is empty. I don't believe this affects anything on the site but to keep it consistent with the structure of the other credit files, I think it would be best to remove the empty line.

What are your thoughts @angelenelm?

@angelenelm
Copy link
Member

@Adastros I think that's a fair change to request :)

Copy link
Member

@arpitapandya arpitapandya left a comment

Choose a reason for hiding this comment

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

  • Hey @wendybarrios could you remove space from line no. 11.
  • Also in your comment on the top of the Pull request, I would suggest keeping it short. for e.g. 'removed type field and update content filed' in bullet points. Instead of copying the checkboxes from the original issue.
  • You can look at other PR's for the format

Copy link
Member

@arpitapandya arpitapandya left a comment

Choose a reason for hiding this comment

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

Thank you for making the changes. It's correct. Good job! @wendybarrios

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

4 participants