Added require reasons to MixinMatchOrders and cleaned up some comments.

1 files changed, 15 insertions, 20 deletions

diff --git a/packages/contracts/src/utils/types.ts b/packages/contracts/src/utils/types.ts
index ce7fbcbe1..0e3b2c9a8 100644
--- a/packages/contracts/src/utils/types.ts
+++ b/packages/contracts/src/utils/types.ts
@@ -75,26 +75,21 @@ export interface Token {
 }
 
 export enum ExchangeStatus {
-    /// Default Status ///
-    INVALID, // General invalid status
-
-    /// General Exchange Statuses ///
-    SUCCESS, // Indicates a successful operation
-    ROUNDING_ERROR_TOO_LARGE, // Rounding error too large
-    INSUFFICIENT_BALANCE_OR_ALLOWANCE, // Insufficient balance or allowance for token transfer
-    TAKER_ASSET_FILL_AMOUNT_TOO_LOW, // takerAssetFillAmount is <= 0
-    INVALID_SIGNATURE, // Invalid signature
-    INVALID_SENDER, // Invalid sender
-    INVALID_TAKER, // Invalid taker
-    INVALID_MAKER, // Invalid maker
-
-    /// Order State Statuses ///
-    ORDER_INVALID_MAKER_ASSET_AMOUNT, // Order does not have a valid maker asset amount
-    ORDER_INVALID_TAKER_ASSET_AMOUNT, // Order does not have a valid taker asset amount
-    ORDER_FILLABLE, // Order is fillable
-    ORDER_EXPIRED, // Order has already expired
-    ORDER_FULLY_FILLED, // Order is fully filled
-    ORDER_CANCELLED, // Order has been cancelled
+    INVALID,
+    SUCCESS,
+    ROUNDING_ERROR_TOO_LARGE,
+    INSUFFICIENT_BALANCE_OR_ALLOWANCE,
+    TAKER_ASSET_FILL_AMOUNT_TOO_LOW,
+    INVALID_SIGNATURE,
+    INVALID_SENDER,
+    INVALID_TAKER,
+    INVALID_MAKER,
+    ORDER_INVALID_MAKER_ASSET_AMOUNT,
+    ORDER_INVALID_TAKER_ASSET_AMOUNT,
+    ORDER_FILLABLE,
+    ORDER_EXPIRED,
+    ORDER_FULLY_FILLED,
+    ORDER_CANCELLED,
 }
 
 export enum ContractName {