More flexible VSF integration design

[oerp-odoo]

Hello @dduarte-odoogap

Greetings from Versada.

We started using your VSF integration, but noticed that its not very flexible when you need to modify code without touching core (e.g. like you can do with Odoo). So currently we had to make a fork and do modifications separately which long term might not be that great.

I currently see 5 areas where it can be improved to make it more flexible, stable and easier to maintain (though this most likely would need to be new backwards incompatible version from the current one):

• Make GraphQL queries, mutations, types extendable, similarly how Odoo models can be.
• If we can make GraphQL classes extendable, then we should split integration into more appropriate modules, where there should be base module that is needed for basic integration. And then we would have extra modules to enhance functionality (e.g. extra payment options, integrations with other Odoo parts, like CRM etc).
• Start using coding style and pre-commit, so we could have nice and clean code (pretty easy to do)
• Cover it with unittests (then can be more confident in improving code later and catching issues early)
• Have a CI (e.g. using github actions) so everyone can see if their changes break anything (more stable code, more confidence in changes).

You can also find our current implementation in fork here (being developed for Odoo v14 now): https://github.com/versada/vuestorefront/blob/14.0-versada/vuestorefront/README.rst Also for v16: https://github.com/versada/vuestorefront/tree/16.0-versada In readme, there are some changes listed.

Let me know what do you think about such road map.

Have a great day:)

[schiggi]

Quick question, @oerp-odoo: Which version of the VSF middleware/approach have you used for your implementation? The SDK one or the old one?

[oerp-odoo]

Quick question, @oerp-odoo: Which version of the VSF middleware/approach have you used for your implementation? The SDK one or the old one?

We didn't change that one. Currently only odoo part was changed:)

[dduarte-odoogap]

@oerp-odoo sounds great, we will check soon