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

Postbid bugfix #1844

Merged
merged 2 commits into from
Nov 28, 2017
Merged

Postbid bugfix #1844

merged 2 commits into from
Nov 28, 2017

Conversation

jaiminpanchal27
Copy link
Collaborator

Type of change

  • Bugfix

Description of change

Bug fixed for prebid conf variables like timeout, adUnitBids, sizes.
Added build task to update postbid.js. gulp build-postbid

gulpfile.js Outdated
@@ -296,10 +297,18 @@ gulp.task('e2etest-report', function() {
}, 5000);
});

gulp.task('build-postbid', function() {
gulp.task('build-postbid', ['escape-postbid-config'], function() {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you add comment here about his being for OAS Postbid builds

@mkendall07
Copy link
Member

@jaiminpanchal27 need a update here before merge

@mkendall07 mkendall07 merged commit 2e7783a into master Nov 28, 2017
@mkendall07 mkendall07 deleted the postbid-bugfix branch November 28, 2017 20:24
jsnellbaker added a commit that referenced this pull request Nov 28, 2017
jsnellbaker added a commit that referenced this pull request Nov 28, 2017
* Revert "Postbid bugfix  (#1844)"

This reverts commit 2e7783a.

* Revert "Populate adtypes array (#1862)"

This reverts commit 214ba19.

* Revert "Pass bidderRequestId parameter in Adform adapter (#1888)"

This reverts commit 59800f4.

* Revert "Add Arteebee bidder adapter (#1849)"

This reverts commit 18fe42d.

* Revert "fix linting errors for trailing spaces (#1889)"

This reverts commit d41287a.

* Revert "Randomize pbjs global name when testing (#1787)"

This reverts commit f9cb0dd.
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