Node.js express example - #1
Conversation
…r temp and final file
|
Hi there Jeremy, and thanks for this contribution. Just a couple comments and one requested change:
|
|
Hi Ray! Thanks for the feedback. I think I must have used the 404 from code I had based on the original example. Will change for sure. On the client-side code, I understand you wanting to keep them separate, but since I was adapting this from a project I am working on, was concerned that removing too many moving parts would cause breakage, and also wanted to make sure I was capable of testing. Totally your call - if you would rather I pull the client side stuff and replace with comments, I could do that too. Thanks again for looking at it - the newer versions of express make the integration SO much easier than what was given in the original example. I have been hunting high and low for a couple of weeks to find best way to implement, and it turns out it was much easier than I thought. Also, thanks for a great file upload tool! |
|
Jeremy - Thanks for the compliment! You can go ahead and leave the client-side code in. I can always remove it or modify it later if I run into maintenance headaches. The 404 -> 200 adjustment, along with any mention of yourself in the code comments should be the only changes necessary. Also, can you please confirm that you've tested your example in IE9 or older, along with a File API browser (Chrome, Firefox, or Safari for OS X)? |
|
Made the appropriate adjustments, and can verify that it is tested and working in Safari, Chrome, Firefox and IE8. |
|
Looks good @logickal. Thanks again for your contribution! |
Hi there! I've created a more robust (but also more up-to-date) server example for node.js and express. Definitely interested in your feedback - I believe that the current example is based on an older version of express and is now out of date. Additionally, I included front-end code to make it a fully-working example.
Thanks!