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

fix: UniswapV2 path order (#2601)

Browse Source
This commit is contained in:
Jacob Evans
2020-06-12 13:26:56 +10:00
committed by GitHub
parent e936c7c507
commit 81d4803b4d
2 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -85,6 +85,10 @@
{
"note": "Add support for MultiBridge",
"pr": 2593
},
{
"note": "Fix Uniswap V2 path ordering",
"pr": 2601
}
]
},

4
packages/asset-swapper/src/utils/market_operation_utils/orders.ts
View File

@@ -234,7 +234,7 @@ function createBridgeOrder(fill: CollapsedFill, opts: CreateOrderFromPathOpts):
makerAssetData = assetDataUtils.encodeERC20BridgeAssetData(
makerToken,
bridgeAddress,
createUniswapV2BridgeData([makerToken, takerToken]),
createUniswapV2BridgeData([takerToken, makerToken]),
);
break;
case ERC20BridgeSource.UniswapV2Eth:
@@ -246,7 +246,7 @@ function createBridgeOrder(fill: CollapsedFill, opts: CreateOrderFromPathOpts):
makerAssetData = assetDataUtils.encodeERC20BridgeAssetData(
makerToken,
bridgeAddress,
createUniswapV2BridgeData([makerToken, opts.contractAddresses.etherToken, takerToken]),
createUniswapV2BridgeData([takerToken, opts.contractAddresses.etherToken, makerToken]),
);
break;
case ERC20BridgeSource.MultiBridge: