-
Notifications
You must be signed in to change notification settings - Fork 95
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add user email for Advanced Matching Facebook retargeting #9
Comments
Could you please describe the issue in more detail? What did you do exactly, what additional code has been added and where? |
I just realized what you would like to achieve: adding email address of site users is against the terms of service of Google Tag Manager: https://www.google.com/analytics/tag-manager/use-policy/ It does not mean that technically it is not possible but once someone reports this to Google or Google finds this out while some sort of automated pro-active checking process, your GTM container might be shut down. |
OK, I digg into more details at Google and found out that passing an email address into the dataLayer might be OK unless you do not send this email address to any Google service. |
This has been added to the source. |
Thank you! On Sat, Nov 5, 2016, 2:32 AM Thomas Geiger notifications@github.com wrote:
|
User email doesnt print, I tried to add it but its not working
The text was updated successfully, but these errors were encountered: