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 [email protected] 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 https://github.com/ForestAdmin/forest-express/issues/60#issuecomment-289389118, or mute the thread https://github.com/notifications/unsubscribe-auth/AACZzkcOH31mcWCszyYOP-C4JD9xoB_1ks5rp3VHgaJpZM4Moc5V .