Skip to content
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

Vidazoo Bid Adapter: Cookie sync improvements #8834

Merged
merged 24 commits into from
Aug 16, 2022

Conversation

saar120
Copy link
Contributor

@saar120 saar120 commented Aug 15, 2022

Type of change

  • Features
  • Refactoring (no functional changes, no api changes)

Description of change

  • Changed userSync url.
  • Passing cid's to cookie sync.

@ChrisHuie
Copy link
Collaborator

@saar120 looks like you have a failing unit test. FYI

@saar120
Copy link
Contributor Author

saar120 commented Aug 16, 2022

@saar120 looks like you have a failing unit test. FYI

On it - thanks.

@ChrisHuie ChrisHuie self-assigned this Aug 16, 2022
@ChrisHuie ChrisHuie merged commit 91ce750 into prebid:master Aug 16, 2022
JacobKlein26 pushed a commit to nextmillenniummedia/Prebid.js that referenced this pull request Feb 9, 2023
* feat(module): multi size request

* fix getUserSyncs
added tests

* update(module): package-lock.json from master

* feat(module): VidazooBidAdapter - send top query params to server

* feat(module): changed userSync url and passes cids from responses to usersync.

* feat(module): fixed failing test.

Co-authored-by: Udi Talias <[email protected]>
Co-authored-by: roman <[email protected]>
jorgeluisrocha pushed a commit to jwplayer/Prebid.js that referenced this pull request May 23, 2023
* feat(module): multi size request

* fix getUserSyncs
added tests

* update(module): package-lock.json from master

* feat(module): VidazooBidAdapter - send top query params to server

* feat(module): changed userSync url and passes cids from responses to usersync.

* feat(module): fixed failing test.

Co-authored-by: Udi Talias <[email protected]>
Co-authored-by: roman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants