search

dwyl / time-mvp-node

The most basic version of our Time App possible to start *using* it internally!
https://time-mvp.herokuapp.com
GNU General Public License v2.0
13 stars 2 forks source link

Separate Database (CRUD) code from server.js to make the server more readable #18

Closed nelsonic closed 7 years ago

nelsonic commented 7 years ago

the server.js file is getting pretty large because of the DB-related code ... see: https://github.com/nelsonic/time-mvp/blob/8fa7cf278fcdfc8f163cb1b9b2051efbd0697468/lib/server.js

I propose that we split it out into a ./lib/db.js file and add a few Docs comments while we're at it. 😉

nelsonic commented 7 years ago

Took less than T25m because the code was quite straightforward. https://github.com/nelsonic/time-mvp/blob/94d4e237f3a1759f7c6f0f69e98f923c014797cd/lib/server.js much simpler / more readable. :tada: