DamianUS / acme-market

2 stars 1 forks source link

Delivery de pedidos #4

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.

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

  1. El usuario logeado es el clerk asignado (403 si no).
  2. Que no tiene cancelationMoment.