Skip to content

Remove pixels on non-event and add onBidWon#22

Merged
SublimeLeo merged 7 commits intosublime-adapterfrom
feature/remove-pixels
Apr 8, 2020
Merged

Remove pixels on non-event and add onBidWon#22
SublimeLeo merged 7 commits intosublime-adapterfrom
feature/remove-pixels

Conversation

@SublimeLeo
Copy link

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

Changes:

  • Removed pixels
  • Added onBidWon event pixel

Test parameters for validating bids

{
  bidder: 'sublime',
  params: {
    zoneId: 1
  }
}

ghourlier and others added 3 commits March 12, 2020 11:59
* Update sublimeBidAdapter to 0.5.1
* Add tests for private functions
* Remove window.sublime
* Update pixel name for bid event
@SublimeJeremy
Copy link

@SublimeLeo C'est bien rebase depuis le master de https://github.com/prebid/Prebid.js ?

@SublimeLeo
Copy link
Author

@SublimeJeremy Maintenant oui!

Co-Authored-By: fgcloutier <fg.cloutier@sublimeskinz.com>
@SublimeLeo SublimeLeo requested a review from ghourlier April 8, 2020 12:22
@SublimeLeo SublimeLeo merged commit d45f02e into sublime-adapter Apr 8, 2020
@SublimeLeo SublimeLeo deleted the feature/remove-pixels branch April 8, 2020 12:23
fgcloutier added a commit that referenced this pull request Sep 4, 2020
* add prebid version of adapter

* Feature/update sublime adapter (#21)

* Update sublimeBidAdapter to 0.5.1
* Add tests for private functions
* Remove window.sublime
* Update pixel name for bid event

* Remove pixels on non-event and add onBidWon (#22)

* add prebid version of adapter
* Feature/update sublime adapter (#21)
* Update sublimeBidAdapter to 0.5.1
* Add tests for private functions
* Remove window.sublime
* Update pixel name for bid event
* Remove pixels on non-event and add onBidWon
* Incremente version of sublimeBidAdapter
* Renamed pixel for timeout and introduce gvlid
* Remove unnecessary params for sendEvent

Co-Authored-By: fgcloutier <fg.cloutier@sublimeskinz.com>
Co-authored-by: Gaby <gaby.hourlier@sublimeskinz.com>
Co-authored-by: fgcloutier <fg.cloutier@sublimeskinz.com>

* Remove trailing-space

* Fix version in tests

Co-authored-by: Gaby <gaby.hourlier@sublimeskinz.com>
Co-authored-by: fgcloutier <fg.cloutier@sublimeskinz.com>
o3LL pushed a commit that referenced this pull request Jun 1, 2021
prebid#6442)

* add prebid version of adapter

* Feature/update sublime adapter (#21)

* Update sublimeBidAdapter to 0.5.1
* Add tests for private functions
* Remove window.sublime
* Update pixel name for bid event

* Remove pixels on non-event and add onBidWon (#22)

* add prebid version of adapter
* Feature/update sublime adapter (#21)
* Update sublimeBidAdapter to 0.5.1
* Add tests for private functions
* Remove window.sublime
* Update pixel name for bid event
* Remove pixels on non-event and add onBidWon
* Incremente version of sublimeBidAdapter
* Renamed pixel for timeout and introduce gvlid
* Remove unnecessary params for sendEvent

Co-Authored-By: fgcloutier <fg.cloutier@sublimeskinz.com>
Co-authored-by: Gaby <gaby.hourlier@sublimeskinz.com>
Co-authored-by: fgcloutier <fg.cloutier@sublimeskinz.com>

* Remove trailing-space

* Fix version in tests

* sublimeAdapter: Improve pixels data

* sublimeAdapter: Update tests

* sublimeAdapter: Rename data

Co-authored-by: Gaby <gaby.hourlier@sublimeskinz.com>
Co-authored-by: Léo <51166933+SublimeLeo@users.noreply.github.com>
Co-authored-by: fgcloutier <fg.cloutier@sublimeskinz.com>
Co-authored-by: Léo GRAND <leo.grand@sublimeskinz.com>
o3LL pushed a commit that referenced this pull request Jul 2, 2021
* #22 Removing a/b logic and using result from server side

* #22 Better docs

* #22 Added a check in a test case

* fix typo

Co-authored-by: Marco Cosentino <mcosentino@id5.io>
Co-authored-by: Scott Menzer <scott@id5.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Comments