There are certain circumstances when it is wise to hide the ads for specific content (ex. You write an article on a plane crash and one of your major advertisers is Delta) ...

This should come with a new permission as well

Comments

bleen’s picture

bleen commented

note to self: we may want to add a "house ads only" checkbox as well which overrides the Ad Unit Id to something like "house_ads" or whatever (configurable?)

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented

@bleen18, I don't think hiding ads on certain content is really necessary in the the core, cause it is already available in alter hook right?. We can also use context or block configurations to manage that, so I think this one is nor really necessary in the DFP core.

+1 for house ads only feature, we really need it.

bleen’s picture

bleen commented

Cant the exact same mechanisms be used to add "house_only" as can be used to hide ads from the page? ... that said, my thought here was that its a pain to create contexts for an individual node and even more of a pain to write code for an individual node.

Thoughts?

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented

I think "house_only" make sense to have in the DFP core, like "hide empty tags" right?. If site builders knows how to use Context (I believe most of them) then it won't be a big effort, same for the developers too.

How do you plan to do that, by altering the node edit?.

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Issue summary: View changes
Status: Active » Postponed (maintainer needs more info)

There was no activity for many moons! 🌝
Is this still required?

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Status: Postponed (maintainer needs more info) » Closed (outdated)

Thank you for your contributions.
Drupal 7 is no longer supported.
Closing issue as outdated.