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

Edit content field and remove type field from security.yml credits file #3695

Merged

Conversation

Adastros
Copy link
Member

@Adastros Adastros commented Nov 5, 2022

Fixes #2858

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 (if any, please do not screen shot code changes)

Updating fields in credits file. No visual changes to the website.

@github-actions
Copy link

github-actions bot commented Nov 5, 2022

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 Adastros-update-fields-security-credits-2858 gh-pages
git pull https://github.com/Adastros/website.git update-fields-security-credits-2858

@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 5, 2022
@luiderek luiderek self-requested a review November 5, 2022 05:34
@luiderek
Copy link
Member

luiderek commented Nov 5, 2022

Availability: 10PM-12PM
ETA: 10min

@luiderek
Copy link
Member

luiderek commented Nov 5, 2022

Nice work @Adastros. It's the right changes to the right file, and your branch / commit naming conventions are clean.

Copy link
Member

@luiderek luiderek left a comment

Choose a reason for hiding this comment

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

Learning GitHub is definitely a process. See above.

@MattPereira MattPereira self-requested a review November 5, 2022 21:00
@MattPereira
Copy link
Contributor

MattPereira commented Nov 5, 2022

Review ETA: End of Day 11/6/22
Availability: 4 - 6 PM

@Suman2795 Suman2795 self-requested a review November 6, 2022 16:19
@Suman2795
Copy link
Member

Review ETA: 9 PM 6/11/2022
Availability : 6 /11/2022

Copy link
Member

@Suman2795 Suman2795 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 @Adastros ! Your changes look correct.

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.

Good work on this one @Adastros ! 👍

You linked the issue number properly and removed the redundant code from secuirty.yml. Keep up the good work!

@jyaymie jyaymie self-requested a review November 7, 2022 15:05
@jyaymie
Copy link
Member

jyaymie commented Nov 7, 2022

Review ETA: 11/7/22 7:30am
Availability: 11/7/22 Now - 9am

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.

You've linked the issue and implemented all its action items. I like to include a rationale for my changes in the body of the PR, but I see it's pretty clear in the issue itself. Great work!

@jdingeman jdingeman merged commit b9be42a into hackforla:gh-pages Nov 7, 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: 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 security.yml credits file
6 participants