DamianUS / acme-market

2 stars 1 forks source link

Cancelación de pedidos #3

Open DamianUS opened 1 year ago

DamianUS commented 1 year ago

Requisito funcional no implementado.

Bajo mi punto de vista sería un nuevo endpoint porque no sería conceptualmente un put, ya que sería más bien establecer la fecha de envío (PATCH).

El enunciado dice: - Obviously, an order cannot be cancelled after it is delivered and it cannot be delivered if it was cancelled. => para mí sería más bien que no puede ser cancelado si ya tiene asignado un clerk, pero cualquiera vale.

Además, se debería añadir un middleware para comprobar que:

  1. El usuario logeado es el customer en cuestión. (403)
  2. Que no tiene clerk/deliveryMoment.