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 GA script and stale revive.bocoup.com script; self-host Lato #225

Closed
wants to merge 2 commits into from

Conversation

timmywil
Copy link
Contributor

@timmywil timmywil commented Sep 6, 2024

Remove third party dependencies from gruntjs.com, including:

  • Google Analytics. We're not sure who has access to that account, but basic analytics can be found in the cloudflare dashboard.
  • Ads from revive.bocoup.com. These no longer work.
  • Lato from google fonts. This is is now self-hosted.

Fixes #224

@Krinkle
Copy link
Contributor

Krinkle commented Sep 8, 2024

@timmywil When I try this locally, the font is never loaded. It always uses the Arial/sans-serif fallback (tried in Firefox, Safari, and Chrome). Looks like something in the @font-face syntax isn't quite right?

@timmywil
Copy link
Contributor Author

timmywil commented Sep 9, 2024

Turns out I hadn't tried it in production mode. [email protected] was removing the commas from the Lato font-face declarations, making them invalid. I've updated the plugin to 1.6.0, which seems to fix it.

@Krinkle Krinkle closed this in 64f9d43 Sep 10, 2024
@timmywil timmywil deleted the csp branch September 10, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reduce gruntjs.com third-party dependencies
2 participants