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

RESOLVED FIXED

Status

www.mozilla.org
Analytics
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: Luka Cempre (formerly owned by James Lorence), Assigned: Luka Cempre (formerly owned by James Lorence))

Tracking

Production

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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'.

Comment 2

a year ago
agibson - can you please evaluate this request (and provide LOE to do complete it)
Flags: needinfo?(agibson)
Created attachment 8768659 [details] [review]
GitHub pull request

This is already being worked on by James
Flags: needinfo?(agibson)
Assignee: nobody → james.lorence
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true

Comment 4

a year ago
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
Last Resolved: a year ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.