Shelter assignment

[jonahgoldwastaken]

All done! I really liked this assignment, as I learnt a lot about trys and catches! I'm for sure going to use those in future projects!

[wooorm]

Nice! tip for future forks: open the issue tab through settings so people can provide feedback there (but it’s fine for be assignments).

• I believe http-status is also something that’s available in express already through https://github.com/jshttp/statuses!
• tip: use a different script for nodemon, e.g., npm run watch or live maybe?
• don’t use global dependencies in your project! nodemon, when run through an npm script, can be a normal dev-dependency
• file-filter: as file.mimetype != 'image/jpeg' is either true or false, you could shorten the function to cb(null, file.mimetype == 'image/jpeg')
• don’t forget to return if you send something back!. This fails!
• maybe use content-type or type-is to check the content type of a request?
• maybe redirect to the new animal after the image is moved, as maybe now users will see a 404 because the file isn’t done moving (unlikely, so maybe discard this tip)
• Maybe instead of a for..in use Object.assign?
• Awesome that you’re removing the image as well, I totally forgot to do that in my solution haha 😬
• Cool to see the onerror helper!
• Don’t forget a link to the form that adds animals!
• I can’t add a weight of 1.1, I believe you have to add a step="0.1" to allow weights smaller than an integer on numerical inputs!

Otherwise, awesome work! 💪