Onevideo supporting both CCPA and GDPR present case #4571
Conversation
|
This pull request introduces 2 alerts when merging 4a744c0 into 8c4039b - view on LGTM.com new alerts:
|
…hiNeeladri/Prebid.js into onevideo_ccpa_gdpr_support
|
please add/modify unit test(s) |
|
@bretg added test cases, all are passing in my local but ci/circleci is failing with error "Build was canceled" |
|
@DeepthiNeeladri We're having some issues (for some reason) with browserstack keeping the browser sessions connected. I tried to cancel the build since it was just restarting the sessions in a loop. Trying to run it again. |
|
@bretg @jsnellbaker review pls |
| usp: { | ||
| cmpApi: 'iab', | ||
| timeout: 1000, | ||
| allowAuctionWithoutConsent: 'cancel' |
There was a problem hiding this comment.
for the record, usp doesn't support the allowAuctionWithoutConsent parameter
|
Will go ahead and merge this, but note that you'll probably want to add a docs PR so you show as one of the adapters supporting CCPA. Set the |
|
Also note -- so far it's only merged into 3.1. If you want CCPA supported in the 2.x branch please open another PR against the 2.44.x-legacy branch |
* outstream changes * removing global filtet * reverting page * message * adapter change * remove space * testcases * testpage * spaces for test page * renderer exist case * reverting package-lock.json * adding schain object * adding tagid * syntaxx error fix * video.html * space trailing * space * tagid * inventoryId and placement * rewarded video * added unit test case * ccpa cahnges * ccpa change * test page * test page change * test page change 2 * change the variable * handling the case if both GDPR and CCPA case * handiling both cases * test cases
Type of change
Description of change
For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:
Other information