Make the integration system pluggable

[louisremi]

Hi,

I'd like to integrate InvoiceNinja into Forest, by taking inspiration on the Stripe integration. After digging around the integration code, I realize it will be difficult for me to do that without forking forest-express.
It seems it could be easy to change https://github.com/ForestAdmin/forest-express/blob/devel/integrations/index.js#L5 to initialize modules with values passed through the liana options.

Before I open a pull request, I'd like to know if you would be willing to make the integration system pluggable.

Thanks :-)

[arnaudbesnier]

Hey @louisremi , that's an interesting use case!
Why wouldn't you fork the repository to add InvoiceNinja integration? The community could be happy to use your contribution.

[louisremi]

I'm happy to open-source my integration, but it will probably not be as complete as the Stripe integration, so I'm not sure it's a good idea to make it an official integration just yet :-)

…

On 27 March 2017 at 10:37, Arnaud Besnier ***@***.***> wrote: Hey @louisremi <https://github.com/louisremi> , that's an interesting use case! Why wouldn't you fork the repository to add InvoiceNinja integration? The community could be happy to use your contribution. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACZzkcOH31mcWCszyYOP-C4JD9xoB_1ks5rp3VHgaJpZM4Moc5V> .