You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on May 12, 2021. It is now read-only.
This is a no-brainer, really. CRUD ops should live in their own happy little module. Things inside these files are very spaghetti; no function should be as long as some of the ones in these files are.
Easy to solve by factoring the operations out, would also DRY things up a whole lot.
Had to ship though, so these were written in a very non-ideal rush.
The text was updated successfully, but these errors were encountered:
Create.js jas since been refactored a bit, and some operations factored out into their own modules.
Edit.js is a plate of spaghetti and C&P (OMG EMBARRASSING) which hopefully I'll get to cleaning up soon.
There is still not enough separation between routing flow and the DB ops though. More work to be done on both of them there once edit.js is a bit more sane.
This is a no-brainer, really. CRUD ops should live in their own happy little module. Things inside these files are very spaghetti; no function should be as long as some of the ones in these files are.
Easy to solve by factoring the operations out, would also DRY things up a whole lot.
Had to ship though, so these were written in a very non-ideal rush.
The text was updated successfully, but these errors were encountered: