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

DFP [7.x-2.x] Maintainer #1

Open
scottalan opened this issue Jun 21, 2018 · 6 comments
Open

DFP [7.x-2.x] Maintainer #1

scottalan opened this issue Jun 21, 2018 · 6 comments

Comments

@scottalan
Copy link

I just came across your module and I'd like to help out. I maintain the 7.x-2.x branch of the DFP module. I noticed someone (maybe someone you know) grabbed the same namespace. Is this where you are keeping development of this module or do you have a project page on D.O. as well?

I like the idea of using ctools exports to store data about each of the bidders. I will probably have to update a few things to get it to work for our client so I might be creating a few PRs. I had to do something similar for DFP which is why there is a 2.x branch.

@heshanlk
Copy link
Owner

Yes @scottalan I saw that but have no idea who that is so I opened this https://www.drupal.org/project/prebidjs/issues/2934453.

Feel free to fork and send me PRs or take full control 👍

@scottalan
Copy link
Author

@heshanlk Thanks for responding. I'll see if I can't get some help moving that issue along.

@heshanlk
Copy link
Owner

That'd be great!. Let me know if I can help you with anything.

@scottalan
Copy link
Author

@heshanlk, from what I can gather I'm seeing that there is a legacy version and I'm assuming this is anything < 1.x.x? It looks like even prebid.org is behind unless it is actually using the "latest" and they just haven't updated the option.

ss-from-scott 2018-06-21 at 6 32 52 pm

I wasn't sure what the versioning requirements were for the master branch but I'm required to use the "latest" (1.13.0) for the current project I'm working on. I can update bits and pieces here and there checking for the $library['version'], but I feel this could get out of control at some point.

Maybe master could become a 7.x-1.x-dev on D.O. and maybe what I'm working on could be a 7.x-2.x-dev branch that requires the newer version?

Let me know what you think.

@heshanlk
Copy link
Owner

@scottalan the idea of using Libraries API was to remove that kind of version dependencies, don't you think using just the version you want in sites/all/libraries would do it?.

I can't remember how much the syntax is different for those two versions so feel free to brank off if that makes sense.

@scottalan
Copy link
Author

I ended just adding a submodule (prebid) to the 7.x-2.x branch of DFP. The fact that there is no standardization to the bidder parameters is insane IMO. I opted to go with a plugin system via ctools that allows a module to create individual bidder plugins the handle unique use cases for each of the bidders.

For example:

One of the bidders we are using is index exchange IX.
The following are the parameters it requires:

'params' => array(
    'siteId' => '',
    'size' => array(),
  ),

I had to define a different siteId for each of the ad slots. The plugin system allowed me to do this for this particular bidder.

Here's the patch we are testing now https://www.drupal.org/project/dfp/issues/2980845#comment-12730946

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants