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

No Bid Adapter: added bidResponse.meta and advertiserDomains #6862

Merged
merged 15 commits into from
May 28, 2021

Conversation

redaguermas
Copy link
Contributor

Type of change

  • Feature

Description of change

Added bidResponse.meta attribute and specifically "advertiserDomains" which is planned to be required in Prebid.js version 5.

redaguermas and others added 14 commits March 4, 2020 15:39
* 'master' of https://github.com/prebid/Prebid.js: (49 commits)
  Submitting Clicktripz bid adapter (prebid#4929)
  add UNICORN bid adapter (prebid#4917)
  3.12.0-pre
  3.11.0 release
  Eids liveintent ext fix (prebid#4944)
  add mediaforce bid adapter (prebid#4933)
  update logic in adpod module for playersize (prebid#4953)
  Module - Size Mapping V2 (prebid#4690)
  Lifestreet adapter 3.0 (prebid#4927)
  IX Adapter - Increase banner TTL to 300s (prebid#4957)
  assert string returned not that we break things (prebid#4962)
  added option to url parser to ignore decoding entire url (prebid#4938)
  adding user-id support in medianet adapter (prebid#4925)
  removing the log (prebid#4960)
  add import extensions (prebid#4959)
  Add 7xbid adapter to compatible with prebid 3.0 (prebid#4908)
  Fix failing code-coverage command (prebid#4892)
  enable no-console eslint rule for project (prebid#4802)
  update audigent tests to fix larger test suite issue (prebid#4952)
  use bidId or bidIds in the payload (prebid#4903)
  ...
* 'master' of https://github.com/prebid/Prebid.js: (102 commits)
  Marsmedia - Add vastXml and fix id response (prebid#5067)
  PubMatic adapter to support image sync (prebid#5104)
  minor consentManagement fix (prebid#5050)
  fix circle ci failing tests (prebid#5113)
  Add Relaido Adapter (prebid#5101)
  Add new bid adapter for ConnectAd (prebid#4806)
  change payload (prebid#5105)
  Utils updates (prebid#5092)
  Read OpenRTB app objects if set in config + bug fix for when ad units are reloaded (prebid#5086)
  Criteo : added first party data mapping to bidder request (prebid#4954)
  updateAdGenerationManual (prebid#5032)
  New bid adapter: Wipes (prebid#5051)
  Prebid manager analytics utm tags (prebid#4998)
  CRITEO RTUS Integration with Yieldmo Prebid (prebid#5075)
  isSafariBrowser update  (prebid#5077)
  Support min &max duration for onevideo (prebid#5079)
  increment pre version
  Prebid 3.15.0 release
  prebid#5011 Fix to set Secure attribute on cookie when SameSite=none (prebid#5064)
  Prebid adapter for windtalker (prebid#5040)
  ...
# Conflicts:
#	test/spec/modules/nobidBidAdapter_spec.js
@smenzer
Copy link
Collaborator

smenzer commented May 28, 2021

can you please get rid of all the extra files in this PR? it should only contain your specific code

@ChrisHuie ChrisHuie changed the title No bid version 1.3.0 No Bid Adapter: added bidResponse.meta and advertiserDomains May 28, 2021
@ChrisHuie
Copy link
Collaborator

Yeah looks like you added your ide settings @redaguermas

@redaguermas
Copy link
Contributor Author

Hello,
This error does not seem to be coming from my code. Do you know what we need to do to pass it?

Thanks

@ChrisHuie
Copy link
Collaborator

ChrisHuie commented May 28, 2021

@redaguermas I just reran the test. A recent pr fixes this flaky test so not in your code. When you pull out the extra files it should run again.

@ChrisHuie ChrisHuie self-requested a review May 28, 2021 16:02
@ChrisHuie ChrisHuie self-assigned this May 28, 2021
@redaguermas
Copy link
Contributor Author

@ChrisHuie ,
I removed the IE files. Can you check the test please.

Thanks

@ChrisHuie
Copy link
Collaborator

@redaguermas looks good :)

@ChrisHuie ChrisHuie merged commit 00b1afa into prebid:master May 28, 2021
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