💭 Discussion: Modernize Architecture - Monorepo with React + TypeScript #64
Replies: 5 comments 9 replies
-
Beta Was this translation helpful? Give feedback.
-
|
I agree with separate frontend code from backend code, but I disagree with making this a React project especially with Typescript. I also really don't want another React project either. I have at least 5 others. Perhaps the better task would be separating the frontend code here from the backend code. However we still have 8-10 open PRs to get through before making any significant changes Thanks though |
Beta Was this translation helpful? Give feedback.
-
|
@gbowne1 |
Beta Was this translation helpful? Give feedback.
-
|
The proposed structure looks clean and well organized, especially the clear separation between frontend and backend. From a long term perspective, this kind of modularity definitely helps with maintainability. One additional small thought for the future: it might be helpful to reserve a minimal database or data-layer file/config early on (even as a placeholder). As the project grows, introducing a DB later can become more difficult, so having a basic structure in place could reduce friction without adding much complexity now. |
Beta Was this translation helpful? Give feedback.
-
|
I'll be a bit less apprehensive about merging once the CI workflow PR and some tests land. Ensure that this runs in node 20/22/24 |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
Current state
The project uses:
controllers/,middleware/,models/) and frontend code (js/,css/) are both insrc/Concerns
src/, which can confuse contributors.Proposed architecture
Convert to a monorepo with clear separation:
Benefits
Migration considerations
Questions for discussion
Next steps (if approved)
What are your thoughts? Should we proceed with this modernization, or discuss alternatives?
Beta Was this translation helpful? Give feedback.
All reactions