Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Reverted syntax used by readBytes4 in AssetProxyOwner to be compatible with ↵ | Greg Hysen | 2018-08-30 | 1 | -5/+1 |
| | | | | Solidity v0.4.10 | ||||
* | Updated comments for ERC20 proxy to clarify how we load the token address ↵ | Greg Hysen | 2018-08-30 | 2 | -9/+60 |
| | | | | from calldata (3.18 from audit) | ||||
* | Updated readBytes4 to match spec + added unit tests. These are 3.5/3.6 from ↵ | Greg Hysen | 2018-08-30 | 2 | -3/+17 |
| | | | | audit | ||||
* | Making rounding consistent in calculateFillResults | Amir Bandeali | 2018-08-29 | 1 | -2/+2 |
| | |||||
* | safeGetPartialAmount (#1035) | Amir Bandeali | 2018-08-29 | 5 | -23/+128 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Added Test "Should transfer correct amounts when left order is fully filled and values pass isRoundingErrorCeil but fail isRoundingErrorFloor" * Added RoundingError exception to reference function for getPartialAmount * Added RoundingError exception to reference function for getPartialAmount * Added isRoundingErrorCeil to getPartialAmountCeil reference funtion * Computed new values for "Should give right maker a better buy price when correct price is not integral" that does not have a rounding error * Almost all tests for match orders are passing after adding isRoundingErrorCeil check * WIP commit: Added rounding error checks to getPartialAmount * WIP commit: Added rounding error checks to getPartialAmount * Use safe versions of getPartialAmount * Update Exchange internals tests * Run linter * Found new values for "Should transfer correct amounts when right order fill amount deviates from amount derived by `Exchange.fillOrder`" * Fixed merge conflicts * Run all tests * Cleaned up some comments on match Orders tests * Fix tests for geth | ||||
* | Add clarifying comments | Amir Bandeali | 2018-08-28 | 2 | -0/+2 |
| | |||||
* | Remove redundant mstores from fillOrderNoThrow | Amir Bandeali | 2018-08-28 | 1 | -8/+1 |
| | |||||
* | Merge pull request #986 from 0xProject/feature/contracts/assertions | Amir Bandeali | 2018-08-25 | 3 | -30/+113 |
|\ | | | | | Add more assertions to assertValidFill | ||||
| * | Merge branch 'fix/contracts/robustMatching' of github.com:0xProject/0x.js ↵ | Remco Bloemen | 2018-08-25 | 16 | -95/+591 |
| |\ | | | | | | | | | | into fix/contracts/robustMatching | ||||
| * | | Revert maker not equal taker check | Remco Bloemen | 2018-08-25 | 1 | -6/+1 |
| | | | |||||
| * | | Fix mixin api | Remco Bloemen | 2018-08-25 | 3 | -21/+36 |
| | | | |||||
| * | | Split into assertFillable and assertValidFill | Remco Bloemen | 2018-08-25 | 2 | -29/+43 |
| | | | |||||
| * | | Disallow self filling | Remco Bloemen | 2018-08-25 | 1 | -1/+7 |
| | | | |||||
| * | | Remove BUG_ from revert reasons | Remco Bloemen | 2018-08-25 | 1 | -3/+3 |
| | | | |||||
| * | | Document accetable price check | Remco Bloemen | 2018-08-25 | 1 | -2/+16 |
| | | | |||||
| * | | Fix taker overpay check | Remco Bloemen | 2018-08-25 | 1 | -1/+1 |
| | | | |||||
| * | | Check fillable early | Remco Bloemen | 2018-08-25 | 1 | -0/+6 |
| | | | |||||
| * | | Update for new assertValidFill signature | Remco Bloemen | 2018-08-25 | 2 | -5/+8 |
| | | | |||||
| * | | Add overfill and price assertion to assertValidFill | Remco Bloemen | 2018-08-25 | 2 | -2/+32 |
| | | | |||||
* | | | Added comments to order matching | Greg Hysen | 2018-08-25 | 1 | -1/+13 |
| | | | |||||
* | | | Rephrased some of the math in MixinMatchOrders to improve readability | Greg Hysen | 2018-08-25 | 1 | -3/+3 |
| | | | |||||
* | | | Added temporary @todo to MixinMatchOrders | Greg Hysen | 2018-08-25 | 1 | -4/+4 |
| | | | |||||
* | | | Wording in MixinMatchOrders | Greg Hysen | 2018-08-25 | 1 | -2/+2 |
| | | | |||||
* | | | Removed a redundant comment from matchOrders | Greg Hysen | 2018-08-25 | 1 | -5/+1 |
| | | | |||||
* | | | Rounding for fees in match orders addressed, plus example | Greg Hysen | 2018-08-25 | 1 | -4/+4 |
| | | | |||||
* | | | Added fees to matchOrders (previously in calculateFillResults | Greg Hysen | 2018-08-25 | 1 | -0/+24 |
| | | | |||||
* | | | Removed calculateFillResults from matchOrders workflow. Eliminates ↵ | Greg Hysen | 2018-08-25 | 1 | -38/+29 |
| |/ |/| | | | | | compounded rounding errors. | ||||
* | | Only use one nonReentrant modifier, remove modifier from fillOrderNoThrow ↵ | Amir Bandeali | 2018-08-25 | 5 | -57/+11 |
| | | | | | | | | variations | ||||
* | | Update ReentrantERC20Token with new functions and check that revert is ↵ | Amir Bandeali | 2018-08-25 | 1 | -35/+75 |
| | | | | | | | | occuring for correct reason | ||||
* | | Add nonReentrant modifiers on functions that use getCurrentContextAddress ↵ | Amir Bandeali | 2018-08-25 | 4 | -15/+17 |
| | | | | | | | | only, add lockMutex modifier on functions that make external calls | ||||
* | | Split modifiers into check only and check, lock, unlock | Amir Bandeali | 2018-08-25 | 1 | -0/+13 |
| | | |||||
* | | Add ReentrantToken | Amir Bandeali | 2018-08-25 | 1 | -0/+173 |
| | | |||||
* | | Add internal fill functions, add reentrancy guard to public functions that ↵ | Amir Bandeali | 2018-08-25 | 6 | -41/+148 |
| | | | | | | | | make external calls | ||||
* | | Add ReentrancyGuard contract | Amir Bandeali | 2018-08-25 | 1 | -0/+44 |
| | | |||||
* | | Remove redundant sload from getCurrentContextAddress | Amir Bandeali | 2018-08-25 | 1 | -1/+2 |
| | | |||||
* | | Merge pull request #1003 from 0xProject/feature/contracts/roundup | Amir Bandeali | 2018-08-25 | 9 | -36/+198 |
|\ \ | | | | | | | [contracts] Add getPartialAmountCeil and isRoundingErrorCeil | ||||
| * | | Stylistic fixes | Remco Bloemen | 2018-08-25 | 1 | -5/+20 |
| | | | |||||
| * | | Append -Floor to getPartialAmount and isRoundingError | Remco Bloemen | 2018-08-25 | 9 | -25/+25 |
| | | | |||||
| * | | Disambiguate the operator precedence | Remco Bloemen | 2018-08-25 | 1 | -2/+2 |
| | | | |||||
| * | | Add tests | Remco Bloemen | 2018-08-25 | 2 | -1/+52 |
| | | | |||||
| * | | Add docs | Remco Bloemen | 2018-08-25 | 1 | -2/+7 |
| | | | |||||
| * | | Fix incorect modulus | Remco Bloemen | 2018-08-25 | 1 | -4/+2 |
| | | | |||||
| * | | Add DIVISION_BY_ZERO to getPartialAmount for consistency | Remco Bloemen | 2018-08-25 | 1 | -4/+3 |
| | | | |||||
| * | | Improve getPartialAmountCeil docs | Remco Bloemen | 2018-08-25 | 1 | -3/+3 |
| | | | |||||
| * | | Add tests for getPartialAmountCeil | Remco Bloemen | 2018-08-25 | 1 | -0/+17 |
| | | | |||||
| * | | Add isRoundingErrorCeil | Remco Bloemen | 2018-08-25 | 1 | -0/+29 |
| | | | |||||
| * | | Add getPartialAmountCeil | Remco Bloemen | 2018-08-25 | 1 | -3/+32 |
| | | | |||||
| * | | Handle zero case | Remco Bloemen | 2018-08-25 | 1 | -3/+15 |
| | | | |||||
| * | | Fix isRoundingError | Remco Bloemen | 2018-08-25 | 1 | -10/+17 |
| | | | |||||
* | | | Skip self-transfers | Remco Bloemen | 2018-08-25 | 1 | -1/+1 |
| |/ |/| | |||||
* | | Remove Trezor SignatureType | Amir Bandeali | 2018-08-25 | 2 | -30/+1 |
| | | |||||
* | | Remove SignatureType.Caller | Amir Bandeali | 2018-08-25 | 3 | -31/+16 |
| | | |||||
* | | Rethrow Wallet and Validator errors | Amir Bandeali | 2018-08-25 | 1 | -4/+28 |
| | | |||||
* | | Rename TestStaticCall => TestStaticCallReceiver | Amir Bandeali | 2018-08-25 | 1 | -1/+2 |
| | | |||||
* | | Revert if undefined function called in AssetProxies | Amir Bandeali | 2018-08-25 | 2 | -0/+6 |
| | | |||||
* | | Remove pragma experimental v0.5.0 and use staticcall is assembly | Amir Bandeali | 2018-08-25 | 4 | -4/+128 |
| | | |||||
* | | Add pragma experimental v0.5.0 to SignatureValidator and add tests | Amir Bandeali | 2018-08-25 | 3 | -0/+66 |
|/ | |||||
* | Add getBalancesAndAllowances | Amir Bandeali | 2018-08-22 | 1 | -0/+19 |
| | |||||
* | Add ZRX balances and allowances to TraderInfo | Amir Bandeali | 2018-08-22 | 1 | -7/+16 |
| | |||||
* | Don't throw if ERC721 token isn't owned | Amir Bandeali | 2018-08-22 | 1 | -1/+51 |
| | |||||
* | Add OrderValidator contract | Amir Bandeali | 2018-08-22 | 14 | -23/+163 |
| | |||||
* | Remove redundant constructor args | Amir Bandeali | 2018-08-22 | 2 | -8/+8 |
| | |||||
* | Fix typos, run prettier, and allow anyone to call on DummyERC721Token | Amir Bandeali | 2018-08-17 | 5 | -6/+6 |
| | |||||
* | Add ERC721Receiver that returns incorrect value | Amir Bandeali | 2018-08-17 | 1 | -0/+66 |
| | |||||
* | Increase max mintable amount | Amir Bandeali | 2018-08-17 | 1 | -1/+1 |
| | |||||
* | Fix DummyERC721Receiver | Amir Bandeali | 2018-08-17 | 1 | -32/+32 |
| | |||||
* | Add tests for ERC20 token with no return values | Amir Bandeali | 2018-08-17 | 1 | -0/+116 |
| | |||||
* | Add constructor back to DummyERC721Token | Amir Bandeali | 2018-08-17 | 1 | -0/+12 |
| | |||||
* | Restructure directories | Amir Bandeali | 2018-08-17 | 6 | -52/+103 |
| | |||||
* | Update ERC20Token | Amir Bandeali | 2018-08-17 | 6 | -47/+113 |
| | |||||
* | Optimize ERC721Token | Amir Bandeali | 2018-08-17 | 5 | -510/+462 |
| | |||||
* | Change withdrawERC20 => withdrawAsset, reuse transfer logic | Amir Bandeali | 2018-08-10 | 4 | -18/+15 |
| | |||||
* | Make marketBuy functions revert if entire amount not filled | Amir Bandeali | 2018-08-10 | 5 | -98/+29 |
| | |||||
* | Revert incrementing memory ptr in dispatchTransferFrom | Amir Bandeali | 2018-08-09 | 3 | -9/+3 |
| | |||||
* | Test abiEncodeFillOrder as part of combinatorial follOrder tests | Amir Bandeali | 2018-08-09 | 1 | -0/+1 |
| | |||||
* | Increment free memory pointer for internal Exchange functions that use asm | Amir Bandeali | 2018-08-09 | 3 | -12/+34 |
| | |||||
* | fix comments and styling for MixinSignatureValidator | Amir Bandeali | 2018-08-09 | 1 | -4/+9 |
| | |||||
* | Use asm for hashEIP712Message, increment free memory pointer after asm ↵ | Amir Bandeali | 2018-08-09 | 2 | -9/+28 |
| | | | | hashing functions | ||||
* | Fix comments, styling, and optimize hashOrder | Amir Bandeali | 2018-08-08 | 3 | -25/+33 |
| | |||||
* | Add combinatorial tests for internal Exchange functions (#807) | Alex Browne | 2018-07-27 | 2 | -1/+120 |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * WIP add combinatorial tests for internal Exchange functions * Change combinitorial testing strategy based on feedback * Check value of filled[orderHash] in updateFilledState tests * Add combinatorial tests for addFillResults * Add combinatorial tests for getPartialAmount * Implement generic `testWithReferenceFuncAsync` * Implement generic `testCombinatoriallyWithReferenceFuncAsync` * Add combinatorial tests for isRoundingError * Add combinatorial tests for calculateFillResults * Add support for Geth in internal contract tests * Fix contract artifacts * Change DECIMAL_PLACES to 78 and add a note. * Document new functions in utils * Optimize tests by only reseting state when needed * Rename/move some files * Print parameter names on failure in testWithReferenceFuncAsync * Add to changelog for utils package * Appease various linters * Rename some more things related to FillOrderCombinatorialUtils * Remove .only from test/exchange/internal.ts * Remove old test for isRoundingError and getPartialAmount * Appease linters again * Remove old todos * Fix typos, add comments, rename some things * Re-add some LibMath tests * Update contract internal tests to use new SafeMath revert reasons * Apply PR feedback from Amir * Apply PR feedback from Remco * Re-add networks to ZRXToken artifact * Remove duplicate Whitelist in compiler.json | ||||
* | Modify WETH9 event arg names to be identical to ERC20 standard suggested names | Fabio Berger | 2018-07-26 | 1 | -4/+4 |
| | |||||
* | Update orderEpoch comments | Amir Bandeali | 2018-07-25 | 2 | -2/+2 |
| | |||||
* | Remove ERC721 callback functions | Amir Bandeali | 2018-07-23 | 4 | -49/+3 |
| | |||||
* | Use != instead of > in loops, add sanity checks to market fill functions | Amir Bandeali | 2018-07-23 | 2 | -18/+18 |
| | |||||
* | Add more tests and fixes | Amir Bandeali | 2018-07-23 | 5 | -5/+8 |
| | |||||
* | Remove MConstants and MixinConstants for LibConstants | Amir Bandeali | 2018-07-23 | 8 | -57/+23 |
| | |||||
* | Remove redundant external call by reimplementing fillOrderNoThrow | Amir Bandeali | 2018-07-23 | 6 | -166/+355 |
| | |||||
* | Remove orders length check | Amir Bandeali | 2018-07-23 | 2 | -11/+0 |
| | |||||
* | Add assertValidFillResults | Amir Bandeali | 2018-07-23 | 1 | -61/+88 |
| | |||||
* | Store orders length in varible before looping over orders | Amir Bandeali | 2018-07-23 | 2 | -14/+25 |
| | |||||
* | Use transferFrom instead of safeTransferFrom | Amir Bandeali | 2018-07-23 | 1 | -5/+2 |
| | |||||
* | Fix rounding error issues, use different logic when makerAsset is ZRX | Amir Bandeali | 2018-07-23 | 6 | -52/+95 |
| | |||||
* | Rename marketSellEth => marketSellWeth | Amir Bandeali | 2018-07-23 | 2 | -3/+3 |
| | |||||
* | Update percentage constants | Amir Bandeali | 2018-07-23 | 3 | -17/+18 |
| | |||||
* | Update transferEthFeeAndRefund, add check to ERC721 transfer | Amir Bandeali | 2018-07-23 | 6 | -26/+54 |
| | |||||
* | Refactor forwarding contract architecture, remove batch functions | Amir Bandeali | 2018-07-23 | 20 | -1038/+572 |
| | |||||
* | Add tests to verify ZRX_ASSET_DATA | Amir Bandeali | 2018-07-23 | 1 | -0/+57 |
| | |||||
* | Add comments for deploying with constant ZRX_ASSET_DATA | Amir Bandeali | 2018-07-23 | 2 | -2/+15 |
| | |||||
* | Move example contracts into new examples dir | Amir Bandeali | 2018-07-23 | 6 | -6/+11 |
| | |||||
* | Merge pull request #877 from 0xProject/feature/contracts/removeERC721Callback | Amir Bandeali | 2018-07-23 | 3 | -79/+25 |
|\ | | | | | Removed receiverData and `onReceive` callback from ERC721 proxy. | ||||
| * | Removed receiverData from ERC721 Proxy Id generation | Greg Hysen | 2018-07-18 | 3 | -6/+4 |
| | | |||||
| * | Removed receiverData and `onReceive` callback from ERC721 proxy. | Greg Hysen | 2018-07-18 | 1 | -73/+21 |
| | | |||||
* | | Hard code fillOrder selector into abiEncodeFillOrder | Amir Bandeali | 2018-07-20 | 1 | -7/+4 |
| | | |||||
* | | Returns byte array instead of memory range for encoding fillOrder calldata | Greg Hysen | 2018-07-20 | 2 | -24/+26 |
| | | |||||
* | | Created LibAbiEncoder with `fillOrderNoThrow` | Greg Hysen | 2018-07-20 | 2 | -170/+235 |
|/ | |||||
* | Reorder checks in getOrderInfo | Amir Bandeali | 2018-07-18 | 1 | -7/+9 |
| | |||||
* | Add getOrdersInfo function | Amir Bandeali | 2018-07-18 | 2 | -0/+24 |
| | |||||
* | Remove extra ERC20ProxyId from Forwarder constructor | Jacob Evans | 2018-07-11 | 1 | -1/+0 |
| | |||||
* | Add Migrations for Forwarder | Jacob Evans | 2018-07-11 | 1 | -2/+2 |
| | |||||
* | Add revert reasons and optimization to safeMath | Amir Bandeali | 2018-07-11 | 1 | -3/+15 |
| | |||||
* | Use 0.4.10 in AssetProxyOwner, add readBytes4 to contract and remove LibBytes | Amir Bandeali | 2018-07-11 | 3 | -10/+43 |
| | |||||
* | Move functions out of Forwarder.sol | Amir Bandeali | 2018-07-08 | 12 | -42/+35 |
| | |||||
* | Add interfaces and mixins | Amir Bandeali | 2018-07-08 | 16 | -61/+573 |
| | |||||
* | Combine mixins | Amir Bandeali | 2018-07-08 | 6 | -284/+218 |
| | |||||
* | Use interfaces instead of full contracts | Amir Bandeali | 2018-07-08 | 4 | -7/+39 |
| | |||||
* | Remove TODOs that won't be addressed | Amir Bandeali | 2018-07-07 | 4 | -10/+3 |
| | |||||
* | Remove AbiV2 where unused | Amir Bandeali | 2018-07-07 | 23 | -38/+27 |
| | |||||
* | Only reset currentContextAddress if it was previously updated | Amir Bandeali | 2018-07-07 | 1 | -8/+11 |
| | |||||
* | Change uint => uint256 | Amir Bandeali | 2018-07-07 | 1 | -8/+7 |
| | |||||
* | Apply new linter rules | Amir Bandeali | 2018-07-07 | 73 | -298/+350 |
| | |||||
* | Merge pull request #522 from 0xProject/feature/contracts/forwarder | Jacob Evans | 2018-07-06 | 11 | -1/+1102 |
|\ | | | | | Forwarding contract | ||||
| * | Forwarding contract (squashed commits) | Jacob Evans | 2018-07-06 | 11 | -1/+1102 |
| | | |||||
* | | Contracts 2.0.0 were linted with solhint | Alex Shafranovich | 2018-07-06 | 60 | -1/+78 |
|/ | |||||
* | Update file structure | Amir Bandeali | 2018-06-30 | 107 | -3375/+1 |
| | |||||
* | Make registerAssetProxy append only | Amir Bandeali | 2018-06-30 | 6 | -60/+25 |
| | |||||
* | Change getTransactionReceipt to awaitTransactionMined | Amir Bandeali | 2018-06-30 | 1 | -1/+1 |
| | |||||
* | Update expectRevertReasonOrAlwaysFailingTransactionAsync to check status codes | Alex Browne | 2018-06-30 | 1 | -6/+35 |
| | |||||
* | Fix deployment to geth | Amir Bandeali | 2018-06-30 | 1 | -1/+1 |
| | |||||
* | Fix tests | Amir Bandeali | 2018-06-30 | 3 | -7/+10 |
| | |||||
* | Remove transferFrom, implement in fallback function | Amir Bandeali | 2018-06-30 | 6 | -290/+267 |
| | |||||
* | Merge pull request #774 from 0xProject/style/contracts/abiEncodeAsmComments | Amir Bandeali | 2018-06-30 | 2 | -0/+22 |
|\ | | | | | Added some comments for ASM hashing | ||||
| * | Updated hashing comments as-per PR | Greg Hysen | 2018-06-29 | 1 | -9/+4 |
| | | |||||
| * | Comments for ASM hashing | Greg Hysen | 2018-06-26 | 2 | -0/+27 |
| | | |||||
* | | Remove unused constants | Fabio Berger | 2018-06-28 | 1 | -2/+0 |
| | | |||||
* | | Reduce the userAddresses to the individual addresses actually used by tests, ↵ | Fabio Berger | 2018-06-28 | 1 | -1/+2 |
| | | | | | | | | as well as only deploy the number of ERC20 tokens needed for each test suite | ||||
* | | Add await time constant | Fabio Berger | 2018-06-28 | 1 | -0/+2 |
| | | |||||
* | | Add missing awaitTransactionSuccessAsync calls | Fabio Berger | 2018-06-28 | 1 | -6/+10 |
| | | |||||
* | | Merge branch 'v2-prototype' into feature/combinatorial-testing | Fabio Berger | 2018-06-27 | 12 | -517/+312 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * v2-prototype: Improve 'web3-provider-engine' typings Fix build Update artifacts Add more revert reasons to tests Make dispatchTransferFrom revert with reason from AssetProxy on failure Remove redundant files, hard code function selector in dispatchTransferFrom, and modify revert reason Updated offset of receiverData length to reflect new assetData encoding. ERC20 Custom storage layout Golf the authorization check ERC20 manual memory layout ERC20 inline return ERC721 inline return Golf ERC721 transferFrom Inline ERC20 tranferFrom selector constant Optimize ERC20 transferFrom Polish asset picker layout Update relayer grid fallback icon Change metamask install link based on browser Use ethereumjs-tx@1.3.5 instead of our fork # Conflicts: # packages/contracts/test/asset_proxy/proxies.ts # packages/contracts/test/exchange/core.ts # packages/migrations/artifacts/2.0.0/DummyERC721Token.json | ||||
| * | | Make dispatchTransferFrom revert with reason from AssetProxy on failure | Amir Bandeali | 2018-06-27 | 1 | -5/+4 |
| | | | |||||
| * | | Remove redundant files, hard code function selector in dispatchTransferFrom, ↵ | Amir Bandeali | 2018-06-27 | 13 | -701/+198 |
| | | | | | | | | | | | | and modify revert reason | ||||
| * | | Updated offset of receiverData length to reflect new assetData encoding. | Greg Hysen | 2018-06-27 | 1 | -2/+2 |
| | | | |||||
| * | | ERC20 Custom storage layout | Remco Bloemen | 2018-06-27 | 2 | -3/+5 |
| | | | |||||
| * | | Golf the authorization check | Remco Bloemen | 2018-06-27 | 2 | -122/+132 |
| | | | |||||
| * | | ERC20 manual memory layout | Remco Bloemen | 2018-06-27 | 1 | -11/+6 |
| | | | |||||
| * | | ERC20 inline return | Remco Bloemen | 2018-06-27 | 1 | -6/+11 |
| | | | |||||
| * | | ERC721 inline return | Remco Bloemen | 2018-06-27 | 1 | -8/+9 |
| | | | |||||
| * | | Golf ERC721 transferFrom | Remco Bloemen | 2018-06-27 | 2 | -3/+166 |
| | | | |||||
| * | | Inline ERC20 tranferFrom selector constant | Remco Bloemen | 2018-06-27 | 1 | -4/+3 |
| | | | |||||
| * | | Optimize ERC20 transferFrom | Remco Bloemen | 2018-06-27 | 1 | -2/+126 |
| | | | |||||
* | | | Fix type issue | Fabio Berger | 2018-06-27 | 1 | -2/+2 |
| | | | |||||
* | | | Merge branch 'refactor/check-revert-reasons' into feature/combinatorial-testing | Fabio Berger | 2018-06-27 | 1 | -1/+5 |
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * refactor/check-revert-reasons: Temporarily switch revert reasons to `TransferFailed`. Should be `InvalidAmount` but because of an oversight in the assembly implementation of `dispatchTransferFrom`, it always throws `TransferFailed` Expect RevertReason be passed in, not string Rename RevertReasons to RevertReason since singular enum names are more common # Conflicts: # packages/contracts/test/asset_proxy/proxies.ts # packages/contracts/test/exchange/core.ts | ||||
| * | | Expect RevertReason be passed in, not string | Fabio Berger | 2018-06-26 | 1 | -1/+5 |
| | | | |||||
| * | | Merge branch 'v2-prototype' into refactor/check-revert-reasons | Fabio Berger | 2018-06-26 | 23 | -202/+370 |
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * v2-prototype: (21 commits) Don't use variables for revert reasons Add comments to dispatchTransferFrom Document IAssetData Compute bytes4 proxyid constants Return inplace bytes in LibBytes.readBytesWithLength Add Greg's documentation to MixinErc721Transfer Add Greg's documentation to MixinErc20Transfer Gas golf the transfer success logic Optimize like in PR #767 Run prettier and fix linting errors Fix tests Update tests Fix AssetProxyUtils Update tests Fix ERC721Proxy TokenId Update AssetProxyUtils Add IAssetData WIP ABI encoded asset data Remove proxyId argument from dispatchTransferFrom ... # Conflicts: # packages/contracts/test/asset_proxy/proxies.ts | ||||
* | | | | Update AbstractAssetWrapper | Fabio Berger | 2018-06-26 | 1 | -1/+1 |
| | | | | |||||
* | | | | Fix merge issues | Fabio Berger | 2018-06-26 | 4 | -16/+17 |
| | | | | |||||
* | | | | Merge branch 'v2-prototype' into feature/combinatorial-testing | Fabio Berger | 2018-06-26 | 23 | -202/+370 |
|\ \ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * v2-prototype: (21 commits) Don't use variables for revert reasons Add comments to dispatchTransferFrom Document IAssetData Compute bytes4 proxyid constants Return inplace bytes in LibBytes.readBytesWithLength Add Greg's documentation to MixinErc721Transfer Add Greg's documentation to MixinErc20Transfer Gas golf the transfer success logic Optimize like in PR #767 Run prettier and fix linting errors Fix tests Update tests Fix AssetProxyUtils Update tests Fix ERC721Proxy TokenId Update AssetProxyUtils Add IAssetData WIP ABI encoded asset data Remove proxyId argument from dispatchTransferFrom ... # Conflicts: # packages/contracts/test/asset_proxy/proxies.ts | ||||
| * | | | Don't use variables for revert reasons | Amir Bandeali | 2018-06-26 | 13 | -84/+82 |
| | | | | |||||
| * | | | Add comments to dispatchTransferFrom | Amir Bandeali | 2018-06-26 | 1 | -30/+56 |
| | | | | |||||
| * | | | Document IAssetData | Remco Bloemen | 2018-06-26 | 1 | -0/+3 |
| | | | | |||||
| * | | | Compute bytes4 proxyid constants | Remco Bloemen | 2018-06-26 | 2 | -2/+2 |
| | | | | |||||
| * | | | Return inplace bytes in LibBytes.readBytesWithLength | Remco Bloemen | 2018-06-26 | 1 | -9/+7 |
| | | | | |||||
| * | | | Add Greg's documentation to MixinErc721Transfer | Remco Bloemen | 2018-06-26 | 1 | -33/+44 |
| | | | | |||||
| * | | | Add Greg's documentation to MixinErc20Transfer | Remco Bloemen | 2018-06-26 | 1 | -13/+35 |
| | | | | |||||
| * | | | Gas golf the transfer success logic | Remco Bloemen | 2018-06-26 | 1 | -11/+10 |
| | | | | |||||
| * | | | Optimize like in PR #767 | Remco Bloemen | 2018-06-26 | 3 | -34/+157 |
| | | | | |||||
| * | | | Fix tests | Amir Bandeali | 2018-06-26 | 1 | -16/+0 |
| | | | | |||||
| * | | | Fix ERC721Proxy TokenId | Remco Bloemen | 2018-06-26 | 1 | -1/+1 |
| | | | | |||||
| * | | | WIP | Remco Bloemen | 2018-06-26 | 2 | -2/+2 |
| | | | | |||||
| * | | | ABI encoded asset data | Remco Bloemen | 2018-06-26 | 9 | -29/+51 |
| | | | | |||||
| * | | | Remove proxyId argument from dispatchTransferFrom | Remco Bloemen | 2018-06-26 | 6 | -34/+16 |
| | |/ | |/| | |||||
* | | | Prettier fixes | Fabio Berger | 2018-06-26 | 1 | -4/+1 |
| | | | |||||
* | | | Fix tslint issues | Fabio Berger | 2018-06-26 | 4 | -18/+3 |
| | | | |||||
* | | | Check revert reasons in declarative and combinatorial tests | Fabio Berger | 2018-06-25 | 1 | -7/+8 |
| | | | |||||
* | | | merge check-revert-reasons | Fabio Berger | 2018-06-25 | 1 | -44/+0 |
|\ \ \ | | |/ | |/| | |||||
| * | | Move RevertReasons to @0xproject/types package | Fabio Berger | 2018-06-25 | 1 | -44/+0 |
| | | | |||||
* | | | merge check-revert-reasons | Fabio Berger | 2018-06-25 | 5 | -21/+63 |
|\| | | |||||
| * | | Merge branch 'v2-prototype' into refactor/check-revert-reasons | Fabio Berger | 2018-06-25 | 19 | -340/+424 |
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * v2-prototype: (48 commits) Fix typos in comments Add modifier and tests for removeAuthorizedAddressAtIndex Update and add tests Change removeAuthorizedAddress => removeAuthorizedAddressAtIndex Move isFunctionRemoveAuthorizedAddress to test Fix usage of `popLastByte` Fix LibBytes is a library Remove `areBytesEqual` Fix usage of `contentAddress()` Clean low bits in bytes4 Clean high bits in address Refactor LibBytes.readBytes4 for consistency Fix LibBytes.equals Add trailing garbage testcase for LibBytes.equals Rename bytes.equals Add slice and sliceDestructive Rename bytes.rawAddress and add bytes.contentAddress Rename read/writeBytesWithLength Using LibBytes for bytes Make LibBytes a library ... # Conflicts: # packages/contracts/src/contracts/current/utils/Ownable/Ownable.sol # packages/contracts/test/libraries/lib_bytes.ts | ||||
| * | | Rename ContractLibErrors to RevertReasons | Fabio Berger | 2018-06-23 | 1 | -1/+1 |
| | | | |||||
| * | | Remove revert reason 'Exchange' prefix | Fabio Berger | 2018-06-23 | 1 | -2/+2 |
| | | | |||||
| * | | Move constants over to ContractLibError enum and update all tests | Fabio Berger | 2018-06-23 | 2 | -15/+10 |
| | | | |||||
| * | | Standardize ERC20 error strings given convention | Fabio Berger | 2018-06-23 | 1 | -3/+2 |
| | | | |||||
| * | | Fix test now that contract reverts with message | Fabio Berger | 2018-06-22 | 1 | -0/+1 |
| | | | |||||
| * | | Merge branch 'v2-prototype' into refactor/check-revert-reasons | Fabio Berger | 2018-06-22 | 3 | -27/+48 |
| |\ \ | | | | | | | | | | | | | | | | | * v2-prototype: Check that assetProxy exists before attempting transfer | ||||
| * | | | Change Whitelist error messages to conform to rest and added revert reason ↵ | Fabio Berger | 2018-06-22 | 2 | -5/+7 |
| | | | | | | | | | | | | | | | | checks to transactions tests | ||||
| * | | | Merge branch 'v2-prototype' into refactor/check-revert-reasons | Fabio Berger | 2018-06-22 | 26 | -352/+454 |
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * v2-prototype: (40 commits) Use make-promises-safe as a preloader instead of manually importing Updated compiler runs to be 1,000,000 Add event to setSignatureValidatorApproval, rename signer => signerAddress accross all contracts Add senderAddress to Fill and Cancel logs, add comments to events and types Fix Island component Add missing image assets for Chris and Mel Fix some bugs in sol-cov Remove unreachable PreSigned check Fix linting Buttons look hella disabled now Remove border radius, fix width issue for unlock step Add Chris and Mel to about page fix linter issues only call getLocationByOffset if source if defined Set settleOrder and settleMatchedOrders to private Prevent prettier issue Support mobile friendly onboarding flows Removed MixinSettlement. Moved `settleOrder` into `MixinExchangeCore` and `settleMatchedOrders` into `MixinMatchOrders` Migrations after rebasing Linter ... | ||||
| * | | | | Check revert reasons in Authorizable tests | Fabio Berger | 2018-06-21 | 1 | -0/+1 |
| | | | | | |||||
| * | | | | Change revert reason in ownable to be similar to all other revert reasons | Fabio Berger | 2018-06-21 | 1 | -1/+1 |
| | | | | | |||||
| * | | | | Update core tests to actually check revert message | Fabio Berger | 2018-06-21 | 2 | -0/+44 |
| | | | | | |||||
* | | | | | Merge branch 'v2-prototype' into feature/combinatorial-testing | Fabio Berger | 2018-06-25 | 38 | -703/+910 |
|\ \ \ \ \ | | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * v2-prototype: (97 commits) Fix typos in comments Add modifier and tests for removeAuthorizedAddressAtIndex Update and add tests Change removeAuthorizedAddress => removeAuthorizedAddressAtIndex Move isFunctionRemoveAuthorizedAddress to test Fix usage of `popLastByte` Fix LibBytes is a library Remove `areBytesEqual` Fix usage of `contentAddress()` Clean low bits in bytes4 Clean high bits in address Refactor LibBytes.readBytes4 for consistency Fix LibBytes.equals Add trailing garbage testcase for LibBytes.equals Rename bytes.equals Add slice and sliceDestructive Rename bytes.rawAddress and add bytes.contentAddress Rename read/writeBytesWithLength Using LibBytes for bytes Make LibBytes a library ... # Conflicts: # packages/contracts/src/utils/constants.ts # packages/contracts/test/exchange/core.ts | ||||
| * | | | | Fix typos in comments | Remco Bloemen | 2018-06-23 | 1 | -2/+2 |
| | | | | | |||||
| * | | | | Add modifier and tests for removeAuthorizedAddressAtIndex | Amir Bandeali | 2018-06-23 | 2 | -2/+10 |
| | | | | | |||||
| * | | | | Update and add tests | Amir Bandeali | 2018-06-23 | 2 | -3/+5 |
| | | | | | |||||
| * | | | | Change removeAuthorizedAddress => removeAuthorizedAddressAtIndex | Amir Bandeali | 2018-06-23 | 2 | -13/+22 |
| | | | | | |||||
| * | | | | Move isFunctionRemoveAuthorizedAddress to test | Remco Bloemen | 2018-06-23 | 3 | -14/+50 |
| | | | | | |||||
| * | | | | Fix usage of `popLastByte` | Remco Bloemen | 2018-06-23 | 1 | -2/+2 |
| | | | | | |||||
| * | | | | Fix LibBytes is a library | Remco Bloemen | 2018-06-23 | 6 | -15/+12 |
| | | | | | |||||
| * | | | | Remove `areBytesEqual` | Remco Bloemen | 2018-06-23 | 1 | -35/+0 |
| | | | | | |||||
| * | | | | Fix usage of `contentAddress()` | Remco Bloemen | 2018-06-23 | 1 | -2/+2 |
| | | | | | |||||
| * | | | | Clean low bits in bytes4 | Remco Bloemen | 2018-06-23 | 1 | -0/+3 |
| | | | | | |||||
| * | | | | Clean high bits in address | Remco Bloemen | 2018-06-23 | 1 | -0/+4 |
| | | | | | |||||
| * | | | | Refactor LibBytes.readBytes4 for consistency | Remco Bloemen | 2018-06-23 | 3 | -11/+18 |
| | | | | | |||||
| * | | | | Fix LibBytes.equals | Remco Bloemen | 2018-06-23 | 1 | -21/+4 |
| | | | | | |||||
| * | | | | Add trailing garbage testcase for LibBytes.equals | Remco Bloemen | 2018-06-23 | 1 | -0/+11 |
| | | | | | |||||
| * | | | | Rename bytes.equals | Remco Bloemen | 2018-06-23 | 2 | -2/+37 |
| | | | | | |||||
| * | | | | Add slice and sliceDestructive | Remco Bloemen | 2018-06-23 | 1 | -0/+46 |
| | | | | | |||||
| * | | | | Rename bytes.rawAddress and add bytes.contentAddress | Remco Bloemen | 2018-06-23 | 2 | -8/+25 |
| | | | | | |||||
| * | | | | Rename read/writeBytesWithLength | Remco Bloemen | 2018-06-23 | 3 | -7/+7 |
| | | | | | |||||
| * | | | | Using LibBytes for bytes | Remco Bloemen | 2018-06-23 | 6 | -26/+38 |
| | | | | | |||||
| * | | | | Make LibBytes a library | Remco Bloemen | 2018-06-23 | 6 | -35/+30 |
| | | | | | |||||
| * | | | | Merge LibMem and LibBytes | Remco Bloemen | 2018-06-23 | 6 | -206/+150 |
| | | | | | |||||
| * | | | | Merge pull request #749 from 0xProject/feature/contracts/transactionHashAsm | Amir Bandeali | 2018-06-23 | 1 | -8/+18 |
| |\ \ \ \ | | |_|_|/ | |/| | | | Converted `hashZeroExTransaction` to assembly. Saves 1k gas | ||||
| | * | | | Apply mask to address to zero-out unused bytes. | Greg Hysen | 2018-06-23 | 1 | -1/+1 |
| | | | | | |||||
| | * | | | Converted `hashZeroExTransaction` to assembly. Saves 1k gas | Greg Hysen | 2018-06-22 | 1 | -8/+18 |
| | | |/ | | |/| | |||||
| * | | | Merge pull request #748 from 0xProject/fix/contracts/nullAssetProxy | Fabio Berger | 2018-06-22 | 3 | -27/+48 |
| |\ \ \ | | |/ / | |/| | | Check that assetProxy exists before attempting transfer | ||||
| | * | | Check that assetProxy exists before attempting transfer | Amir Bandeali | 2018-06-22 | 3 | -27/+48 |
| | | | | |||||
| * | | | Add event to setSignatureValidatorApproval, rename signer => signerAddress ↵ | Amir Bandeali | 2018-06-22 | 14 | -68/+94 |
| | | | | | | | | | | | | | | | | accross all contracts | ||||
| * | | | Add senderAddress to Fill and Cancel logs, add comments to events and types | Amir Bandeali | 2018-06-22 | 4 | -44/+45 |
| |/ / | |||||
| * | | Remove unreachable PreSigned check | Austin Roberts | 2018-06-22 | 1 | -4/+0 |
| | | | | | | | | | | | | This code was unreachable, as it had the exact same condition as line 206. | ||||
| * | | Set settleOrder and settleMatchedOrders to private | Greg Hysen | 2018-06-21 | 4 | -25/+2 |
| | | | |||||
| * | | Removed MixinSettlement. Moved `settleOrder` into `MixinExchangeCore` and ↵ | Greg Hysen | 2018-06-21 | 8 | -238/+205 |
| | | | | | | | | | | | | `settleMatchedOrders` into `MixinMatchOrders` | ||||
| * | | Renamed constants in test wallet/validator | Greg Hysen | 2018-06-21 | 2 | -11/+11 |
| | | | |||||
| * | | Few more minor #nit wording changes | Greg Hysen | 2018-06-21 | 2 | -5/+5 |
| | | | |||||
| * | | Fixed two mislabelled revert reasons + Signature Validator revert tests ↵ | Greg Hysen | 2018-06-21 | 2 | -2/+7 |
| | | | | | | | | | | | | working on Geth | ||||
| * | | Minor style tweaks | Greg Hysen | 2018-06-21 | 3 | -8/+8 |
| | | | |||||
| * | | Twenty new tests for MixinSignatureValidator. Fixed handling of unsupported ↵ | Greg Hysen | 2018-06-21 | 6 | -11/+141 |
| |/ | | | | | | | types. Fixed trezor prefix. | ||||
* | | Add support for approveAll in assetWrapper and fillOrderScenarios | Fabio Berger | 2018-06-21 | 3 | -4/+41 |
| | | |||||
* | | Improve verbose logging | Fabio Berger | 2018-06-21 | 1 | -2/+5 |
| | | |||||
* | | Fix merge variable name issue | Fabio Berger | 2018-06-20 | 1 | -1/+1 |
| | | |||||
* | | Add support for setting allowance to unlimited | Fabio Berger | 2018-06-20 | 2 | -7/+35 |
| | | |||||
* | | Add noop conditionals for clarity | Fabio Berger | 2018-06-20 | 1 | -0/+10 |
| | | |||||
* | | Rename for clarity | Fabio Berger | 2018-06-20 | 1 | -8/+8 |
| | | |||||
* | | Split balance and allowance amount types in prep of adding the "unlimited" ↵ | Fabio Berger | 2018-06-20 | 2 | -38/+45 |
| | | | | | | | | allowance enum value | ||||
* | | Remove unneeded explicit type def | Fabio Berger | 2018-06-20 | 1 | -1/+1 |
| | | |||||
* | | Add TS 2.7 numeric separators to improve readability | Fabio Berger | 2018-06-20 | 1 | -6/+6 |
| | | |||||
* | | Merge branch 'v2-prototype' into feature/combinatorial-testing | Fabio Berger | 2018-06-20 | 15 | -82/+301 |
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * v2-prototype: (22 commits) Fix closing parens in liborder Update after rebase ERC721Proxy Always call safeTransferFrom Rename makerEpoch => orderEpoch Make cancelOrdersUpTo compatible with sender abstraction Update PR template Use Image component instead of img tag Assembler orderHash function Optimize and remove redundant encodePacked Fix linting issue Fix bug where we do fetch balances on wallet login Check network state immediately instead of waiting for delay Fix onboarding persisting when changing routes Consolidate account state messaging logic Only elevate wallet zIndex when onboarding is in progress Rebase and update feedback Run linter Add Portal v2 logging Simplified handling of source < 32 edge case Basic EIP712 encoder ... | ||||
| * | Merge pull request #637 from 0xProject/bug/contracts/eip712-191-prefix | Jacob Evans | 2018-06-20 | 6 | -42/+159 |
| |\ | | | | | | | Add missing EIP191 prefix for EIP712 and Execute Transaction | ||||
| | * | Fix closing parens in liborder | Jacob Evans | 2018-06-20 | 1 | -16/+15 |
| | | | |||||
| | * | Assembler orderHash function | Remco Bloemen | 2018-06-19 | 1 | -18/+22 |
| | | | |||||
| | * | Optimize and remove redundant encodePacked | Jacob Evans | 2018-06-19 | 4 | -13/+23 |
| | | | |||||
| | * | Merge branch 'v2-prototype' into bug/contracts/eip712-191-prefix | Jacob Evans | 2018-06-18 | 46 | -311/+1056 |
| | |\ | |||||
| | * | | Rebase and update feedback | Jacob Evans | 2018-06-18 | 4 | -81/+85 |
| | | | | | | | | | | | | | | | | | | | | | | | | Cache the domain separator data with address this Use the EIP712Types enum for types everywhere Rename EIP712 struct ExecuteTransaction to ZeroExTransaction | ||||
| | * | | Basic EIP712 encoder | Jacob Evans | 2018-06-08 | 1 | -10/+11 |
| | | | | |||||
| | * | | Update Order utils to use eip712 | Jacob Evans | 2018-06-06 | 7 | -170/+44 |
| | | | | |||||
| | * | | Add missing EIP191 prefix for EIP712 | Jacob Evans | 2018-06-05 | 9 | -48/+272 |
| | | | | |||||
| * | | | Update after rebase | Jacob Evans | 2018-06-20 | 1 | -7/+1 |
| | | | | |||||
| * | | | Rename makerEpoch => orderEpoch | Amir Bandeali | 2018-06-20 | 5 | -26/+29 |
| | | | | |||||
| * | | | Make cancelOrdersUpTo compatible with sender abstraction | Amir Bandeali | 2018-06-20 | 5 | -7/+110 |
| | | | | |||||
| * | | | Merge pull request #683 from 0xProject/fix/contracts/memcpy | Remco Bloemen | 2018-06-19 | 1 | -10/+12 |
| |\ \ \ | | | | | | | | | | | Simplified handling of source < 32 edge case | ||||
| | * | | | Simplified handling of source < 32 edge case | Remco Bloemen | 2018-06-11 | 1 | -10/+12 |
| | | | | | |||||
* | | | | | Add ability for verbose logging | Fabio Berger | 2018-06-20 | 1 | -2/+10 |
| | | | | |