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
mediaimpact: startRender API #5420
Conversation
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.
LGTM
For a record, this is a re-try of #5333
@adtechnology could you fix the linter error so I can merge for you? |
@adtechnology some more linter errors :-) FYI, you can run lint locally: |
F**K, it was linted already, but I pasted your line from #5333 and not linted afterwards... |
d14f73f
to
0837cdb
Compare
@adtechnology there are still type check errors. Could you fix? |
8826cbe
to
edcc347
Compare
/* eslint google-camelcase/google-camelcase: 0 */ | ||
global.sas_loadHandler = function(f) { | ||
if (f.hasAd) { | ||
f.crea || (f.crea1 = { |
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.
just want to make sure this is not a typo here. should it be f.crea
or f.crea1
?
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.
Yeah, right, THX again...
added loadHandler functionality to support startRender API changed lib file url changed prefetch and extended preconnect settings
edcc347
to
bdce2e6
Compare
added loadHandler functionality to support startRender API changed lib file url changed prefetch and extended preconnect settings
added loadHandler functionality to support startRender API changed lib file url changed prefetch and extended preconnect settings
added loadHandler functionality to support startRender API changed lib file url changed prefetch and extended preconnect settings
added loadHandler functionality to support startRender API
changed lib file url
changed prefetch and extended preconnect settings