binshops / prestashop-rest

PrestaShop REST API module to expose your PrestaShop website's REST endpoints
https://www.binshops.com/prestashop-api
Academic Free License v3.0
95 stars 33 forks source link

Plugin improvements #38

Closed amirping closed 1 year ago

amirping commented 1 year ago

Hello there,

We really did love your plugin, it was really helpful to allow us to create mobile applications with the Prestashop backend. We added some improvements to the plugin (PRO version) to suit our needs.

We also plan to add more features that are required for better mobile UX and support other Prestashop plugins (such as loyalty points etc...)

We want also to share this progress and create a small nice community around this plugin, we are looking forward to contributing.

samberrry commented 1 year ago

Hello, Thanks for your attention, I highly appreciate your contributions. I try to help you as much as I can.

To add more APIs to the module, we need to take two important considerations: 1- We should use core classes and controllers as much as possible, so we should have less touch and custom codes in our APIs. 2- Every module must have its own API, therefore as a general rule, module overrides would be a good solution. You know, now Binshops REST API enables third-party modules to have their own APIs. If we want to have APIs within Binshops REST module, we should override their controllers to add API support. It's not a good idea to load modules within Binshops REST API to add API support. We can use module overrides, or we can create new controllers to add API support.

Anyway, we can coordinate everything together. Thank you

samberrry commented 1 year ago

As the issue is inactive I'm going to close the issue, feel free to open it if needed.