Skip to content

Commit

Permalink
'currency' isn't a bidder-specific param
Browse files Browse the repository at this point in the history
Update PR following this remark on another one:
#3228 (comment)
  • Loading branch information
piotr-yuxuan committed Oct 25, 2018
1 parent 50b8e37 commit 5bd6d47
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 8 deletions.
6 changes: 3 additions & 3 deletions modules/adikteevBidAdapter.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import {registerBidder} from 'src/adapters/bidderFactory';
import {BANNER} from 'src/mediaTypes';
import * as utils from '../src/utils';
import {config} from 'src/config';

export const BIDDER_CODE = 'adikteev';
export const ENDPOINT_URL = 'https://serve-adserver.adikteev.com/api/prebid/bid';
Expand Down Expand Up @@ -32,8 +33,6 @@ export const spec = {
return !!(
bid &&
bid.params &&
bid.params.currency &&
bid.params.currency === 'EUR' &&
bid.params.bidFloorPrice &&
bid.params.placementId &&
bid.bidder === BIDDER_CODE &&
Expand All @@ -45,7 +44,8 @@ export const spec = {
const payload = {
validBidRequests,
bidderRequest,
referrerInfo: bidderRequest.refererInfo,
refererInfo: bidderRequest.refererInfo,
currency: config.getConfig('currency'),
userAgent: navigator.userAgent,
screen: {
width: window.screen.width,
Expand Down
1 change: 0 additions & 1 deletion modules/adikteevBidAdapter.md
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ Module that connects to Adikteev's demand sources
bidder: 'adikteev',
params: {
placementId: 12345,
currency: 'EUR',
bidFloorPrice: 0.1,
}
}
Expand Down
6 changes: 2 additions & 4 deletions test/spec/modules/adikteevBidAdapter_spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,6 @@ describe('adikteevBidAdapter', () => {
bidder: 'adikteev',
params: {
placementId: 12345,
currency: 'EUR',
bidFloorPrice: 0.1,
},
mediaTypes: {
Expand Down Expand Up @@ -62,7 +61,6 @@ describe('adikteevBidAdapter', () => {
bidder: '', // invalid bidder
params: {
placementId: 12345,
currency: 'EUR',
bidFloorPrice: 0.1,
},
mediaTypes: {
Expand All @@ -75,7 +73,6 @@ describe('adikteevBidAdapter', () => {
bidder: 'adikteev',
params: {
placementId: '', // invalid placementId
currency: 'EUR',
bidFloorPrice: 0.1,
},
mediaTypes: {
Expand All @@ -88,7 +85,6 @@ describe('adikteevBidAdapter', () => {
bidder: 'adikteev',
params: {
placementId: 12345,
currency: 'EUR',
bidFloorPrice: 0.1,
},
mediaTypes: {
Expand Down Expand Up @@ -121,6 +117,8 @@ describe('adikteevBidAdapter', () => {
'screen',
'language',
'cookies',
// 'refererInfo',
// 'currency',
'prebidUpdateVersion',
);
expect(requestData.validBidRequests).to.deep.equal(validBidRequests);
Expand Down

0 comments on commit 5bd6d47

Please sign in to comment.