Skip to content
This repository has been archived by the owner. It is now read-only.

Enable sendBeacon for all admin analytics events #92

Merged
merged 1 commit into from Oct 30, 2015
Merged

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Oct 30, 2015

In some situations we will be tracking events that are triggered just before the page unloads, in these situations we will need sendBeacon.

Enable this for all events so the implementor doesn’t need to consider the nature of the event they’re creating. There’s no known downside to this approach.

https://developers.google.com/analytics/devguides/collection/analyticsjs/field-reference#transport

Needed as part of:
https://trello.com/c/k5F2wDcQ/151-add-google-analytics-tracking-to-2sv-user-journeys-pt-1-medium

In some situations we will be tracking events that are triggered just
before the page unloads, in these situations we will need sendBeacon.

Enable this for all events so the implementor doesn’t need to consider
the nature of the event they’re creating. There’s no known downside to
this approach.
@boffbowsh
Copy link
Contributor

@boffbowsh boffbowsh commented Oct 30, 2015

🚨

boffbowsh added a commit that referenced this issue Oct 30, 2015
Enable sendBeacon for all admin analytics events
@boffbowsh boffbowsh merged commit 0411736 into master Oct 30, 2015
1 check passed
@boffbowsh boffbowsh deleted the send-beacon branch Oct 30, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants