Skip to content
This repository has been archived by the owner on Jul 25, 2018. It is now read-only.

Moves anonymization flag before pageview #506

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

konklone
Copy link
Contributor

@konklone konklone commented Oct 1, 2014

(My apologies for 9e99e1a, which I reverted in 59da8f3 - I was editing in-browser and forgot I still had write permissions, I was thinking it would auto-fork.)

This moves the Google Analytics anonymize IP request up before the pageview is sent. The Google Analytics IP anonymization docs say:

If the anonymization function has been called prior to the page tracking function, an additional parameter is added to the pixel request. The IP anonymization parameter looks like this: &aip=1

So that implies that this is the necessary order to me.

@konklone
Copy link
Contributor Author

Can a fella get a merge?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant