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

Audiencerun Bid Adapter 1.1.0 #6919

Merged
merged 6 commits into from
Jun 30, 2021
Merged

Audiencerun Bid Adapter 1.1.0 #6919

merged 6 commits into from
Jun 30, 2021

Conversation

audiencerun
Copy link
Contributor

Type of change

  • Bugfix
  • Feature
  • New bidder adapter
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Does this change affect user-facing APIs or examples documented on http://prebid.org?
  • Other

Description of change

Other information

Geoffroy Hutin added 2 commits June 2, 2021 10:06
Develop

See merge request equipe-formats/audiencerun-prebid-adapter!29
@patmmccann
Copy link
Collaborator

patmmccann commented Jun 2, 2021

Please fix floors issue #6465 , #6650, and make this on timeout behavior configurable

@audiencerun audiencerun changed the title Audiencerun Adapter 1.1 Audiencerun Adapter 1.1.0 Jun 2, 2021
@audiencerun audiencerun changed the title Audiencerun Adapter 1.1.0 Audiencerun Bid Adapter 1.1.0 Jun 2, 2021
@patmmccann
Copy link
Collaborator

@audiencerun can you confirm this is intentionally still in draft?

@audiencerun audiencerun marked this pull request as ready for review June 10, 2021 08:15
@audiencerun
Copy link
Contributor Author

Thanks

@patmmccann
Copy link
Collaborator

patmmccann commented Jun 10, 2021

6465 fixes are required for pb5 inclusion

Making ontimeout configurable would just mean wrapping the call in something like if(!getConfig(...)) {}

Alternatively, you could just add a disclosure you make calls ontimeout to your doc.

@Rothalack
Copy link
Collaborator

@audiencerun Did you still need to make the timeout change?

@Rothalack
Copy link
Collaborator

Hey @audiencerun just checking back in on this. If I'm understanding correctly you need to make a documentation update or an update to how you're doing onTimeout. Are you intending to do either of those or am I misunderstanding the status of this?

Geoffroy Hutin and others added 2 commits June 30, 2021 09:43
develop -> master

See merge request equipe-formats/audiencerun-prebid-adapter!34
@audiencerun
Copy link
Contributor Author

Hi, we have added some fixes:

However, some test cases failed for other adapters on CircleCI. Please let us know If we have to do something on our side or you can simply re-run the tests?

@Rothalack
Copy link
Collaborator

Rerunning the test worked, it failed on a normal timeout issue we see all the time on tests. My tests worked as well. This looks good to go!

@Rothalack Rothalack merged commit 5c00a6d into prebid:master Jun 30, 2021
@audiencerun
Copy link
Contributor Author

Hi @Rothalack,

Just noticed that there is a disclosure for our adapter on https://docs.prebid.org/dev-docs/bidders/audiencerun.html#description about adId (#6381). Should we submit another pull request to update the documentation too?

@Rothalack
Copy link
Collaborator

Hi @Rothalack,

Just noticed that there is a disclosure for our adapter on https://docs.prebid.org/dev-docs/bidders/audiencerun.html#description about adId (#6381). Should we submit another pull request to update the documentation too?

Definitely! You should be able to make that PR and mention this PR to show the code.

prebidtappx pushed a commit to prebidtappx/Prebid.js that referenced this pull request Nov 15, 2021
* audiencerun adapter : remove bid.adId, timeout pixel, gdpr version

* Audiencerun fix: handle advertiserDomains on meta object  and remove adId from spec tests

* AudienceRun fix: add support for floors module and advertiserDomains

Co-authored-by: Geoffroy Hutin <g.hutin@blackcover.fr>
Co-authored-by: Abdessalam Benharira <a.benharira@unlimitedpix.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.

None yet

4 participants