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

Remove fa-solid-900.woff reference from Content Build and Next Build #20219

Open
1 task
Tracked by #20102
timcosgrove opened this issue Jan 7, 2025 · 0 comments
Open
1 task
Tracked by #20102
Assignees
Labels
CMS Team CMS Product team that manages both editor exp and devops

Comments

@timcosgrove
Copy link
Contributor

timcosgrove commented Jan 7, 2025

User Story or Problem Statement

We do not want to reference the fa-solid-900.woff on Content Build and Next Build pages, as the font file is no longer being generated.

Description or Additional Context

Font Awesome was retired, but the reference to the font file was never removed from VA.gov. This is causing metrics problems for some folks.

This was being reported in Siteimprove which is used by VHA and VAMC editors. It's in the <head> of the site so that means it's being found and reported on for every page of each VAMC, which ends up being a lot of errors.

Screenshot of where it is in the code
Image

Screenshot of the Milwaukee VAMC Siteimprove Broken links report
You can see here that Siteimprove is reporting 160 broken links (when in reality it's just 1 link but happens to be on every page)
Image

Because of how Siteimprove is set up, I can't get a report of how many issues total without manually going into each VAMC and then adding up the total reported broken links. But I have gone into a few and reported the total number below so we can review these after the next SI report runs and confirm that these numbers have gone down to 0:

  • Milwaukee: 160
  • Eastern Kansas: 49
  • Alaska: 45
  • Boston: 139

Steps for Implementation

Remove the reference to the file. There is no active harm in this, as the file itself no longer exists.

Acceptance Criteria

  • fa-solid-900.woff is removed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CMS Team CMS Product team that manages both editor exp and devops
Projects
None yet
Development

No branches or pull requests

2 participants