crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

@0x/asset-swapper: Fix fee schedule not being scaled by gas price.

Browse Source
This commit is contained in:
Lawrence Forman 2020-03-13 00:07:48 -04:00
parent baf6372179
commit 5ed919df79
2 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
packages/asset-swapper/CHANGELOG.json
View File

@ -25,7 +25,11 @@
{
"note": "Add `maxFallbackSlippage` option.",
"pr": 2513
}
},
{
"note": "Fix fee schedule not being scaled by gas price.",
"pr": 2522
},
]
},
{

6
packages/asset-swapper/src/utils/swap_quote_calculator.ts
View File

@ -20,7 +20,7 @@ import {
import { fillableAmountsUtils } from './fillable_amounts_utils';
import { MarketOperationUtils } from './market_operation_utils';
import { convertNativeOrderToFullyFillableOptimizedOrders } from './market_operation_utils/orders';
import { ERC20BridgeSource, OptimizedMarketOrder } from './market_operation_utils/types';
import { ERC20BridgeSource, GetMarketOrdersOpts, OptimizedMarketOrder } from './market_operation_utils/types';
import { ProtocolFeeUtils } from './protocol_fee_utils';
import {
isOrderTakerFeePayableWithMakerAsset,
@ -132,9 +132,9 @@ export class SwapQuoteCalculator {
{
// Scale fees by gas price.
const _opts = {
const _opts: GetMarketOrdersOpts = {
...opts,
fees: _.mapValues(opts.feeSchedule, v => v.times(gasPrice)),
feeSchedule: _.mapValues(opts.feeSchedule, v => v.times(gasPrice)),
};
const firstOrderMakerAssetData = !!prunedOrders[0]