search

patternfly / patternfly-org

Home of patternfly.org
https://www.patternfly.org
MIT License
69 stars 136 forks source link

Bug - Google Analytics Warning #3882

Open jessiehuff opened 7 months ago

jessiehuff commented 7 months ago

Google Analytics is giving a warning right now that we should look into to make sure that it's still working:

Image 2-6-24 at 2 29 PM

edonehoo commented 5 months ago

@jessiehuff Looked into this - we need to update the Google Analytics property id across the site to point to the "new" GA4 property. Right now, we're still pointing to the universal analytics property, which Google deprecated a while back.

Tried to play around with local overrides to test this swap and confirm it works, but our GA code setup is a little complex, so I had no luck. I imagine simply swapping ID numbers across our codebase would do the trick, but I also wonder if we should consider removing the manual/onsite GA implementation and replacing it with Google Tag Manager. Not sure if our manual setup is intentional or necessary for any sort of security reasons, but switching to GTM would probably give us more freedom to manage any other website tracking we want to do going forward. Food for thought & happy to brainstorm more with others!