snailbites / node-chat

A simple chat server created with Node.js
1 stars 2 forks source link

My proposal for controllers (or routing) architecture #7

Closed adamhenson closed 9 years ago

adamhenson commented 9 years ago

I like this solution but if you guys don't - let's pool together other ideas. These app methods (get and post) tend to get large, so I think it's good to modularize them and pull them out of 'server.js'. It also creates a sort of controller type of code base.

adamhenson commented 9 years ago

Also added image uploading. Could maybe re-use that code in the future for avatars, etc. This currently allows the user to upload an image directly into the chat. Depends on GraphicsMagick and obviously server space.

jaredmcdonald commented 9 years ago

merged!

adamhenson commented 9 years ago

Nice, thanks man

Sent from my iPhone

On Nov 14, 2014, at 6:29 PM, jared mcdonald notifications@github.com wrote:

merged!

— Reply to this email directly or view it on GitHub.