Make it easier to use validateOrderFillableOrThrowAsync #2096

xianny · 2019-08-24T00:21:27Z

Description

The handwritten ExchangeWrapper in @0x/contract-wrappers < 11.0.0 previously exposed a method validateOrderFillableOrThrowAsync. This method simulated filling an order with or without a taker, and covered this specific edge case where an order trades ZRX and has ZRX fees.

The method validateOrderFillableOrThrowAsync is still exposed in OrderValidationUtils but without the boilerplate that was previously implemented in ExchangeWrapper.

This PR implements a new method simpleValidateOrderFillableOrThrowAsync in OrderValidationUtils that reinstates the interface of the ExchangeWrapper method and performs the same simulation.

Context: This is newly urgent because the signatures fix is forcing devs to upgrade sooner rather than later.

I have one additional question re: the upgrade guide. All of the order validation functions got moved to the order-utils package. It would be helpful to see an example of how to use the order validation utils in the order-utils package in particular how to set up the exchangeTradeEmulator.
#2041 (comment)

Types of changes

New feature (non-breaking change which adds functionality)

Checklist:

Prefix PR title with [WIP] if necessary.
Add tests to cover changes as needed.
Update documentation as needed.
Add new entries to the relevant CHANGELOG.jsons.

xianny · 2019-08-24T00:22:41Z

packages/order-utils/src/asset_balance_and_proxy_allowance_fetcher.ts

@@ -0,0 +1,101 @@
+import { ERC20ProxyContract, ERC20TokenContract, ERC721ProxyContract, ERC721TokenContract } from '@0x/abi-gen-wrappers';


This file was retrieved from https://github.com/0xProject/0x-monorepo/pull/2037/files#diff-7db46b4c4a4907f01f44e8f0018b4219

xianny · 2019-08-24T00:28:48Z

packages/order-utils/src/order_validation_utils.ts

+            const orderRelevantState = await orderStateUtils.getOpenOrderRelevantStateAsync(signedOrder);
+            fillableTakerAssetAmount = orderRelevantState.remainingFillableTakerAssetAmount;
+        }
+


above boilerplate retrieved from https://github.com/0xProject/0x-monorepo/pull/2037/files#diff-540ab868df58adcdec38223c40efc2e1L1160-L1188

xianny · 2019-08-24T00:29:21Z

packages/order-utils/src/order_validation_utils.ts

+        );
+        const balanceAllowanceStore = new BalanceAndProxyAllowanceLazyStore(balanceAllowanceFetcher);
+        const exchangeTradeSimulator = new ExchangeTransferSimulator(balanceAllowanceStore);
+


above boilerplate retrieved from https://github.com/0xProject/0x-monorepo/pull/2037/files#diff-540ab868df58adcdec38223c40efc2e1L1236-L1243

buildsize · 2019-08-24T00:39:42Z

File name	Previous Size	New Size	Change
init.py	7.04 KB	[deleted]
abi_gen_dummy.ts	134.41 KB	[deleted]
lib_dummy.ts	4.84 KB	[deleted]
test_lib_dummy.ts	11.88 KB	[deleted]
environment.pickle	1.56 MB	1.56 MB	0 bytes (0%)
index.doctree	190.98 KB	190.98 KB	0 bytes (0%)
.buildinfo	230 bytes	230 bytes	0 bytes (0%)
genindex.html	5.6 KB	5.6 KB	0 bytes (0%)
index.html	2.52 KB	2.52 KB	0 bytes (0%)
objects.inv	375 bytes	375 bytes	0 bytes (0%)
py-modindex.html	3.07 KB	3.07 KB	0 bytes (0%)
search.html	2.84 KB	2.84 KB	0 bytes (0%)
searchindex.js	5.82 KB	5.82 KB	0 bytes (0%)
index.rst.txt	415 bytes	415 bytes	0 bytes (0%)
alabaster.css	10.92 KB	10.92 KB	0 bytes (0%)
basic.css	11.89 KB	11.89 KB	0 bytes (0%)
custom.css	42 bytes	42 bytes	0 bytes (0%)
doctools.js	9.05 KB	9.05 KB	0 bytes (0%)
documentation_options.js	303 bytes	303 bytes	0 bytes (0%)
file.png	286 bytes	286 bytes	0 bytes (0%)
jquery-[version].js	273.79 KB	273.79 KB	0 bytes (0%)
jquery.js	86.08 KB	86.08 KB	0 bytes (0%)
language_data.js	10.59 KB	10.59 KB	0 bytes (0%)
minus.png	90 bytes	90 bytes	0 bytes (0%)
plus.png	90 bytes	90 bytes	0 bytes (0%)
pygments.css	4.69 KB	4.69 KB	0 bytes (0%)
searchtools.js	15.61 KB	15.61 KB	0 bytes (0%)
underscore-[version].js	34.34 KB	34.34 KB	0 bytes (0%)
underscore.js	11.86 KB	11.86 KB	0 bytes (0%)
contract_addresses.html	17.89 KB	17.89 KB	0 bytes (0%)
contract_artifacts.html	8.24 KB	8.24 KB	0 bytes (0%)
_bootstrap.html	142.73 KB	142.73 KB	0 bytes (0%)
json_schemas.html	12.43 KB	12.43 KB	0 bytes (0%)
order_utils.html	44.83 KB	44.83 KB	0 bytes (0%)
erc20_token.html	88.14 KB	88.14 KB	0 bytes (0%)
exchange.html	506.16 KB	506.16 KB	0 bytes (0%)
tx_params.html	8.83 KB	8.83 KB	0 bytes (0%)
local_message_signer.html	15.07 KB	15.07 KB	0 bytes (0%)
asset_data_utils.html	22.65 KB	22.65 KB	0 bytes (0%)
default_api.html	118.48 KB	118.48 KB	0 bytes (0%)
asset_proxy_owner.html	341.77 KB	341.77 KB	0 bytes (0%)
coordinator.html	127.47 KB	127.47 KB	0 bytes (0%)
coordinator_registry.html	36.15 KB	36.15 KB	0 bytes (0%)
dutch_auction.html	54.84 KB	54.84 KB	0 bytes (0%)
erc20_proxy.html	105.63 KB	105.63 KB	0 bytes (0%)
erc721_proxy.html	105.72 KB	105.72 KB	0 bytes (0%)
erc721_token.html	141.96 KB	141.96 KB	0 bytes (0%)
eth_balance_checker.html	21.03 KB	21.03 KB	0 bytes (0%)
forwarder.html	107.42 KB	107.42 KB	0 bytes (0%)
i_asset_proxy.html	89.89 KB	89.89 KB	0 bytes (0%)
i_validator.html	26.24 KB	26.24 KB	0 bytes (0%)
i_wallet.html	23.61 KB	23.61 KB	0 bytes (0%)
multi_asset_proxy.html	141.39 KB	141.39 KB	0 bytes (0%)
order_validator.html	115.34 KB	115.34 KB	0 bytes (0%)
weth9.html	129.04 KB	129.04 KB	0 bytes (0%)
zrx_token.html	107.34 KB	107.34 KB	0 bytes (0%)
dev_utils.html	339.53 KB	339.53 KB	0 bytes (0%)

coveralls · 2019-08-24T00:44:18Z

Coverage remained the same at 77.532% when pulling eae90e9 on fix/order-utils-validation into d912175 on development.

dekz · 2019-08-24T01:24:34Z

packages/order-utils/src/order_validation_utils.ts

+     * To check if the order is fillable for a non-zero amount, set `validateRemainingOrderAmountIsFillable` to false.)
+     */
+    public async simpleValidateOrderFillableOrThrowAsync(
+        erc20ProxyAddress: string,


We should be able to detect these either via an Exchange lookup or from the Exchange address -> contractAddresses -> erc20ProxyAddress. The exchangeAddress is also on the signedOrder so is redundant.

Thoughts on reducing this to just provider, signedOrder, opts and looking up internally?

dekz · 2019-08-24T01:28:53Z

packages/order-utils/src/order_validation_utils.ts

+     * @param fillTakerAssetAmount Amount we'd like to fill the order for
+     * @param takerAddress The taker of the order
+     */
+    public async simpleValidateFillOrderThrowIfInvalidAsync(


Wasn't this being replaced with the callAsync to fillOrder? Don't think we need this.

dekz · 2019-08-24T01:44:11Z

packages/order-utils/src/order_validation_utils.ts

+            provider,
+            BlockParamLiteral.Latest,
+        );
+        const balanceAllowanceStore = new BalanceAndProxyAllowanceLazyStore(balanceAllowanceFetcher);


ExchangeTransferSimulator is useful for basically one reason, If the user is trading XYZ for ZRX and the order has fees, then the ExchangeTransferSimulator can handle this case (as it simulates the transfer of ZRX to maker prior to fee payment). This is an annoying case to handle for and is often overlooked, resulting in invalid orders for completely valid orders if only Balances and Allowances are validated.

I do not believe DevUtils currently handles this edge case.

Extrapolating to V3. I'm selling a Kitty for 100 WETH, there is a 1 WETH fee. I have 0 WETH. This order is valid as there will be funds for fees during settlement but not prior to settlement.

dekz · 2019-08-24T01:49:27Z

packages/order-utils/src/asset_balance_and_proxy_allowance_fetcher.ts

+        this._provider = provider;
+    }
+    public async getBalanceAsync(assetData: string, userAddress: string): Promise<BigNumber> {
+        const decodedAssetData = assetDataUtils.decodeAssetDataOrThrow(assetData);


We should be able to refactor this to use DevUtils.

devUtils.getBalance(owner, assetData)

Which handles ERC20, ERC721, ERC1155 and MAP.

dekz · 2019-08-24T01:50:02Z

packages/order-utils/src/asset_balance_and_proxy_allowance_fetcher.ts

+        }
+        return balance as BigNumber;
+    }
+    public async getProxyAllowanceAsync(assetData: string, userAddress: string): Promise<BigNumber> {


We should be able to refactor this to use DevUtils.

devUtils.getAssetProxyAllowance.callAsync(owner, assetData)

Which handles ERC20, ERC721, ERC1155 and MAP.

xianny · 2019-09-05T16:22:30Z

packages/migrations/CHANGELOG.json

+        "version": "4.3.2",
+        "changes": [
+            {
+                "note": "Removed dependency on @0x/order-utils",


I had to introduce a dependency on @0x/migrations to @0x/order-utils for tests (which were retrieved from the old exchange wrapper tests). I've run into similar issues with tests introducing circular dependencies for other packages. Would like to discuss the possibility of developing an integration test suite for any tests that run with ganache and the standard 0x migration. Two benefits: 1) potentially speed up our testing time, by doing one migration + snapshots instead of separate migrations in numerous unit test suites, and 2) reduce package size/unnecessary dependency trees in the published packages.

Since we intend to remove these methods in a couple months, this seems like the easiest solution for now to unblock this task.

xianny · 2019-09-05T18:18:06Z

packages/order-utils/test/exchange_transfer_simulator_test.ts

@@ -42,6 +42,7 @@ describe('ExchangeTransferSimulator', async () => {
            from: devConstants.TESTRPC_FIRST_ADDRESS,
        };

+        await blockchainLifecycle.startAsync();


This was affecting the new unit tests in order_validation_utils_test.ts that are also blockchain dependent.

albrow · 2019-09-05T20:45:24Z

@fabioberger should probably review also since he has more context for some of these changes. I was mostly focusing on the contract-addresses package and related changes.

fabioberger · 2019-09-06T09:07:07Z

packages/contract-addresses/src/index.ts

+export function getNetworkIdByExchangeAddressOrThrow(exchangeAddress: string): NetworkId {
+    for (const networkId of Object.keys(networkToAddresses)) {
+        if (networkToAddresses[networkId as any].exchange === exchangeAddress) {
+            return networkId as any;


Why do we need to cast to any here? Why not cast to NetworkId?

Object.keys casts everything to string. I got a string has no overlap with NetworkId error. For clarity we can cast networkId as any as NetworkId instead

I'd prefer that. Ideally we keep things as any for the least amount of time possible.

packages/migrations/src/migration.ts

* Added longer timeout for algolia admin operations. Added an option to apply hidden and custom labels to filters from algolia. Added sorting guides (alphabetically) and tools (by community maintenance and alphabetically). * Fixed some types in guides * Updated webpack, webpack-dev-server, webpack-bundle-analyzer, webpack-cli versions * Updated / cleaned up babel and mdx packages * Updated terser webpack plugin; Removed uglify plugin (unused). Added parallel option for terser. Added target browsers and cache directory option for babel-loader * Webpack changes: using eval-source-map for development, added parallel option to Terser plugin, added cache loader and caching directory to mdx / babel loaders. * Removed unusued pkg: body-scroll-lock * Removed unused jsonschema / @types/jsonschema packages * Removed unused less node package, adjusted less loader rules to match other loaders in webpack config * Removed unusued node-sass package. Left sass-loader for webpack intact. * Removed unused package react-lazyload * Removed unused package react-typist * Removed react-tap-event types as it's not used anymore. Reinstalled node-sass as a dev dependency as it's needed by sass-loader * Moves @0x/types and @0x/typescript-typings to devDependencies * Replaced react loadable (only used in Icon) with the native react lazy / suspense. Removed loadable nad its types from package.json * Removed unused make-promises-safe package * Moved animations to public and compressed files * Added opening step links in new tab if the url is valid * Moved @0x/types back to dependencies. * Added animation wrapper with positioning props. Updated animations for core concepts. * Replaced images for core concepts. Fixed a bug causing duplicate props on image and its container in mdx * Remove legacy FAQ * Update yarn.lock * Add MD docs diff test to CI * Remove unused dep * Update docs to latest format * Update yarn.lock * Update ts-doc-gen with fix for multi-link lines * Final docs gen * Fix home route * Update yarn.lock * Add Golang Mesh RPC package and update versions * Update the tools MDX files from authoritative S3 before build and dev server start * Gitignore the tool MDX files in website repo * Redirect old docs pages and wiki to new doc pages * Move out Algolia indexing command-line tool out of website and remove need to expose admin key * Add updating tools from S3 and indexing content on Algolia to deployment commands * Add note about admin API env var needed for indexing docs * Remove scripts gitignore since we don't have monorepo scripts in each package anymore * Segregate indexes by environment * Fixed 404 display for docs pages * Removed webpack bundle analyzer * Fix search and indexing * Use `yarn` because otherwise the flags don't get passed down properly * Improve indexing comment * Remove legacy doc pages * Fix linter errors * Fix clean command to remove bundles recursively * Revert react and react-dom versions in Instant * Remove old doc md files * Fix links * Fix prettier * Combine @0x/utils exports * Update doc links * Remove unused components * Fix TS error * Add missing new line * Update Guides * Ignore auto-generated MDX in website repo * Prettier fix * Add all guides to the sitemap.txt * Add MMing guide * Fix Get Started links * Add 0x Extensions guide * Replace all links to wiki with links to guides section * Fix all /docs/{pkg} links to /docs/tools/{pkg} * Remove deployer endpoint that is no longer used * Fix links in Core Concepts * Fixed regex to not remove hashes on links beyond the initial one * Fixed huge width on instant screenshot * Cleaned up core concept illustrations * Started updating tool titles * Fixed sectionizing headings in docs: Nested sections do not get wrapped in parent sections anymore; only sectionizing headings that need to be sectionized (i.e. that appear in the table of contents) * Stylstic fix for the new way we sectionize headings / table of contents * Change CoordinatorWrapper constructor to take a provider * Added autolinking headings in mdx docs * Refactor doc generation script to only generate MD docs and to have them reflect what is exported in their respective index.ts files * Update ts-doc-gen * Pass in provider, not Web3Wrapper * Add back logic to prune out private methods * Write algolia_metadata to utils dir * Remove duplicate algolia_meta * Add more packages to those we generate docs for and remove unused flags * Update reference.mdx files * Hide tools from homepage that have isHidden flag * Link 0x.js to it's reference page * Update tool naming to be more human readable * Remove order-watcher reference doc * Add blurb about on-chain API Explorer * Add "On-chain" prefix to section name * Remove committed reference file * Sorting guides by difficulty and alphabetically * Pushed sorting guides titles case-insensitive * Sorting tools by difficulty and alphabetically. Removed icons from featured tools. Extracted common difficultyOrder constant * Update ts-doc-gen so that it properly converts module relative links to anchor links * Update doc references * Remove "Relayers" topic/tag * Prettier fix * Remove hr so we don't have a bunch in a row * Cleaned up CLI tag and added link to tools on API page * Cleaned up some Guide titles * Added bug bounty program guide * Fixed a bug in algolia_meta * Thrid level of nesting / temporary webpack config * Open help link in new tab * Link Python tools to doc pages not PyPi * Fix links in guides * Remove unused variables * Revert webpack config * Extracted common styles for levels of nesting deeper than 1 in the mdx table of contents * Smoother animation on active table of contents state change * Autolink headings in mdx only up to h3. Added heading sizes (in px) for h4,h5,h6 in mdx headings * Fix prettier * Adjusted font-size for paragraph, ordered and unordered lists. Fixed line-heights and colors too. * Update ts-doc-gen and the reference docs * Replace 0xproject.com urls with 0x.org * Update README links * Fix header sizes to display optimally in sidebar * Fixed an issue with other nodes than text being parsed incorrectly in the table of contents * Fixed styles for table of contents * Use a different babel plugin that works in a production webpack build * Unbroke build process / generating toc * Changed line-height to 1.6 for paragraph * Adjust scrollbar / toc height to available space * Fixed some types * Remove unnecessary link * Fix casing on TypeScript * Fix casing of PyPI * Remove doc gen from abi-gen-wrappers since it'll be tested via contract-wrappers * Fix MD docs and make sure generated contract wrappers are being rendered * Remove generated docs in clean * Correctly copying code samples * Several additional fixes to the reference MDX docs * Changed tools to Tools & Libraries - top nav left as just tools * Changed references from Asset Buyer to Asset Swapper * Changed trader keyword to trading * Nevermind, changed back to Trader * Sneaking in updated product dropdown menu titles based on Will's feedback * Removed the word wiki from resources dropdown * Open legal guide in same window * Removed the isCommunity filter / custom filter labels functionality (only used for that before) * Update ts-doc-gen version to one that fixes issues with Object Literals and removes Hierarchy sections and Module headers * update yarn.lock * Remove unused code * Remove unused import * Remove unused imports * Remove unused import * Fix prettier * Update outdating dep version * Created a color scheme for code snippets. Fixed language not being passed to some code blocks in two guides. * Update abi-gen-wrappers dep version * Update Algolia to production account * Fix prettier * Fix prettier in algolia_meta * Remove packages/dev-tools-pages folder and update yarn.lock * Remove other random references to dev-tools-pages * Fix website * Changed the order of indices in search suggestions * Removed a comment * Added scrollbar size adjustment on resize / scroll * Rendering sidebar on mobile without any event listeners / taxing calculations / changes in height * Added a performance comment * Simplified table of contents * Update ganache-cli to later version * Remove bundle dirs too * Added keeping active link in view when scrolling content in the docs * Increase available memory for webpack * Update contract addresses * Update market-making guide to mention Mesh as an alternative to SRA * Update Web3 Provider Explained guide * Re-wrote Ganache Setup guide into a Setting up a local 0x testnet guide * Export more of abi-gen-wrappers from contract-wrappers * Final update of reference docs * Add contract-wrappers MD docs and update it's index.ts so that no types are missing * Fix prettier * Increase the memory allocation for building the website * Remove use of remove type * Check if link is a valid url in mdx inline link: if so, open in a new tab, if not, same tab. * Print out which snapshot version is being downloaded * Updated CHANGELOGS & MD docs * Publish - @0x/contracts-asset-proxy@2.2.7 - @0x/contracts-coordinator@2.0.12 - @0x/contracts-dev-utils@0.0.9 - @0x/contracts-erc1155@1.1.14 - @0x/contracts-erc20@2.2.13 - @0x/contracts-erc721@2.1.14 - @0x/contracts-exchange-forwarder@3.0.11 - @0x/contracts-exchange-libs@3.0.7 - @0x/contracts-exchange@2.1.13 - @0x/contracts-extensions@4.0.7 - @0x/contracts-multisig@3.1.13 - @0x/contracts-test-utils@3.1.15 - @0x/contracts-utils@3.2.3 - 0x.js@7.0.1 - @0x/abi-gen-wrappers@5.3.1 - @0x/abi-gen@4.2.0 - @0x/assert@2.1.5 - @0x/asset-buyer@6.1.13 - @0x/asset-swapper@1.0.3 - @0x/base-contract@5.3.3 - @0x/connect@5.0.18 - @0x/contract-artifacts@2.2.1 - @0x/contract-wrappers@12.0.0 - @0x/contracts-gen@1.0.14 - @0x/dev-tools-pages@0.0.30 - @0x/dev-utils@2.3.2 - ethereum-types@2.1.5 - @0x/fill-scenarios@3.0.18 - @0x/instant@1.0.30 - @0x/json-schemas@4.0.1 - @0x/migrations@4.3.1 - @0x/monorepo-scripts@1.0.36 - @0x/order-utils@8.3.1 - @0x/sol-compiler@3.1.14 - @0x/sol-coverage@3.0.11 - @0x/sol-doc@2.0.18 - @0x/sol-profiler@3.1.13 - @0x/sol-resolver@2.0.10 - @0x/sol-trace@2.0.19 - @0x/sol-tracing-utils@6.0.18 - @0x/sra-spec@2.0.16 - @0x/subproviders@5.0.3 - @0x/testnet-faucets@1.0.86 - @0x/types@2.4.2 - @0x/typescript-typings@4.2.5 - @0x/utils@4.5.1 - @0x/web3-wrapper@6.0.12 - @0x/website@0.0.89 * Remove the website from the monorepo * Update yarn.lock * Remove website files that were accidentally merged back in * Use local EVM for pure functions (#2108) * add local evm for pure functions * increase container size to >=8gb for static-tests * increase max bundle size because of ethereumjs-vm dependency * add declarations for ethereumjs-vm in @0x/typescript-typings * Make `OrdersChannelSubscriptionOpts` conform to SRAv2 * Make it easier to use validateOrderFillableOrThrowAsync (#2096) * make it easier to use validateOrderFillableOrThrowAsync * add unit tests, use DevUtils * remove dependency on @0x/order-utils from @0x/migrations * add devUtils to contract-wrappers (#2146) * Delete docs in S3 bucket before uploading latest version so we only ever have one version uploaded at a time * Update to new ts-doc-gen version that fixes links in "Inherited from" lines * try to fix instant build * Remove redoc-cli as a dep from sra-spec * Smooth out some last incompatible types * Fix prettier * Add terser-webpack-plugin to 0x.js deps * @0x/orderbook * AssetSwapper to use @0x/orderbook * Remove unused imported type * Prevent double websocket connections * Fix build error * fixed consumer inconsistency bug * linted + added convenience functionality * minor changes/typo fixes * updated websocket * updated for prettier * Fix versions and exported types * move log tests to @0x/abi-gen; delete exchange_wrapper_test (#2160) * Remove unused exported types * Fix exports from asset-swapper * Update exported types for docs * Update mesh client dep * Move perPage and 18 precision lower to re-use * fix ABI encoding/decoding functions (#2165) * Fix typo at contract import (#2102) * Updated CHANGELOGS & MD docs * Publish - @0x/contracts-asset-proxy@2.2.8 - @0x/contracts-coordinator@2.0.13 - @0x/contracts-dev-utils@0.0.10 - @0x/contracts-erc1155@1.1.15 - @0x/contracts-erc20@2.2.14 - @0x/contracts-erc721@2.1.15 - @0x/contracts-exchange-forwarder@3.0.12 - @0x/contracts-exchange-libs@3.0.8 - @0x/contracts-exchange@2.1.14 - @0x/contracts-extensions@4.0.8 - @0x/contracts-multisig@3.1.14 - @0x/contracts-test-utils@3.1.16 - @0x/contracts-utils@3.2.4 - 0x.js@7.0.2 - @0x/abi-gen-wrappers@5.3.2 - @0x/abi-gen@4.2.1 - @0x/assert@2.1.6 - @0x/asset-buyer@6.1.14 - @0x/asset-swapper@2.0.0 - @0x/base-contract@5.4.0 - @0x/connect@5.0.19 - @0x/contract-addresses@3.2.0 - @0x/contract-artifacts@2.2.2 - @0x/contract-wrappers@12.1.0 - @0x/contracts-gen@1.0.15 - @0x/dev-utils@2.3.3 - ethereum-types@2.1.6 - @0x/fill-scenarios@3.0.19 - @0x/instant@1.0.31 - @0x/json-schemas@4.0.2 - @0x/migrations@4.3.2 - @0x/monorepo-scripts@1.0.37 - @0x/order-utils@8.4.0 - @0x/orderbook@0.0.2 - @0x/sol-compiler@3.1.15 - @0x/sol-coverage@3.0.12 - @0x/sol-doc@2.0.19 - @0x/sol-profiler@3.1.14 - @0x/sol-resolver@2.0.11 - @0x/sol-trace@2.0.20 - @0x/sol-tracing-utils@6.0.19 - @0x/sra-spec@2.0.17 - @0x/subproviders@5.0.4 - @0x/testnet-faucets@1.0.87 - @0x/types@2.4.3 - @0x/typescript-typings@4.3.0 - @0x/utils@4.5.2 - @0x/web3-wrapper@6.0.13 * fix circleci diff * update changelog

xianny commented Aug 24, 2019

View reviewed changes

xianny force-pushed the fix/order-utils-validation branch from 1119f75 to 82ef80e Compare August 24, 2019 00:32

xianny requested review from fabioberger and dekz August 24, 2019 00:33

xianny marked this pull request as ready for review August 24, 2019 00:34

xianny added this to the Multilang cleanup milestone Aug 24, 2019

dekz suggested changes Aug 24, 2019

View reviewed changes

make it easier to use validateOrderFillableOrThrowAsync

31aa55f

xianny force-pushed the fix/order-utils-validation branch from 82ef80e to 31aa55f Compare September 4, 2019 22:18

xianny requested a review from albrow as a code owner September 4, 2019 22:22

xianny force-pushed the fix/order-utils-validation branch 2 times, most recently from 7fc13f9 to 908764e Compare September 5, 2019 00:30

xianny commented Sep 5, 2019

View reviewed changes

xianny force-pushed the fix/order-utils-validation branch from 923cbee to 363826c Compare September 5, 2019 17:26

xianny added 3 commits September 5, 2019 10:54

add unit tests, use DevUtils

7eae251

remove dependency on @0x/order-utils from @0x/migrations

3bb5446

revert yarn.lock to development

6d7f786

xianny force-pushed the fix/order-utils-validation branch from 363826c to 6d7f786 Compare September 5, 2019 17:55

xianny commented Sep 5, 2019

View reviewed changes

albrow approved these changes Sep 5, 2019

View reviewed changes

fabioberger suggested changes Sep 6, 2019

View reviewed changes

xianny force-pushed the fix/order-utils-validation branch from 165e722 to 6d7f786 Compare September 7, 2019 00:14

fabioberger approved these changes Sep 9, 2019

View reviewed changes

tweak typecast and dependencies

3776e31

xianny force-pushed the fix/order-utils-validation branch from 5f6bf1a to 97b8c39 Compare September 10, 2019 17:09

xianny force-pushed the fix/order-utils-validation branch from 97b8c39 to 2af8b75 Compare September 10, 2019 17:20

Merge branch 'development' into fix/order-utils-validation

eae90e9

xianny force-pushed the fix/order-utils-validation branch from 2af8b75 to eae90e9 Compare September 10, 2019 18:23

xianny merged commit 9eba6d7 into development Sep 10, 2019

albrow mentioned this pull request Sep 18, 2019

Merge development into 3.0 #2170

Merged

4 tasks

xianny deleted the fix/order-utils-validation branch October 8, 2019 00:18

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Make it easier to use validateOrderFillableOrThrowAsync #2096

Make it easier to use validateOrderFillableOrThrowAsync #2096

xianny commented Aug 24, 2019 •

edited

Loading

xianny Aug 24, 2019

xianny Aug 24, 2019

xianny Aug 24, 2019

buildsize bot commented Aug 24, 2019 •

edited

Loading

coveralls commented Aug 24, 2019 •

edited

Loading

dekz Aug 24, 2019

dekz Aug 24, 2019

xianny Sep 4, 2019

dekz Aug 24, 2019

dekz Aug 24, 2019

dekz Aug 24, 2019

xianny Sep 5, 2019 •

edited

Loading

xianny Sep 5, 2019

albrow commented Sep 5, 2019

fabioberger Sep 6, 2019

xianny Sep 6, 2019

fabioberger Sep 9, 2019 •

edited

Loading

		@@ -0,0 +1,101 @@
		import { ERC20ProxyContract, ERC20TokenContract, ERC721ProxyContract, ERC721TokenContract } from '@0x/abi-gen-wrappers';

Make it easier to use validateOrderFillableOrThrowAsync #2096

Make it easier to use validateOrderFillableOrThrowAsync #2096

Conversation

xianny commented Aug 24, 2019 • edited Loading

Description

Types of changes

Checklist:

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

buildsize bot commented Aug 24, 2019 • edited Loading

coveralls commented Aug 24, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

xianny Sep 5, 2019 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

albrow commented Sep 5, 2019

Choose a reason for hiding this comment

Choose a reason for hiding this comment

fabioberger Sep 9, 2019 • edited Loading

Choose a reason for hiding this comment

xianny commented Aug 24, 2019 •

edited

Loading

buildsize bot commented Aug 24, 2019 •

edited

Loading

coveralls commented Aug 24, 2019 •

edited

Loading

xianny Sep 5, 2019 •

edited

Loading

fabioberger Sep 9, 2019 •

edited

Loading