search

idurar / idurar-erp-crm

Free Open Source ERP CRM Accounting Invoicing Software | Node Js React
https://cloud.idurarapp.com
GNU Affero General Public License v3.0
6.27k stars 1.89k forks source link

Consolidate Frontend and Backend Run Scripts into a Single Script #1079

Open Shashan17J opened 5 months ago

Shashan17J commented 5 months ago

Currently, the repository has separate run scripts for the frontend and backend (npm run dev: frontend and npm run dev: backend respectively), even though both scripts essentially do the same thing (npm run dev). This duplication in run scripts adds unnecessary complexity to the project setup and maintenance.

Shashan17J commented 5 months ago

working

Skanderloueti commented 5 months ago

Another viable approach to address this issue is serving the SPA (Single Page Application) files directly from the backend after building them using npm run build. Here's a pull request that implements this alternative method: #1084 . This solution not only simplifies the setup but also streamlines maintenance efforts. It's worth considering for enhancing the project's efficiency and maintainability.

Shashan17J commented 5 months ago

I checked your pull request, I think that it may lead to increased server load, especially for large applications, and can make it harder to scale the frontend and backend independently.