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

Fixes #3197 - call auctionDone() when 'No valid bid requests returned for auction' #3198

Conversation

drdmitry
Copy link
Contributor

call auctionDone() when 'No valid bid requests returned for auction'

Type of change

  • Bugfix

Description of change

When 'No valid bid requests returned for auction' we do call auctionDone() to have bidsBackHandler gets called.

Other information

#3197

@jaiminpanchal27 jaiminpanchal27 self-assigned this Oct 18, 2018
@jaiminpanchal27 jaiminpanchal27 self-requested a review October 18, 2018 14:20
@jaiminpanchal27
Copy link
Collaborator

@drdmitry Code looks good. Please add unit test

@jaiminpanchal27 jaiminpanchal27 merged commit 14d3031 into prebid:master Oct 23, 2018
idettman pushed a commit to rubicon-project/Prebid.js that referenced this pull request Nov 14, 2018
…turned for auction' (prebid#3198)

* Fixes prebid#3197 - call auctionDone() when 'No valid bid requests returned for auction'

* prebid#3197 Added unit test to check if callback is called when bidRequests is empty
ghost pushed a commit to devunrulymedia/Prebid.js that referenced this pull request Jan 30, 2019
…turned for auction' (prebid#3198)

* Fixes prebid#3197 - call auctionDone() when 'No valid bid requests returned for auction'

* prebid#3197 Added unit test to check if callback is called when bidRequests is empty
pedrolopezmrf pushed a commit to Marfeel/Prebid.js that referenced this pull request Mar 18, 2019
…turned for auction' (prebid#3198)

* Fixes prebid#3197 - call auctionDone() when 'No valid bid requests returned for auction'

* prebid#3197 Added unit test to check if callback is called when bidRequests is empty
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.

2 participants