aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorfragosti <francesco.agosti93@gmail.com>2018-10-30 06:48:45 +0800
committerfragosti <francesco.agosti93@gmail.com>2018-10-30 06:48:45 +0800
commit10e6c3cd906c6625ed0c9ae98d20ef9f32676cbe (patch)
tree0f2d0aaf11510fafa84c6bf5821e6c3623a1c07d
parentac72df41883af261b8661cb5f52c01567a6aad62 (diff)
downloaddexon-sol-tools-10e6c3cd906c6625ed0c9ae98d20ef9f32676cbe.tar.gz
dexon-sol-tools-10e6c3cd906c6625ed0c9ae98d20ef9f32676cbe.tar.zst
dexon-sol-tools-10e6c3cd906c6625ed0c9ae98d20ef9f32676cbe.zip
feat: remove isValidProvided orders validation
-rw-r--r--packages/asset-buyer/src/asset_buyer.ts1
-rw-r--r--packages/asset-buyer/src/utils/assert.ts15
-rw-r--r--packages/instant/public/index.html52
3 files changed, 35 insertions, 33 deletions
diff --git a/packages/asset-buyer/src/asset_buyer.ts b/packages/asset-buyer/src/asset_buyer.ts
index 34e2d9639..ed52f2d9d 100644
--- a/packages/asset-buyer/src/asset_buyer.ts
+++ b/packages/asset-buyer/src/asset_buyer.ts
@@ -56,7 +56,6 @@ export class AssetBuyer {
): AssetBuyer {
assert.isWeb3Provider('provider', provider);
assert.doesConformToSchema('orders', orders, schemas.signedOrdersSchema);
- assert.areValidProvidedOrders('orders', orders);
assert.assert(orders.length !== 0, `Expected orders to contain at least one order`);
const orderProvider = new BasicOrderProvider(orders);
const assetBuyer = new AssetBuyer(provider, orderProvider, options);
diff --git a/packages/asset-buyer/src/utils/assert.ts b/packages/asset-buyer/src/utils/assert.ts
index e8cb7f763..7e4eb7175 100644
--- a/packages/asset-buyer/src/utils/assert.ts
+++ b/packages/asset-buyer/src/utils/assert.ts
@@ -31,21 +31,6 @@ export const assert = {
sharedAssert.isHexString(`${variableName}.takerAssetData`, orderFetcherRequest.takerAssetData);
sharedAssert.isNumber(`${variableName}.networkId`, orderFetcherRequest.networkId);
},
- areValidProvidedOrders(variableName: string, orders: SignedOrder[]): void {
- if (orders.length === 0) {
- return;
- }
- const makerAssetData = orders[0].makerAssetData;
- const takerAssetData = orders[0].takerAssetData;
- const filteredOrders = _.filter(
- orders,
- order => order.makerAssetData === makerAssetData && order.takerAssetData === takerAssetData,
- );
- sharedAssert.assert(
- orders.length === filteredOrders.length,
- `Expected all orders in ${variableName} to have the same makerAssetData and takerAssetData.`,
- );
- },
isValidPercentage(variableName: string, percentage: number): void {
assert.isNumber(variableName, percentage);
assert.assert(
diff --git a/packages/instant/public/index.html b/packages/instant/public/index.html
index 68d6c69c4..dc172e6e4 100644
--- a/packages/instant/public/index.html
+++ b/packages/instant/public/index.html
@@ -34,22 +34,40 @@
EXPONENTIAL_AT: 1000,
DECIMAL_PLACES: 78,
});
- const providedOrder = {
- "senderAddress": "0x0000000000000000000000000000000000000000",
- "makerAddress": "0x34a745008a643eebc58920eaa29fb1165b4a288e",
- "takerAddress": "0x0000000000000000000000000000000000000000",
- "makerFee": new BigNumber("0"),
- "takerFee": new BigNumber("0"),
- "makerAssetAmount": new BigNumber("400000000000000000000"),
- "takerAssetAmount": new BigNumber("40000000000000000000"),
- "makerAssetData": "0xf47261b00000000000000000000000008cb3971b8eb709c14616bd556ff6683019e90d9c",
- "takerAssetData": "0xf47261b0000000000000000000000000d0a1e359811322d97991e03f863a0c30c2cf029c",
- "expirationTimeSeconds": new BigNumber("1543046400"),
- "feeRecipientAddress": "0x0000000000000000000000000000000000000000",
- "salt": new BigNumber("47929252863126413473766089649682650973189811771354566206928245255479607883031"),
- "signature": "0x1c0bf8ba709ceb5b32e6b0b5a8bb7f07e9d19aba88d8530715f8a298d12188e3862fcc0a30ddfad4062b30459f2859323c064052f12cc687466c457934b9419a1b03",
- "exchangeAddress": "0x35dd2932454449b14cee11a94d3674a936d5d7b2"
- }
+ const providedOrders = [
+ {
+ "senderAddress": "0x0000000000000000000000000000000000000000",
+ "makerAddress": "0x34a745008a643eebc58920eaa29fb1165b4a288e",
+ "takerAddress": "0x0000000000000000000000000000000000000000",
+ "makerFee": new BigNumber("0"),
+ "takerFee": new BigNumber("0"),
+ "makerAssetAmount": new BigNumber("400000000000000000000"),
+ "takerAssetAmount": new BigNumber("40000000000000000000"),
+ "makerAssetData": "0xf47261b00000000000000000000000008cb3971b8eb709c14616bd556ff6683019e90d9c",
+ "takerAssetData": "0xf47261b0000000000000000000000000d0a1e359811322d97991e03f863a0c30c2cf029c",
+ "expirationTimeSeconds": new BigNumber("1543046400"),
+ "feeRecipientAddress": "0x0000000000000000000000000000000000000000",
+ "salt": new BigNumber("47929252863126413473766089649682650973189811771354566206928245255479607883031"),
+ "signature": "0x1c0bf8ba709ceb5b32e6b0b5a8bb7f07e9d19aba88d8530715f8a298d12188e3862fcc0a30ddfad4062b30459f2859323c064052f12cc687466c457934b9419a1b03",
+ "exchangeAddress": "0x35dd2932454449b14cee11a94d3674a936d5d7b2"
+ },
+ {
+ "senderAddress": "0x0000000000000000000000000000000000000000",
+ "makerAddress": "0x34a745008a643eebc58920eaa29fb1165b4a288e",
+ "takerAddress": "0x0000000000000000000000000000000000000000",
+ "makerFee": new BigNumber("0"),
+ "takerFee": new BigNumber("0"),
+ "makerAssetAmount": new BigNumber("300000000000000000000"),
+ "takerAssetAmount": new BigNumber("31000000000000000000"),
+ "makerAssetData": "0xf47261b00000000000000000000000002002d3812f58e35f0ea1ffbf80a75a38c32175fa",
+ "takerAssetData": "0xf47261b0000000000000000000000000d0a1e359811322d97991e03f863a0c30c2cf029c",
+ "expirationTimeSeconds": new BigNumber("2524636800"),
+ "feeRecipientAddress": "0x0000000000000000000000000000000000000000",
+ "salt": new BigNumber("64592004666704945574675477805199411288137454783320798602050822322450089238268"),
+ "signature": "0x1c13cacddca8d7d8248e91f412377e68f8f1f9891a59a6c1b2eea9f7b33558c30c4fb86a448e08ab7def40a28fb3a3062dcb33bb3c45302447fce5c4288b7c7f5b03",
+ "exchangeAddress": "0x35dd2932454449b14cee11a94d3674a936d5d7b2"
+ }
+ ];
const queryParams = new Uri(window.location.search);
const renderOptionsDefaults = {
liquiditySource: 'https://api.radarrelay.com/0x/v2/',
@@ -57,7 +75,7 @@
}
const liquiditySourceOverride = queryParams.getQueryParamValue('liquiditySource');
const renderOptionsOverrides = {
- liquiditySource: liquiditySourceOverride === 'provided' ? [providedOrder] : liquiditySourceOverride,
+ liquiditySource: liquiditySourceOverride === 'provided' ? providedOrders : liquiditySourceOverride,
assetData: queryParams.getQueryParamValue('assetData'),
networkId: +queryParams.getQueryParamValue('networkId') || undefined,
defaultAssetBuyAmount: +queryParams.getQueryParamValue('defaultAssetBuyAmount') || undefined,