Skip to content

Instantly share code, notes, and snippets.

@eduwass

eduwass/readme.md

Created December 7, 2021 16:36
Show Gist options
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
  • Save eduwass/7ddcd459e3b7901cc1a559a938a769f3 to your computer and use it in GitHub Desktop.
Save eduwass/7ddcd459e3b7901cc1a559a938a769f3 to your computer and use it in GitHub Desktop.
Download ZIP
BUG <Medium>: React capturing traffic on separate GA Instance
Raw
readme.md

Detected on

PRODUCTION

Description

Originally, we had separate GA Instances for Blog and WP:

CleanShot 2021-12-07 at 16.30.01.png

Looks like a request that was introduced in https://app.shortcut.com/amerlife/story/24457/setting-up-a-gtm-ga-setup-that-works-with-blog was asking to merge tracking into just one instance.

Initial change ( done by César ) introduced a change to load the Production Pagely WordPress in React Blog:

CleanShot 2021-12-07 at 16.25.34.png

I think probably due to confusion, a Reversion of this change was made ( Tianqi ):

CleanShot 2021-12-07 at 16.20.02.png

Additionally, to prevent future confusion, I think we should:

  • Remove Blog instances if they're not going to be used
  • Rename Pagely WordPress to something like: "Production: Main Site + Blog"

Steps to reproduce

  1. Open http://blog.seniorhealthcaredirect.com
  2. On another window open GA, on instance: Production Pagely WordPress

Actual Result

  1. If you go to "Realtime > Overview" you can't see active users from the Blog (because it's loading the other GA Instance: React Blog Production)

Expected Result

  1. If you go to "Realtime > Overview" you should see both Blog and WP active users.

Additionally, to prevent future confusion, I think we should:

  • Remove Blog instances if they're not going to be used
  • Rename Pagely WordPress to something like: "Production: Main Site + Blog"

Attachments

Attach anything that can help better understand what happened, and also help the developer fix it and the QA team validate it.

Notes

Add any notes if you consider necessary.

ROOT CAUSE (eng) ⚠️ ‼️

Explained on Description

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