Skip to content

Conversation

@cymantic
Copy link

@cymantic cymantic commented Feb 8, 2014

I've updated the _id to be the path to the file in mongodb

Also a few other formatting and renaming of files

@brianmc
Copy link
Owner

brianmc commented Feb 9, 2014

Thanks for the PR Tom, lots of good improvements. Just interested in the move of files.js from routes to lib. I'm fairly new to node so just wondering if you were following a standard node/express organization convention and what that was.

Thanks,

Brian

@cymantic
Copy link
Author

cymantic commented Feb 9, 2014

I think lib is more standard from whenever I've used express (or sinatra), especially for files that provide additional functionality. I've never used a routes folder, but if you had a large app with extra routing config, then those files would go there.

@BNoohi
Copy link

BNoohi commented Jan 27, 2015

@brianmc What's the status on this merge? Is this project being actively worked on?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants