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

[BLOCKED] Remove unused assets in content-build #18269

Open
1 task done
randimays opened this issue Jun 4, 2024 · 3 comments
Open
1 task done

[BLOCKED] Remove unused assets in content-build #18269

randimays opened this issue Jun 4, 2024 · 3 comments
Assignees
Labels
Blocked Issues that are blocked on factors other than blocking issues. ghp-backlog points-2 Public Websites Scrum team in the Sitewide crew sitewide Technical debt VA.gov frontend CMS team practice area

Comments

@randimays
Copy link
Contributor

randimays commented Jun 4, 2024

Status

[2024-09-10] [Fran] Ticket blocking this one is In Progress. Will monitor and move to Ready when unblocked.
[2024-08-19] [Fran] Per Randi, this ticket is blocked; moved back to the backlog.
[2024-08-05] [Fran] In Ready column because every little bit of help Content build can get to optimize build time should be addressed, if possible.

Description

As tech debt cleanup, we should remove any media assets from content-build that are no longer used. This helps (marginally) with build time and discourages engineers from using these icons, and instead pushes them to use va-icon where possible for consistency.

Engineering notes / background

Folder with assets: https://github.com/department-of-veterans-affairs/content-build/tree/main/src/site/assets/img

It is important to do a Github-wide search in the organization such as this example. Previously, when removing assets from vets-website, we ran into an issue where a team was pointing to the s3 bucket to retrieve assets (which were put there by VW). Slack context here: https://dsva.slack.com/archives/CBU0KDSB1/p1714066142850419.

You can check things at the folder level when doing the Github-wide search (i.e. img/design vs img/design/background/thread.png) to get through the searching more efficiently, and only drill down when necessary.

Acceptance criteria

  • Github-wide search in the content-build repo for every asset removed (and confirm it's not being used)
@randimays
Copy link
Contributor Author

This was pulled in as stretch for Sprint 10.

@randimays
Copy link
Contributor Author

randimays commented Aug 19, 2024

Jami from the design system reviewed the PR for this and I realized that the cross-repo style issue we have (with CB leaning on VW styles, #18765) should be fixed before we can confidently remove these assets. I did find at least one place where this would be a problem.

The PR is a WIP and has a do not merge label on it and is now dependent on #18765 in order to be merged/complete. I added that ticket as a blocker on this one, and marked this ticket as blocked.

@FranECross I think maybe this ticket should go back to the Backlog for now?

@randimays randimays added the Blocked Issues that are blocked on factors other than blocking issues. label Aug 19, 2024
@FranECross
Copy link

@randimays Thanks so much for this info. Totally agree, and will scoot it back to the backlog.

@FranECross FranECross changed the title Remove unused assets in content-build [BLOCKED] Remove unused assets in content-build Nov 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked Issues that are blocked on factors other than blocking issues. ghp-backlog points-2 Public Websites Scrum team in the Sitewide crew sitewide Technical debt VA.gov frontend CMS team practice area
Projects
None yet
Development

No branches or pull requests

3 participants