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

Img Tag Refactor #4401 #4569

Merged
merged 1 commit into from
Apr 30, 2023
Merged

Conversation

graycodesnu
Copy link
Member

Fixes #4401

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

-Changed an img HTML tag ending with a slash (<img.../>) to an img tag without an ending slash (<img...>) so that the codebase is consistent with how we use img HTML tags.

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

No visual changes.

@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 graycodesnu-img-refactor-4401 gh-pages
git pull https://github.com/graycodesnu/hfla-website.git img-refactor-4401

@github-actions github-actions bot added good first issue Good for newcomers role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers P-Feature: Getting Started https://www.hackforla.org/getting-started Feature: Refactor HTML size: 0.25pt Can be done in 0.5 to 1.5 hours labels Apr 27, 2023
@agosmou agosmou self-requested a review April 27, 2023 08:55
Copy link
Member

@agosmou agosmou left a comment

Choose a reason for hiding this comment

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

LGTM!

@Thinking-Panda Thinking-Panda self-requested a review April 27, 2023 17:14
@Thinking-Panda
Copy link
Member

Availability: M-F 10 AM - 3PM
ETA: 4/27/23

Copy link
Member

@Thinking-Panda Thinking-Panda left a comment

Choose a reason for hiding this comment

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

Hi @graycodesnu, code changes look good and clean. But, while checking website on my local machine, the donate page is the older version and looks different from the published website. May be there is an issue with the branch you are working on.

@graycodesnu
Copy link
Member Author

Hi @Thinking-Panda, can you provide more detail in terms of the changes you see and what adjustments need to be made on my end?
I synced my forked repo + pulled before creating a new branch, and then pulled the upstream changes before submitting my pull request. I also double-checked for any changes via Docker on my end before pushing and creating this request. Additionally, I did not work on a donate page, as these changes were only made to the getting-started.html file.

@Thinking-Panda
Copy link
Member

Hi @Thinking-Panda, can you provide more detail in terms of the changes you see and what adjustments need to be made on my end? I synced my forked repo + pulled before creating a new branch, and then pulled the upstream changes before submitting my pull request. I also double-checked for any changes via Docker on my end before pushing and creating this request. Additionally, I did not work on a donate page, as these changes were only made to the getting-started.html file.

@graycodesnu , looks like you have followed the necessary steps before submitting a pull request. I also understand that you did not work on the donate page. I still see the older version of the donate page on my local machine. Did you try pulling from upstream again?

@graycodesnu
Copy link
Member Author

@agosmou, you've already approved this request, but are you seeing the same issues @Thinking-Panda is? I understand version control differently and don't think this will be an issue, but if I need to take time to brush up, I'd rather have a second set of eyes.

@jdingeman
Copy link
Member

@Thinking-Panda - you may be seeing the page before the cache was emptied and reloaded/the page was rebuilt in Docker. The file he changed doesn't affect the Donate page.

@jdingeman jdingeman dismissed Thinking-Panda’s stale review April 30, 2023 17:24

Changes not needed

@jdingeman jdingeman merged commit f15cfaa into hackforla:gh-pages Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature: Refactor HTML good first issue Good for newcomers P-Feature: Getting Started https://www.hackforla.org/getting-started role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours
Projects
None yet
Development

Successfully merging this pull request may close these issues.

img Tag Refactor: Getting Started page - step-img-icon ada.png
4 participants