Skip to content

moved SDG folder and refactored SDG paths in multiple files#4274

Merged
t-will-gillis merged 4 commits intohackforla:gh-pagesfrom
arcan9:modify-sdg-and-associated-refactoring-4183
Apr 3, 2023
Merged

moved SDG folder and refactored SDG paths in multiple files#4274
t-will-gillis merged 4 commits intohackforla:gh-pagesfrom
arcan9:modify-sdg-and-associated-refactoring-4183

Conversation

@arcan9
Copy link
Member

@arcan9 arcan9 commented Mar 24, 2023

Fixes #4183

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

  • sdg-elements directory is renamed to sdg and moved up one level.
    • Old path: assets/images/about/sdg-elements
    • New path: assets/images/sdg
  • Due to the change, various files had to be refactored to point to the new path (listed in the issue).
  • Reason: to accurately reflect that we use these SDG images on multiples pages of the HfLA website

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

Visuals before changes are applied

Small card images in the "sustainable" sections in mulitple project pages:

hfla-4183-current

Visuals after changes are applied

After the change, the images are still rendered correctly:

hfla-4183-localhost

hfla-4183-localhost-2

@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 arcan9-modify-sdg-and-associated-refactoring-4183 gh-pages
git pull https://github.com/arcan9/website.git modify-sdg-and-associated-refactoring-4183

@github-actions github-actions bot added role: front end Tasks for front end developers Complexity: Medium P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) P-Feature: About Us https://www.hackforla.org/about/ time sensitive Needs to be worked on by a particular timeframe Feature: Refactor JS / Liquid Page is working fine - JS / Liquid needs changes to become consistent with other pages P-Feature: Citizen Engagement https://www.hackforla.org/citizen-engagement size: 1pt Can be done in 4-6 hours p-feature: SDGs SDGs wherever they appear on the site (they all use a color library, set of icons, etc.) labels Mar 24, 2023
@nnr-nnr nnr-nnr self-requested a review March 26, 2023 01:29
@nnr-nnr
Copy link
Member

nnr-nnr commented Mar 26, 2023

Review ETA: EOD Mon. 3/27/23
Availability: 12-4 PM 3/26, 3-8 PM 3/27–328/

@richardmundyiii
Copy link
Member

Availability: 3/26/23 12pm - 3pm
ETA: EOD 3/26/23

nnr-nnr
nnr-nnr previously approved these changes Mar 27, 2023
Copy link
Member

@nnr-nnr nnr-nnr left a comment

Choose a reason for hiding this comment

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

Wonderful work, @arcan9 ! You've cleanly moved and renamed all the SDG image assets, and you've updated all mentions of these files in the repo to reflect these changes. This PR passes the ABCs 👍.

@arcan9
Copy link
Member Author

arcan9 commented Mar 29, 2023

Availability: 3/26/23 12pm - 3pm ETA: EOD 3/26/23

Hi @richardmundyiii are you still available to review this?

Copy link
Member

@blulady blulady left a comment

Choose a reason for hiding this comment

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

@arcan9 All the changes you have done look great!!! However when I pull your branch into my local and search for "assets/images/about/sdg-elements", I find 18 mentions of it in 7 files. I see it in citzen-engagement.html, about\index.html, 311-data.html, access-the-data.html, engage.html, lucky-parking.html and open-community-survey.html. Please update the branch with changes to assets/images/sdg-elements from assets/images/about/sdg-elements.

@nnr-nnr
Copy link
Member

nnr-nnr commented Mar 31, 2023

Hi @blulady, just curious how you found the mentions of assets/images/about/sdg-elements? In VS Code, I use the Cmd + Shift + F (on mac) sequence to search for "assets/images/about/sdg-elements", but this doesn't turn up any mentions. I'm also not seeing the mentions of this file path in the files you listed, though I do see the updated file path that Issue #4183 says to use (ie assets/images/sdg). I think I might be missing something here... Thank you!

@arcan9
Copy link
Member Author

arcan9 commented Mar 31, 2023

@blulady: Like @nnr-nnr, I can't seem to find any mentions of assets/images/about/sdg-elements in the files you mentioned. And interestingly, I couldn't find those html files. Could you clarify please? Thank you.

@arcan9
Copy link
Member Author

arcan9 commented Apr 1, 2023

@t-will-gillis @bzzz-coding @roslynwythe Would love feedback on this!

t-will-gillis
t-will-gillis previously approved these changes Apr 3, 2023
Copy link
Member

@t-will-gillis t-will-gillis left a comment

Choose a reason for hiding this comment

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

Hey @arcan9 - Great job! Your branches are correct, as is your linked issue, and your description is super with the clips. I checked the changes on Docker and did not see any unexpected changes. Also did a final check and it appears that all assets/images/about/sdg-elements paths are changed.

Thanks for your efforts and great job again

@blulady
Copy link
Member

blulady commented Apr 3, 2023

Hey @arcan9 - Great job! Your branches are correct, as is your linked issue, and your description is super with the clips. I checked the changes on Docker and did not see any unexpected changes. Also did a final check and it appears that all assets/images/about/sdg-elements paths are changed.

Thanks for your efforts and great job again

Hey @t-will-gillis feel free to merge this pr.

@blulady blulady removed their request for review April 3, 2023 04:26
@t-will-gillis t-will-gillis dismissed blulady’s stale review April 3, 2023 04:29

Please see comments above

@t-will-gillis t-will-gillis self-requested a review April 3, 2023 04:29
Copy link
Member

@t-will-gillis t-will-gillis left a comment

Choose a reason for hiding this comment

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

Merge conflicts have been resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Complexity: Medium Feature: Refactor JS / Liquid Page is working fine - JS / Liquid needs changes to become consistent with other pages P-Feature: About Us https://www.hackforla.org/about/ P-Feature: Citizen Engagement https://www.hackforla.org/citizen-engagement P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) p-feature: SDGs SDGs wherever they appear on the site (they all use a color library, set of icons, etc.) role: front end Tasks for front end developers size: 1pt Can be done in 4-6 hours time sensitive Needs to be worked on by a particular timeframe

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move and rename sdg-elements directory and refactor associated code

5 participants