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

Bid Viewability module documentation #2659

Merged
merged 7 commits into from
Feb 11, 2021
Merged

Conversation

pm-harshad-mane
Copy link
Contributor

@pm-harshad-mane pm-harshad-mane commented Jan 26, 2021

Refer PrebidJS PR: prebid/Prebid.js#6206
Also added an API fix in documentation of Instream tracking module

Copy link
Contributor

@bretg bretg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As long as the "bidder only gets their own stuff" rule is confirmed, this looks good. Thanks.

@pm-harshad-mane
Copy link
Contributor Author

Thank you @bretg !
Waiting for code PR to merge.

@pm-harshad-mane
Copy link
Contributor Author

Hello @bretg ,
The code PR is merged , can you please merge this PR too?

@pm-harshad-mane
Copy link
Contributor Author

@bretg, gentle reminder to merge the PR

@pm-harshad-mane
Copy link
Contributor Author

Hello @bretg, gentle reminder to merge the PR

@bretg
Copy link
Contributor

bretg commented Feb 11, 2021

@pm-harshad-mane - we don't merge docs PRs until after the code is released -- not after it's merged. Because it's live on the site immediately once merged.

For the record, there's a website rotation and the person on-duty should be merging PRs that are ready. It's not my week, but I'll merge this for you.

@bretg bretg merged commit 310463a into prebid:master Feb 11, 2021
osazos pushed a commit to onfocusio/prebid.github.io that referenced this pull request Feb 23, 2021
* API fix

* bidViewability module documentation

* wordsmithing

* adding link

* update for onBidViewable

* now we add ? in URL if not present

* change in params, added enabled param

Co-authored-by: bretg <bgorsline@gmail.com>
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.

2 participants