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

AdView: support multi imps request, related to prebid-server adapter PR #3355 #5126

Merged
merged 8 commits into from
Mar 8, 2024

Conversation

AdviewOpen
Copy link
Contributor

@AdviewOpen AdviewOpen commented Feb 2, 2024

🏷 Type of documentation

  • new bid adapter
  • update bid adapter
  • new feature
  • text edit only (wording, typos)
  • bugfix (code examples)
  • new examples

📋 Checklist

Copy link

netlify bot commented Feb 2, 2024

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 685657a
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/65eab14ca442a7000884232c
😎 Deploy Preview https://deploy-preview-5126--prebid-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

dev-docs/bidders/adview.md Show resolved Hide resolved
@muuki88 muuki88 added the LGTM label Feb 3, 2024
@AdviewOpen
Copy link
Contributor Author

Add support for safe frames.

bidder_supports_deals: true
pbjs: false
pbs: true
pbs_app_supported: true
prebid_member: false
sidebarType: 1
Copy link
Contributor

Choose a reason for hiding this comment

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

can you add back sidebarType

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sidebarType add, many thanks.

@AdviewOpen
Copy link
Contributor Author

add sidebarType.

@@ -7,23 +7,24 @@ tcfeu_supported: true
gvl_id: 1022
usp_supported: true
coppa_supported: true
gpp_supported: true
Copy link
Contributor

Choose a reason for hiding this comment

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

gpp_supported is deprecated. You have to be more specific on what you actually support

Suggested change
gpp_supported: true
gpp_sids: // the sections you support. List of cfeu, tcfca, usnat, usstate_all, usp

see https://docs.prebid.org/dev-docs/bidder-adaptor.html

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok, changed gpp_support to gpp_sids, pls check it again.

@muuki88 muuki88 added needs work and removed LGTM labels Mar 7, 2024
@AdviewOpen AdviewOpen requested a review from muuki88 March 8, 2024 06:38
@muuki88 muuki88 added LGTM and removed needs work labels Mar 8, 2024
@muuki88 muuki88 merged commit 0ef2e00 into prebid:master Mar 8, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants