ai-cfia / nachet-frontend

Frontend application for seed classification of images acquired from digital microscopes
MIT License
2 stars 2 forks source link

Code Quality Modularization Decoupling Testing #123

Open ChromaticPanic opened 1 month ago

ChromaticPanic commented 1 month ago

Some components are too highly coupled and violate the single responsibility principle making some components hard to test and debug

I would like to propose the following solution:

ChromaticPanic commented 1 month ago

For the Body specifically, I think we can move all the popup JSX code down into the component tree close to the code that triggers them if we show them as Modals.

rngadam commented 1 month ago

Ensure that @CFIALeronB understands the plan for change so he can adequately comment on it.