Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Architecture Reorg #61

Open
2 tasks
bvelitchkine opened this issue Nov 10, 2023 · 0 comments
Open
2 tasks

Architecture Reorg #61

bvelitchkine opened this issue Nov 10, 2023 · 0 comments
Assignees
Labels
improvement Improvement of smthg that already existed infra Scope is infrastructure

Comments

@bvelitchkine
Copy link
Member

bvelitchkine commented Nov 10, 2023

Both Gauvain and I are dissatisfied with the current project's architecture. It's become messy, both in the front and in the back.

Background

There should be a clear separation between: services, resolvers/controllers, providers and repositories in the background. This was the topic of a meeting.

Front-end

There's room for improvement especially in the Form.tsx file that has become very long and messy. Some key action points:

  • Subdivide in smaller components
  • Make a better use of react hooks (useContext, useFormHook, etc.)
@bvelitchkine bvelitchkine added improvement Improvement of smthg that already existed infra Scope is infrastructure labels Nov 10, 2023
@bvelitchkine bvelitchkine self-assigned this Nov 10, 2023
@bvelitchkine bvelitchkine changed the title Minor Architecture Reorg Architecture Reorg Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement of smthg that already existed infra Scope is infrastructure
Projects
None yet
Development

No branches or pull requests

1 participant