search

vmeretail / multi-tenancy-bundle

A multi tenancy bundle for Symfony 2 apps
25 stars 11 forks source link

Make recurly optional #26

Closed StephenGillCoop closed 9 years ago

StephenGillCoop commented 9 years ago

It shouldn't force entries to be completed in parameters.yml. It should be completely optional, so users who don't want recurly integration aren't affected.

pmartelletti commented 9 years ago

:+1: Totally agreed.

pmartelletti commented 9 years ago

@stevergill in fact, this is optional at the moment! I knew I have already worked on this! :smile: 

       DependencyInjection/TahoeMultiTenancyExtension.php
       ....
       if (array_key_exists('gateways', $config)) {
            $loader->load('gateways.yml');
            $this->loadGatewaysParameters($container, $config['gateways']);
        }

So, if you just omit the gateways entry in the config, you won't need to setup recurly params. But, if you enable the gateways option, you would need to enter the recurly parameters as, at the moment, is the only one we support.

In the future, the developer would be able to chose one or many from:

There are many out there that we could support (or give third party developers the ability to integrate easily). But at the moment, if they want to integrate a gateway, the should use Recurly. :smile: