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

Collapse on-chain sources into DexForwarderBridge (#2560)

Browse Source 
* Collapse on-chain sources into DexForwarderBridge

* Fix tests. CHANGELOG
This commit is contained in:
Jacob Evans 2020-04-23 13:12:46 +10:00 committed by GitHub
parent 07acc9529e
commit cac6f5234f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 6 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -49,6 +49,10 @@
{
"note": "Add support for indicative (non-committal) quotes via RFQ-T",
"pr": 2555
},
{
"note": "Collapse `LiquidityProvider` into `DexForwarderBridge`",
"pr": 2560
}
]
},

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

@ -153,19 +153,10 @@ export function createOrdersFromPath(path: Fill[], opts: CreateOrderFromPathOpts
++i;
continue;
}
// Liquidity Provider must be called by ERC20BridgeProxy
if (collapsedPath[i].source === ERC20BridgeSource.LiquidityProvider) {
orders.push(createBridgeOrder(collapsedPath[i], opts));
++i;
continue;
}
// If there are contiguous bridge orders, we can batch them together.
const contiguousBridgeFills = [collapsedPath[i]];
for (let j = i + 1; j < collapsedPath.length; ++j) {
if (
collapsedPath[j].source === ERC20BridgeSource.Native ||
collapsedPath[j].source === ERC20BridgeSource.LiquidityProvider
) {
if (collapsedPath[j].source === ERC20BridgeSource.Native) {
break;
}
contiguousBridgeFills.push(collapsedPath[j]);

2
packages/asset-swapper/test/market_operation_utils_test.ts
View File

@ -659,7 +659,7 @@ describe('MarketOperationUtils tests', () => {
}),
],
Web3Wrapper.toBaseUnitAmount(10, 18),
{ excludedSources: SELL_SOURCES, numSamples: 4, bridgeSlippage: 0 },
{ excludedSources: SELL_SOURCES, numSamples: 4, bridgeSlippage: 0, shouldBatchBridgeOrders: false },
);
expect(result.length).to.eql(1);
expect(result[0].makerAddress).to.eql(liquidityProviderAddress);