Closed Bug 1279606 Opened 9 years ago Closed 9 years ago

Page ID Is pushed to late to dataLayer for scrolling tracking to work properly

Categories

(www.mozilla.org :: Analytics, defect)

Production
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: luka.cempre, Assigned: luka.cempre)

Details

Attachments

(1 file)

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.84 Safari/537.36
For situations where a user lands on a page part way down and there is scroll tracking, the scroll tracking events are happening before the Core DataLayer Object is pushed to the dataLayer, which when we make GTM CSP compliant and reply on the 'pageId' property being available in the dataLayer this will break the event data model. I'd like to propose to move the 'pageId' property out of the Core DataLayer Object into a new JS file that will get bundled under 'site-bundle.js'.
agibson - can you please evaluate this request (and provide LOE to do complete it)
Flags: needinfo?(agibson)
Attached file GitHub pull request
This is already being worked on by James
Flags: needinfo?(agibson)
Assignee: nobody → james.lorence
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Commits pushed to master at https://github.com/mozilla/bedrock https://github.com/mozilla/bedrock/commit/fe5a26f87c6d3234e647711fbe42b7fd31a0238c [bug 1279606] Move Page ID dataLayer push before GTM loads https://github.com/mozilla/bedrock/commit/4efbbee4ea4530f7a6fed518ee380de8a563c938 Merge pull request #4189 from analytics-pros/bug-1279606-move-page-id [bug 1279606] Move Page ID dataLayer push before GTM loads
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: