diff options
author | Jacob Evans <jacob@dekz.net> | 2017-11-23 12:14:33 +0800 |
---|---|---|
committer | Jacob Evans <jacob@dekz.net> | 2017-11-23 12:14:33 +0800 |
commit | 924b96ce2af038a9635d5a19802b07698e19b056 (patch) | |
tree | 756bbdd73e9b5ee031a52599b8436ddc87f65862 | |
parent | 47236dbaecf9613fad470014a40785076cbd6d8a (diff) | |
download | dexon-sol-tools-924b96ce2af038a9635d5a19802b07698e19b056.tar.gz dexon-sol-tools-924b96ce2af038a9635d5a19802b07698e19b056.tar.zst dexon-sol-tools-924b96ce2af038a9635d5a19802b07698e19b056.zip |
Fix nits
-rw-r--r-- | packages/0x.js/src/order_watcher/remaining_fillable_calculator.ts | 9 | ||||
-rw-r--r-- | packages/0x.js/test/order_state_watcher_test.ts | 2 |
2 files changed, 5 insertions, 6 deletions
diff --git a/packages/0x.js/src/order_watcher/remaining_fillable_calculator.ts b/packages/0x.js/src/order_watcher/remaining_fillable_calculator.ts index 30f79e786..c77d4428c 100644 --- a/packages/0x.js/src/order_watcher/remaining_fillable_calculator.ts +++ b/packages/0x.js/src/order_watcher/remaining_fillable_calculator.ts @@ -51,16 +51,15 @@ export class RemainingFillableCalculator { } } private calculatePartiallyFillableMakerTokenAmount(): BigNumber { - // The number of times the maker can fill the order, if each fill only required the transfer of a single - // baseUnit of fee tokens. // Given an order for 200 wei for 2 ZRXwei fee, find 100 wei for 1 ZRXwei. Order ratio is then 100:1 const orderToFeeRatio = this.signedOrder.makerTokenAmount.dividedBy(this.signedOrder.makerFee); - // Maximum number of times the Maker can fill the order, given the fees + // The number of times the maker can fill the order, if each fill only required the transfer of a single + // baseUnit of fee tokens. // Given 2 ZRXwei, the maximum amount of times Maker can fill this order, in terms of fees, is 2 const fillableTimesInFeeTokenBaseUnits = BigNumber.min(this.transferrableMakerFeeTokenAmount, this.remainingMakerFeeAmount); - // Maximum number of times the Maker can fill the order, given the Maker Token Balance - // Assuming a balance of 150 wei, maker can fill this order 1 time. + // The number of times the Maker can fill the order, given the Maker Token Balance + // Assuming a balance of 150 wei, and an orderToFeeRatio of 100:1, maker can fill this order 1 time. let fillableTimesInMakerTokenUnits = this.transferrableMakerTokenAmount.dividedBy(orderToFeeRatio); if (this.isMakerTokenZRX) { // If ZRX is the maker token, the Fee and the Maker amount need to be removed from the same pool; diff --git a/packages/0x.js/test/order_state_watcher_test.ts b/packages/0x.js/test/order_state_watcher_test.ts index ab6174f71..4c19a3dc2 100644 --- a/packages/0x.js/test/order_state_watcher_test.ts +++ b/packages/0x.js/test/order_state_watcher_test.ts @@ -25,7 +25,7 @@ import {FillScenarios} from './utils/fill_scenarios'; import {DoneCallback} from '../src/types'; import {BlockchainLifecycle} from './utils/blockchain_lifecycle'; import {reportCallbackErrors} from './utils/report_callback_errors'; -import { constants as constants } from './utils/constants'; +import {constants as constants} from './utils/constants'; const TIMEOUT_MS = 150; |