Hi,

I have ported this project to D8 : https://git.serval-agency.fr/drupal-project/youtube_api

Can I have git permissions?

Regards

Comments

netsliver created an issue.

googletorp’s picture

Hi.

Thanks for taking your time to port the module.

Contributing to a module and maintaining a module is two very different things. I don't mind adding you as a co-maintainer to the module, but I've had some bad experiences in the past, so I would like for this to be a transition, so we can get to know each other and figure out how we work together.

In order to use the code you provided, we need it as a patch or on a git based on this repository, otherwise we can't use it.

I looked at the code very quickly, and for me it seems like the module is built for a very specific case, I'm guessing you built it for a customer, but in order to have it on drupal.org, we don't want your custom look and feel inside the module. Your templates look like they are designed for material design, but we should not have this part of the module as a open for all module.

My vision for this module, is to make it easy for developers that need to integrate into youtube, to not need to understand all of the details, but use this as a tool to get video data etc. I don't think it makes sense to make fields for youtube or other features related to displaying youtube videos, as so many modules do this already and some features are part of Drupal core.

Would you be willing to make adjustments for the module so that it's not made just only for you, but is avaialble to many people and is more generic.

netsliver’s picture

Hi

Actually I forgot to remove the bootstrap classes.

I have two approaches in fact on the module, I have a global page that displays all the videos of a channel, and if you want I have a type of field that can display a playlist in a content.

I can of course remove bootstrap classes, the rest to a sense in my opinion.

netsliver’s picture

Hi,

I removed bootstrap and added the cache on the global page.

Regards