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

Sublime Bid Adapter : send notid in pixel & use timeout from timeoutData #7167

Merged
merged 2 commits into from
Jul 19, 2021

Conversation

o3LL
Copy link
Contributor

@o3LL o3LL commented Jul 12, 2021

Type of change

  • Feature

Description of change

  • Replace trId with notid (notifyId) in our pixels
  • Use timeoutData to retrieve timeout value
  • Use bidRequest to retrieve timeout value
  • Updating tests
  • Bumping version to 0.7.3

Contact email of the adapter’s maintainer : pbjs@sublime.xyz

Léo and others added 2 commits July 7, 2021 11:49
* Remove trId and use notid instead
* Update version to 0.7.3
* Set default value to empty string

Co-authored-by: SublimeJeremy <jeremy@sublimeskinz.com>
* Update timeout value with onTimeout data
* Add test for onTimeout
* Update state with timeout asap
@o3LL
Copy link
Contributor Author

o3LL commented Jul 12, 2021

It seems like test are failing due to flakiness ?
Let us know if there is something to do on our side 👍

@ChrisHuie ChrisHuie requested a review from msm0504 July 12, 2021 15:49
@ChrisHuie ChrisHuie changed the title Sublime Bid Adapter : v0.7.3 - Send notid in pixel & use timeout from timeoutData Sublime Bid Adapter : send notid in pixel & use timeout from timeoutData Jul 12, 2021
@msm0504 msm0504 merged commit 4d38732 into prebid:master Jul 19, 2021
prebidtappx pushed a commit to prebidtappx/Prebid.js that referenced this pull request Nov 15, 2021
…ata (prebid#7167)

* DL-1746: Remove trId and use notid instead (prebid#31)

* Remove trId and use notid instead
* Update version to 0.7.3
* Set default value to empty string

Co-authored-by: SublimeJeremy <jeremy@sublimeskinz.com>

* DL-1780: Fix pubtimeout (prebid#32)

* Update timeout value with onTimeout data
* Add test for onTimeout
* Update state with timeout asap

Co-authored-by: SublimeJeremy <jeremy@sublimeskinz.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

3 participants