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 amp-analytics vendor config for INFOnline #2020
Conversation
e1640f7
to
a208943
Compare
'pageview': { | ||
'on': 'visible', | ||
'request': 'pageview', | ||
'iframePing': true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change this to
'iframe' + /* TEMPORARY EXCEPTION */ + 'Ping'
to pass presubmit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
a208943
to
42ff54d
Compare
PTAL |
'vars': { | ||
"sv": "ke", | ||
"ap": "1", | ||
"url": "https://3p.ampproject.net/custom/amp-analytics-infonline.html" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please move this into the request.
LGTM with one comment |
'pageview': { | ||
'on': 'visible', | ||
'request': 'pageview', | ||
'iframe' + /* TEMPORARY EXCEPTION */ + 'Ping' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Delete this line and add this to the end of the file:
ANALYTICS_CONFIG['infonline']['triggers']['pageview'][ 'iframe' + /* TEMPORARY EXCEPTION */ + 'Ping'] = true;
42ff54d
to
06cb3b5
Compare
}; | ||
ANALYTICS_CONFIG['infonline']['triggers']['pageview'][ 'iframe' + /* TEMPORARY EXCEPTION */ + 'Ping'] = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will need a line break for lint.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lint is happy now
cea4e30
to
6abb6fe
Compare
Added canonicalHost and canonicalPath to request. PTAL. |
LGTM |
Add amp-analytics vendor config for INFOnline
No description provided.