Be Consistent - response

[Nizar7zak]

shoestore/server/controller/getProductID.js

Line 5 in b9c46b5

.then(({ rows }) => res.status(201).json(rows))

in responses you send the 
1.   .json({ status: 201, message: 'Product Added', product: rows[0] }))
2.  .json({ status: 200, message: 'Product Deleted' }))
3. .json(rows))
4. .json({ status: 200, message: 'Product Updated', product: rows[0] }))

should be consistent when we send a response for client side, moreover why you send a status in the body of response if we have it in status > res.status(200)

here is a question for you all, what if the rowCount equal 0, I mean what if there is no products in data base? In the same context we handle the 4xx errors in catch of promise, not the internal server error, we handle this error in Express error middleware like this ;

const serverError = (err, req, res, next) => {
  return res.status(500).json({ message: 'internal server error 500' });
};

you did a good work, all issues just to be a professional developers ;)