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

Adomik Analytics Adapter: change bid.placementCode to bid.adUnitCode #7610

Merged
merged 2 commits into from
Oct 28, 2021

Conversation

liabas-b
Copy link
Contributor

@liabas-b liabas-b commented Oct 21, 2021

Type of change

  • Feature

Description of change

  • change the placementCode we save on bidRequested events to from bid.placementCode to bid.adUnitCode
  • do not split sent events in GET requests unless browser is IE and version < 11

@liabas-b liabas-b force-pushed the adomik-analytics-adapter-update branch from cd835fb to fc48dbd Compare October 21, 2021 11:35
@ChrisHuie ChrisHuie changed the title Adomik analytics adapter update Adomik Analytics Adapter: change bid.placementCode to bid.adUnitCode Oct 25, 2021
@ChrisHuie ChrisHuie requested a review from gwhigs October 25, 2021 14:12
@gwhigs gwhigs merged commit 199eb7d into prebid:master Oct 28, 2021
cpabst pushed a commit to sovrn/Prebid.js that referenced this pull request Jan 10, 2022
…rebid#7610)

* AdomikAnalyticsAdapter: change placementCode to adUnitCode

* AdomikAnalyticsAdapter: split GET URLs only for MSIE browser
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.

3 participants