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

Update documentation #5461

Merged
merged 3 commits into from
Jul 2, 2024
Merged

Conversation

DimaIntentIQ
Copy link
Contributor

🏷 Type of documentation

  • update bid adapter
  • new examples

📋 Checklist

Copy link

netlify bot commented Jun 27, 2024

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 2609097
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/667d190bfe9b100008227d41
😎 Deploy Preview https://deploy-preview-5461--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.

Copy link

netlify bot commented Jun 27, 2024

Deploy Preview for prebid-docs-preview ready!

Name Link
🔨 Latest commit 0aed216
🔍 Latest deploy log https://app.netlify.com/sites/prebid-docs-preview/deploys/667d7565c2867a000827530f
😎 Deploy Preview https://deploy-preview-5461--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.

| name | Required | String | The name of this module: "intentIqId" | `"intentIqId"` |
| params | Required | Object | Details for IntentIqId initialization. | |
| params.partner | Required | Number | This is the partner ID value obtained from registering with IntentIQ. | `1177538` |
| params.percentage | Required | Number | This a percentage value for our A/B testing group distribution. The values supposed to be in range of 0 to 100. We suggest to set it to 95 percent for optimal balance ofbetween prefromance and preceision. | `95` |
Copy link
Contributor

Choose a reason for hiding this comment

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

doesn't removing this and adding callback break implementations?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No, it doesn't break the implementation. We now receive the percentage and A/B testing group from our server. Additionally, a callback is added to synchronize partners with us, as they typically have their own logic implemented on the page based on the callback.

@ChrisHuie ChrisHuie self-assigned this Jul 1, 2024
@ChrisHuie ChrisHuie added the LGTM label Jul 2, 2024
@ChrisHuie ChrisHuie merged commit 0e41c85 into prebid:master Jul 2, 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.

2 participants