From c2454d142841aa3a78a489f055692233f2fa93de Mon Sep 17 00:00:00 2001
From: Brandon Millman <brandon.millman@gmail.com>
Date: Tue, 21 Aug 2018 13:52:27 -0700
Subject: Add more TODO items

---
 packages/forwarder-helper/src/forwarder_helper_impl.ts | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'packages')

diff --git a/packages/forwarder-helper/src/forwarder_helper_impl.ts b/packages/forwarder-helper/src/forwarder_helper_impl.ts
index 39e84b477..43b2962da 100644
--- a/packages/forwarder-helper/src/forwarder_helper_impl.ts
+++ b/packages/forwarder-helper/src/forwarder_helper_impl.ts
@@ -56,6 +56,7 @@ export class ForwarderHelperImpl implements ForwarderHelper {
             orders,
             remainingFillableMakerAssetAmounts,
         );
+        // TODO: provide a feeRate to the sorting function to more accurately sort based on the current market for ZRX tokens
         const sortedOrderWithAmounts = sortingUtils.sortOrdersByFeeAdjustedRate(orderWithAmounts);
         const unbundledSortedOrderWithAmounts = ForwarderHelperImpl._unbundleSignedOrderWithAmounts(
             sortedOrderWithAmounts,
@@ -108,6 +109,7 @@ export class ForwarderHelperImpl implements ForwarderHelper {
             throw new Error(ForwarderHelperError.InsufficientZrxLiquidity);
         }
         // TODO: calculate min and max eth usage
+        // TODO: optimize orders call data
         return {
             makerAssetFillAmount,
             orders: resultOrders,
-- 
cgit