Discource-C
/
discourse-adplugin
mirror of https://github.com/discourse/discourse-adplugin.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #22 from nlalonde/master 

DFP: send a 0 value for discourse-category targeting on /latest, /categories, etc so that ads can show there while targeting categories
This commit is contained in:
Robin Ward 2015-10-28 12:40:55 -04:00
parent f9ba0e4ae1 85d357b007
commit 89dc8b50df
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
assets/javascripts/discourse/components/google-dfp-ad.js.es6
View File

@ -199,7 +199,7 @@ export default Ember.Component.extend({
if (this.get('loadedGoogletag') && this.get('refreshOnChange')) { if (this.get('loadedGoogletag') && this.get('refreshOnChange')) {
googletag.cmd.push(function() { googletag.cmd.push(function() {
ad.setTargeting('discourse-category', self.get('category') ? self.get('category') : null); ad.setTargeting('discourse-category', self.get('category') ? self.get('category') : '0');
googletag.pubads().refresh([ad]); googletag.pubads().refresh([ad]);
}); });
} }
@ -212,7 +212,7 @@ export default Ember.Component.extend({
googletag.cmd.push(function() { googletag.cmd.push(function() {
var ad = defineSlot(self.get('placement'), self.siteSettings); var ad = defineSlot(self.get('placement'), self.siteSettings);
if (ad) { if (ad) {
ad.setTargeting('discourse-category', self.get('category') ? self.get('category') : null); ad.setTargeting('discourse-category', self.get('category') ? self.get('category') : '0');
googletag.display(self.get('divId')); googletag.display(self.get('divId'));
googletag.pubads().refresh([ad]); googletag.pubads().refresh([ad]);
} }