-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
ZetaGlobalSsp Bid Adapter : support topics module #10803
Merged
patmmccann
merged 51 commits into
prebid:master
from
asurovenko-zeta:zeta_global_ssp_adapter_topics
Dec 27, 2023
Merged
Changes from 47 commits
Commits
Show all changes
51 commits
Select commit
Hold shift + click to select a range
da0b4a5
Merge pull request #1 from prebid/master
asurovenko-zeta f4ec68e
Merge branch 'master' of git://github.com/prebid/Prebid.js
surovenko 01990a9
Merge remote-tracking branch 'upstream/master'
surovenko 62f969f
Merge remote-tracking branch 'upstream/master'
surovenko ec7c8f0
Merge remote-tracking branch 'upstream/master'
surovenko 89fe5b1
Merge remote-tracking branch 'upstream/master'
surovenko 6dc0255
Merge remote-tracking branch 'upstream/master'
surovenko 8dc9ac2
Merge remote-tracking branch 'upstream/master'
surovenko 35353f0
Merge remote-tracking branch 'upstream/master'
surovenko e6f982a
Merge remote-tracking branch 'upstream/master'
surovenko 70e546c
Merge remote-tracking branch 'upstream/master'
surovenko cc56875
Merge remote-tracking branch 'upstream/master'
surovenko f4f2a1e
Merge remote-tracking branch 'upstream/master'
surovenko 3f96c6a
Merge remote-tracking branch 'upstream/master'
surovenko 8e503cd
Merge remote-tracking branch 'upstream/master'
surovenko 1781bfd
Merge remote-tracking branch 'upstream/master'
surovenko 459917b
Merge remote-tracking branch 'upstream/master'
surovenko 9ce50bd
Merge remote-tracking branch 'upstream/master'
surovenko 8f7f147
Merge remote-tracking branch 'upstream/master'
surovenko a280757
Merge remote-tracking branch 'upstream/master'
surovenko 036d5f1
Merge remote-tracking branch 'upstream/master'
surovenko 33e883f
Merge remote-tracking branch 'upstream/master'
surovenko e729eb9
Merge remote-tracking branch 'upstream/master'
surovenko c202e93
Merge remote-tracking branch 'origin/master'
surovenko 95877a7
Merge remote-tracking branch 'upstream/master'
surovenko bf8f638
Merge remote-tracking branch 'upstream/master'
surovenko 461374f
Merge remote-tracking branch 'upstream/master'
surovenko b6a2fdc
Merge remote-tracking branch 'upstream/master'
surovenko 92b44e3
Merge remote-tracking branch 'upstream/master'
surovenko 42381eb
Merge remote-tracking branch 'upstream/master'
surovenko f629f34
Merge remote-tracking branch 'upstream/master'
surovenko 9627d5d
Merge remote-tracking branch 'upstream/master'
surovenko 3d0c2b2
Merge remote-tracking branch 'upstream/master'
surovenko 0dfdb1c
Merge remote-tracking branch 'upstream/master'
surovenko 9438a2e
Merge remote-tracking branch 'upstream/master'
4198c51
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta 4064f1f
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta 6b380a1
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta 6cd4e3f
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta 1794ae1
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta 42d2317
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta 691c896
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta e3a97f6
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta 3669809
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta 312817c
-
asurovenko-zeta 050688f
Merge remote-tracking branch 'upstream/master'
asurovenko-zeta dd3ae50
add getTopics()
asurovenko-zeta cc89a68
provide get segments from ortb2
asurovenko-zeta 76ff546
rename const
asurovenko-zeta d462410
additional check
asurovenko-zeta f0b3ceb
fix test
asurovenko-zeta File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
document.browsingTopics() return the promise.
So it will not be working right now.
I didn't find any example with async context in adapters. I don't know is that ok to add async context to buildRequests function
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.
@ChrisHuie may I ask you to help me with understanding this
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.
you're not allowed to call this; the topics module exists and does it; you can read its results in ortb2.user.data