aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Update relayer grid fallback iconBrandon Millman2018-06-272-1/+1
|
* Merge pull request #777 from 0xProject/feature/website/browser-detectionBrandon Millman2018-06-276-18/+61
|\ | | | | Change metamask install link based on browser
| * Change metamask install link based on browserBrandon Millman2018-06-266-18/+104
| |
* | Merge pull request #776 from 0xProject/ethereum-js-tx-1.3.5Alex Browne2018-06-274-5/+54
|\ \ | | | | | | Use ethereumjs-tx@1.3.5 instead of our fork
| * | Use ethereumjs-tx@1.3.5 instead of our forkAlex Browne2018-06-264-5/+54
| | |
* | | Merge pull request #760 from 0xProject/refactor/check-revert-reasonsFabio Berger2018-06-2716-140/+255
|\ \ \ | |_|/ |/| | Check Revert Reasons in Contract Tests
| * | Temporarily switch revert reasons to `TransferFailed`. Should be ↵Fabio Berger2018-06-271-2/+2
| | | | | | | | | | | | `InvalidAmount` but because of an oversight in the assembly implementation of `dispatchTransferFrom`, it always throws `TransferFailed`
| * | Expect RevertReason be passed in, not stringFabio Berger2018-06-261-1/+5
| | |
| * | Rename RevertReasons to RevertReason since singular enum names are more commonFabio Berger2018-06-2611-96/+96
| | |
| * | Merge branch 'v2-prototype' into refactor/check-revert-reasonsFabio Berger2018-06-2636-364/+521
| |\ \ | |/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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
* | | Merge pull request #743 from 0xProject/feature/contracts/abi-encodedAmir Bandeali2018-06-2635-361/+478
|\ \ \ | |_|/ |/| | ABI encode assetData fields
| * | Don't use variables for revert reasonsAmir Bandeali2018-06-2614-86/+86
| | |
| * | Add comments to dispatchTransferFromAmir Bandeali2018-06-261-30/+56
| | |
| * | Document IAssetDataRemco Bloemen2018-06-261-0/+3
| | |
| * | Compute bytes4 proxyid constantsRemco Bloemen2018-06-262-2/+2
| | |
| * | Return inplace bytes in LibBytes.readBytesWithLengthRemco Bloemen2018-06-261-9/+7
| | |
| * | Add Greg's documentation to MixinErc721TransferRemco Bloemen2018-06-261-33/+44
| | |
| * | Add Greg's documentation to MixinErc20TransferRemco Bloemen2018-06-261-13/+35
| | |
| * | Gas golf the transfer success logicRemco Bloemen2018-06-261-11/+10
| | |
| * | Optimize like in PR #767Remco Bloemen2018-06-263-34/+157
| | |
| * | Run prettier and fix linting errorsAmir Bandeali2018-06-263-55/+54
| | |
| * | Fix testsAmir Bandeali2018-06-265-21/+4
| | |
| * | Update testsRemco Bloemen2018-06-261-2/+2
| | |
| * | Fix AssetProxyUtilsRemco Bloemen2018-06-261-1/+1
| | |
| * | Update testsRemco Bloemen2018-06-262-33/+20
| | |
| * | Fix ERC721Proxy TokenIdRemco Bloemen2018-06-261-1/+1
| | |
| * | Update AssetProxyUtilsRemco Bloemen2018-06-261-90/+41
| | |
| * | Add IAssetDataRemco Bloemen2018-06-262-1/+2
| | |
| * | WIPRemco Bloemen2018-06-265-23/+36
| | |
| * | ABI encoded asset dataRemco Bloemen2018-06-269-29/+51
| | |
| * | Remove proxyId argument from dispatchTransferFromRemco Bloemen2018-06-268-41/+20
|/ /
* | Merge pull request #772 from 0xProject/add-coverage-instructionsFabio Berger2018-06-261-3/+43
|\ \ | | | | | | Improve Contracts README Testing Section
| * | Document all the contract test options in READMEFabio Berger2018-06-251-3/+43
|/ /
| * Move RevertReasons to @0xproject/types packageFabio Berger2018-06-2512-66/+58
| |
| * Add missing importFabio Berger2018-06-251-1/+4
| |
| * Merge branch 'v2-prototype' into refactor/check-revert-reasonsFabio Berger2018-06-2546-908/+1178
| |\ | |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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
* | Merge pull request #735 from feuGeneA/metacoin-independenceLeonid Logvinov2018-06-252-3/+16
|\ \ | | | | | | Metacoin independence
| * | ethereum-types now only has v0.0.2, no 0.0.1F. Eugene Aumson2018-06-201-1/+1
| | |
| * | add missing metacoin dependenciesF. Eugene Aumson2018-06-201-0/+4
| | |
| * | make metacoin tsconfig independentF. Eugene Aumson2018-06-201-2/+11
| | |
* | | Merge pull request #758 from 0xProject/fix/order-utils/known-order-hashJacob Evans2018-06-251-8/+4
|\ \ \ | | | | | | | | Fix known order hash value test in order-utils
| * | | Fix known order hash value testJacob Evans2018-06-221-8/+4
| | | |
* | | | Merge pull request #698 from 0xProject/fix/contracts/using-libbytesRemco Bloemen2018-06-2325-700/+947
|\ \ \ \ | | | | | | | | | | Fixes and refactoring regarding LibBytes
| * | | | Fix typos in commentsRemco Bloemen2018-06-231-2/+2
| | | | |
| * | | | Add modifier and tests for removeAuthorizedAddressAtIndexAmir Bandeali2018-06-233-2/+79
| | | | |
| * | | | Update and add testsAmir Bandeali2018-06-234-173/+274
| | | | |
| * | | | Change removeAuthorizedAddress => removeAuthorizedAddressAtIndexAmir Bandeali2018-06-232-13/+22
| | | | |
| * | | | Move isFunctionRemoveAuthorizedAddress to testRemco Bloemen2018-06-236-22/+70
| | | | |
| * | | | Fix usage of `popLastByte`Remco Bloemen2018-06-231-2/+2
| | | | |
| * | | | Fix LibBytes is a libraryRemco Bloemen2018-06-236-15/+12
| | | | |
| * | | | Remove `areBytesEqual`Remco Bloemen2018-06-231-35/+0
| | | | |
| * | | | Fix usage of `contentAddress()`Remco Bloemen2018-06-231-2/+2
| | | | |
| * | | | Clean low bits in bytes4Remco Bloemen2018-06-231-0/+3
| | | | |
| * | | | Clean high bits in addressRemco Bloemen2018-06-231-0/+4
| | | | |
| * | | | Refactor LibBytes.readBytes4 for consistencyRemco Bloemen2018-06-234-14/+25
| | | | |
| * | | | Fix LibBytes.equalsRemco Bloemen2018-06-231-21/+4
| | | | |
| * | | | Add trailing garbage testcase for LibBytes.equalsRemco Bloemen2018-06-232-0/+21
| | | | |
| * | | | Rename bytes.equalsRemco Bloemen2018-06-233-15/+50
| | | | |
| * | | | Add slice and sliceDestructiveRemco Bloemen2018-06-231-0/+46
| | | | |
| * | | | Rename bytes.rawAddress and add bytes.contentAddressRemco Bloemen2018-06-232-8/+25
| | | | |
| * | | | Rename read/writeBytesWithLengthRemco Bloemen2018-06-234-36/+36
| | | | |
| * | | | Using LibBytes for bytesRemco Bloemen2018-06-236-26/+38
| | | | |
| * | | | Make LibBytes a libraryRemco Bloemen2018-06-236-35/+30
| | | | |
| * | | | Merge LibMem and LibBytesRemco Bloemen2018-06-2311-399/+322
| | | | |
| * | | | Use provided mem in refernce memcpyRemco Bloemen2018-06-231-2/+2
|/ / / /
* | | | Merge pull request #749 from 0xProject/feature/contracts/transactionHashAsmAmir Bandeali2018-06-231-8/+18
|\ \ \ \ | | | | | | | | | | Converted `hashZeroExTransaction` to assembly. Saves 1k gas
| * | | | Apply mask to address to zero-out unused bytes.Greg Hysen2018-06-231-1/+1
| | | | |
| * | | | Converted `hashZeroExTransaction` to assembly. Saves 1k gasGreg Hysen2018-06-221-8/+18
| | | | |
* | | | | Remove extra border from the bottom of the walletBrandon Millman2018-06-231-3/+11
| | | | |
* | | | | Do not render ZRX or WETH as removable tracked tokensBrandon Millman2018-06-231-1/+4
| | | | |
* | | | | Center items in portal drawer headerBrandon Millman2018-06-231-1/+1
| | | | |
* | | | | Merge pull request #764 from 0xProject/feature/website/relay-grid-stacking-bugFrancesco Agosti2018-06-232-5/+10
|\ \ \ \ \ | | | | | | | | | | | | Fix issue where hovered tiles appear over top bar
| * | | | | Fix issue where hovered tiles appear over top barfragosti2018-06-232-5/+10
|/ / / / /
* | | | | Merge pull request #746 from 0xProject/feature/website/relayer-grid-polishFrancesco Agosti2018-06-2311-152/+138
|\ \ \ \ \ | | | | | | | | | | | | Relayer grid polish
| * | | | | Remove unecessary position relative rulesfragosti2018-06-232-2/+4
| | | | | |
| * | | | | Lint and fix typofragosti2018-06-222-2/+2
| | | | | |
| * | | | | Fix z-index issuesfragosti2018-06-226-49/+63
| | | | | |
| * | | | | Fix issues that arose from mergefragosti2018-06-222-4/+8
| | | | | |
| * | | | | Remove Background componentBrandon Millman2018-06-225-57/+12
| | | | | |
| * | | | | Padding adjustmentBrandon Millman2018-06-221-1/+2
| | | | | |
| * | | | | Remove styles from Portal componentBrandon Millman2018-06-221-24/+13
| | | | | |
| * | | | | Add Background componentBrandon Millman2018-06-223-14/+35
| | | | | |
| * | | | | Various relayer grid ui polish itemsBrandon Millman2018-06-226-42/+42
| |/ / / /
* | | | | Fix tslint errorfragosti2018-06-231-10/+10
| | | | |
* | | | | Merge https://github.com/0xProject/0x-monorepo into v2-prototypefragosti2018-06-232-14/+16
|\ \ \ \ \
| * \ \ \ \ Merge pull request #750 from 0xProject/bug/website/fix-add-tokenBrandon Millman2018-06-232-14/+16
| |\ \ \ \ \ | | | | | | | | | | | | | | Various add token flow bugs
| | * | | | | Pulled out reset AssetPicker state into a common variableBrandon Millman2018-06-231-15/+9
| | | | | | |
| | * | | | | Fix state mutation in reducerBrandon Millman2018-06-221-4/+4
| | | | | | |
| | * | | | | Do not add empty token to local storageBrandon Millman2018-06-221-0/+8
| | |/ / / /
* | | | | | Remove diverse from about pagefragosti2018-06-231-3/+3
| | | | | |
* | | | | | Use Ops instead of Operations in Mels descriptionfragosti2018-06-231-1/+1
|/ / / / /
* | | | | Comment out Chris, and move Mel up in about pagefragosti2018-06-232-12/+12
| | | | |
| | | | * Fix tslintFabio Berger2018-06-234-4/+4
| | | | |
| | | | * Fix prettierFabio Berger2018-06-235-15/+13
| | | | |
| | | | * Rename ContractLibErrors to RevertReasonsFabio Berger2018-06-2311-96/+96
| | | | |
| | | | * Remove revert reason 'Exchange' prefixFabio Berger2018-06-232-3/+3
| | | | |
| | | | * Move constants over to ContractLibError enum and update all testsFabio Berger2018-06-235-44/+42
| | | | |
| | | | * Standardize ERC20 error strings given conventionFabio Berger2018-06-231-3/+2
| | | | |
| | | | * Fix test now that contract reverts with messageFabio Berger2018-06-222-5/+4
| | | | |
| | | | * Merge branch 'v2-prototype' into refactor/check-revert-reasonsFabio Berger2018-06-223-27/+48
| | | | |\ | |_|_|_|/ |/| | | | | | | | | | | | | | * v2-prototype: Check that assetProxy exists before attempting transfer
* | | | | Merge pull request #748 from 0xProject/fix/contracts/nullAssetProxyFabio Berger2018-06-223-27/+48
|\ \ \ \ \ | |_|/ / / |/| | | | Check that assetProxy exists before attempting transfer
| * | | | Check that assetProxy exists before attempting transferAmir Bandeali2018-06-223-27/+48
| | | | |
| | | | * Add revert reason checks to wrapper testsFabio Berger2018-06-221-10/+19
| | | | |
| | | | * Fix testsFabio Berger2018-06-221-2/+2
| | | | |
| | | | * Change Whitelist error messages to conform to rest and added revert reason ↵Fabio Berger2018-06-223-21/+31
| | | | | | | | | | | | | | | | | | | | checks to transactions tests
| | | | * Add revert reason checks to match_orders testsFabio Berger2018-06-221-7/+18
| | | | |
| | | | * Fix testsFabio Berger2018-06-221-2/+2
| | | | |
| | | | * Merge branch 'v2-prototype' into refactor/check-revert-reasonsFabio Berger2018-06-226-21/+268
| | | | |\ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * v2-prototype: Update ganache-core in yarn.lock Add coverage and test:circleci command to sra-report Add missing artifacts Skip failing tests temporarily Fix web3-wrapper test fix name Add missing package tests from circleci
* | | | | Merge pull request #757 from 0xProject/fix/circle-ciFabio Berger2018-06-228-127/+337
|\ \ \ \ \ | | | | | | | | | | | | Add missing package tests from circleci
| * | | | | Update ganache-core in yarn.lockFabio Berger2018-06-221-105/+68
| | | | | |
| * | | | | Add coverage and test:circleci command to sra-reportFabio Berger2018-06-222-0/+4
| | | | | |
| * | | | | Add missing artifactsFabio Berger2018-06-222-0/+194
| | | | | |
| * | | | | Skip failing tests temporarilyFabio Berger2018-06-221-2/+6
| | | | | |
| * | | | | Fix web3-wrapper testFabio Berger2018-06-221-1/+1
| | | | | |
| * | | | | fix nameFabio Berger2018-06-221-1/+1
| | | | | |
| * | | | | Add missing package tests from circleciFabio Berger2018-06-221-19/+64
|/ / / / /
| | | | * Fix expected version in testFabio Berger2018-06-221-1/+1
| | | | |
| | | | * Merge branch 'v2-prototype' into refactor/check-revert-reasonsFabio Berger2018-06-22146-887/+1891
| | | | |\ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * 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 ...
* | | | | Merge pull request #752 from 0xProject/make-promises-safe-preloaderFabio Berger2018-06-2271-72/+17
|\ \ \ \ \ | | | | | | | | | | | | Use make-promises-safe as a preloader instead of manually importing
| * | | | | Use make-promises-safe as a preloader instead of manually importingAlex Browne2018-06-2271-72/+17
| | |/ / / | |/| | |
* | | | | Merge pull request #751 from 0xProject/feature/contracts/compilerRunsAmir Bandeali2018-06-221-1/+1
|\ \ \ \ \ | |/ / / / |/| | | | Updated compiler runs to be 1,000,000
| * | | | Updated compiler runs to be 1,000,000Greg Hysen2018-06-221-1/+1
| | | | |
* | | | | Merge pull request #740 from 0xProject/feature/contracts/updateEventsAmir Bandeali2018-06-2218-112/+189
|\ \ \ \ \ | | | | | | | | | | | | Update events
| * | | | | Add event to setSignatureValidatorApproval, rename signer => signerAddress ↵Amir Bandeali2018-06-2215-70/+144
| | | | | | | | | | | | | | | | | | | | | | | | accross all contracts
| * | | | | Add senderAddress to Fill and Cancel logs, add comments to events and typesAmir Bandeali2018-06-225-44/+47
|/ / / / /
* | | | | Merge pull request #747 from 0xProject/fix/sol-cov-bugsAlex Browne2018-06-222-7/+23
|\ \ \ \ \ | |_|/ / / |/| | | | Fix some bugs in sol-cov
| * | | | Fix some bugs in sol-covAlex Browne2018-06-222-7/+23
| |/ / /
* | | | Merge pull request #744 from NoteGio/v2-prototype-unreachableAmir Bandeali2018-06-221-4/+0
|\ \ \ \ | | | | | | | | | | Remove unreachable PreSigned check
| * | | | Remove unreachable PreSigned checkAustin Roberts2018-06-221-4/+0
| | | | | | | | | | | | | | | | | | | | This code was unreachable, as it had the exact same condition as line 206.
* | | | | Fix Island componentfragosti2018-06-221-2/+2
| | | | |
* | | | | Add missing image assets for Chris and Melfragosti2018-06-222-0/+0
| | | | |
* | | | | Merge pull request #739 from ↵Francesco Agosti2018-06-2212-159/+267
|\ \ \ \ \ | |/ / / / |/| | | | | | | | | | | | | | 0xProject/feature/website/mobile-friendly-onboarding Add support for mobile-friendly onboarding flows
| * | | | Fix lintingfragosti2018-06-221-2/+2
| | | | |
| * | | | Buttons look hella disabled nowfragosti2018-06-222-9/+17
| | | | |
| * | | | Remove border radius, fix width issue for unlock stepfragosti2018-06-224-5/+12
| | | | |
| * | | | Merge branch 'v2-prototype' of https://github.com/0xProject/0x-monorepo into ↵fragosti2018-06-2270-647/+1798
| |\| | | | | | | | | | | | | | | | | | feature/website/mobile-friendly-onboarding
| * | | | Prevent prettier issuefragosti2018-06-211-10/+8
| | | | |
| * | | | Support mobile friendly onboarding flowsfragosti2018-06-219-87/+180
| | | | |
| * | | | Scroll to wallet for mobile onboardingfragosti2018-06-211-23/+28
| | | | |
| * | | | Change Island to use styled-componentsfragosti2018-06-201-15/+8
| | | | |
| * | | | Show start onboarding flow button on top of wallet on mobilefragosti2018-06-201-25/+29
| | | | |
* | | | | Merge pull request #742 from 0xProject/feature/website/chris-mel-about-pageFrancesco Agosti2018-06-222-2/+21
|\ \ \ \ \ | |_|/ / / |/| | | | Add Chris and Mel to about page
| * | | | Add Chris and Mel to about pagefragosti2018-06-222-2/+21
| | | | |
* | | | | Merge pull request #734 from 0xProject/feature/website/shadow-stylingBrandon Millman2018-06-224-10/+8
|\ \ \ \ \ | | | | | | | | | | | | Update Island shadow styling and use Island in more places
| * | | | | Update Island shadow styling and use Island in more placesBrandon Millman2018-06-204-10/+8
| | |/ / / | |/| | |
* | | | | Merge pull request #702 from perissology/feature/missing-sourcesLeonid Logvinov2018-06-223-3/+9
|\ \ \ \ \ | | | | | | | | | | | | [sol-cov] Only collect coverage for provided sources
| * | | | | fix linter issuesperissology2018-06-221-1/+1
| | | | | |
| * | | | | only call getLocationByOffset if source if definedperissology2018-06-211-4/+1
| | | | | |
| * | | | | Collect coverage for provided sourcesperissology2018-06-153-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When solidity generates source maps during contract compilation, the contracts are referred to by an id, which corresponds to an array index. We may not want to cover all sources that were included in a compilation, but because we use array indexes (vs. the id that is provided by solidity compiler) to map the contract to the sourceMap, the provided sourceCodes array must include the code at the correct index. This can result in empty slots in the sourceCodes array. This commit allows the coverage to only be collected for the contracts with provided sourceCode.
* | | | | | Merge pull request #692 from 0xProject/refactor/contracts/RemovedMixinSettlementRemco Bloemen2018-06-219-295/+249
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | Removed MixinSettlement
| * | | | | Set settleOrder and settleMatchedOrders to privateGreg Hysen2018-06-215-85/+104
| | | | | |
| * | | | | Removed MixinSettlement. Moved `settleOrder` into `MixinExchangeCore` and ↵Greg Hysen2018-06-219-297/+232
|/ / / / / | | | | | | | | | | | | | | | `settleMatchedOrders` into `MixinMatchOrders`
* | | | | Merge pull request #733 from 0xProject/bug/website/identicon-layoutBrandon Millman2018-06-213-3/+6
|\ \ \ \ \ | | | | | | | | | | | | Move some layout logic out of the Identicon component
| * | | | | Move some layout logic out of the Identicon componentBrandon Millman2018-06-203-3/+6
| | |_|/ / | |/| | |
* | | | | Merge pull request #736 from ↵Greg Hysen2018-06-2127-218/+857
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 0xProject/feature/contracts/MixinSignatureValidatorTests MixinSignatureValidator Test Cases + Bug Fixes
| * | | | | Migrations after rebasingGreg Hysen2018-06-2113-188/+320
| | | | | |
| * | | | | LinterGreg Hysen2018-06-211-2/+1
| | | | | |
| * | | | | Renamed constants in test wallet/validatorGreg Hysen2018-06-212-11/+11
| | | | | |
| * | | | | Few more minor #nit wording changesGreg Hysen2018-06-212-5/+5
| | | | | |
| * | | | | Changed wording of two tests #nitGreg Hysen2018-06-211-2/+2
| | | | | |
| * | | | | Minor improvements to MixinSignatureValidator testsGreg Hysen2018-06-211-14/+8
| | | | | |
| * | | | | Remove .onlyGreg Hysen2018-06-211-1/+1
| | | | | |
| * | | | | Fixed two mislabelled revert reasons + Signature Validator revert tests ↵Greg Hysen2018-06-213-8/+17
| | | | | | | | | | | | | | | | | | | | | | | | working on Geth
| * | | | | Minor style tweaksGreg Hysen2018-06-213-8/+8
| | | | | |
| * | | | | Fixed how trezor messages are signed in `signature_utils`. Now uses ↵Greg Hysen2018-06-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | `byteLength` instead of `length`.
| * | | | | Fixed `callAsync` template. Previously did not append `callData`, so we ↵Greg Hysen2018-06-211-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | couldn’t specify `from` address
| * | | | | Twenty new tests for MixinSignatureValidator. Fixed handling of unsupported ↵Greg Hysen2018-06-2110-27/+531
|/ / / / / | | | | | | | | | | | | | | | types. Fixed trezor prefix.
* | | | | Merge pull request #725 from 0xProject/feature/revert-trace-code-snippetsAlex Browne2018-06-219-39/+282
|\ \ \ \ \ | | | | | | | | | | | | Include source code snippets in revert stack traces
| * | | | | Update version for @types/solidity-parser-antlrAlex Browne2018-06-212-4/+4
| | | | | |
| * | | | | Update to match latest type definitions and other small changesAlex Browne2018-06-212-9/+11
| | | | | |
| * | | | | Update sol-cov CHANGELOG.jsonAlex Browne2018-06-211-12/+20
| | | | | |
| * | | | | Include source code snippets in revert stack tracesAlex Browne2018-06-215-21/+254
|/ / / / /
| | | | * Check revert reason in dispatcher testsFabio Berger2018-06-222-5/+13
| | | | |
| | | | * remove unused importFabio Berger2018-06-221-1/+0
| | | | |
| | | | * Test revert reasons in proxy testsFabio Berger2018-06-211-11/+23
| | | | |
| | | | * remove onlyFabio Berger2018-06-211-1/+1
| | | | |
| | | | * Check revert reasons in Authorizable testsFabio Berger2018-06-212-6/+15
| | | | |
| | | | * Change revert reason in ownable to be similar to all other revert reasonsFabio Berger2018-06-211-1/+1
| | | | |
| | | | * remove .onlyFabio Berger2018-06-211-1/+1
| | | | |
| | | | * Update yarn.lockFabio Berger2018-06-211-105/+68
| | | | |
| | | | * Update core tests to actually check revert messageFabio Berger2018-06-213-34/+97
| | | | |
| | | | * Add command to remove all node_modules, nested and hoistedFabio Berger2018-06-211-0/+1
| |_|_|/ |/| | |
* | | | Merge pull request #738 from 0xProject/fix/changelogsAlex Browne2018-06-215-0/+112
|\ \ \ \ | | | | | | | | | | Fix CHANGELOG entries changed during incorrect merge (2a4456b)
| * | | | Fix CHANGELOG entries changed during incorrect merge (2a4456b)Fabio Berger2018-06-215-0/+112
|/ / / /
* | | | Merge pull request #637 from 0xProject/bug/contracts/eip712-191-prefixJacob Evans2018-06-2011-107/+305
|\ \ \ \ | | | | | | | | | | Add missing EIP191 prefix for EIP712 and Execute Transaction
| * | | | Fix closing parens in liborderJacob Evans2018-06-201-16/+15
| | | | |
| * | | | Merge pull request #730 from ↵Jacob Evans2018-06-201-18/+22
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 0xProject/bug/contracts/eip712-191-prefix-asm-orderhash Assembler orderHash function
| | * | | | Assembler orderHash functionRemco Bloemen2018-06-191-18/+22
| |/ / / /
| * | | | Optimize and remove redundant encodePackedJacob Evans2018-06-196-18/+23
| | | | |
| * | | | Merge branch 'v2-prototype' into bug/contracts/eip712-191-prefixJacob Evans2018-06-18475-8090/+10335
| |\ \ \ \
| * | | | | Rebase and update feedbackJacob Evans2018-06-188-114/+114
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cache the domain separator data with address this Use the EIP712Types enum for types everywhere Rename EIP712 struct ExecuteTransaction to ZeroExTransaction
| * | | | | Basic EIP712 encoderJacob Evans2018-06-083-40/+61
| | | | | |
| * | | | | Update Order utils to use eip712Jacob Evans2018-06-0612-183/+121
| | | | | |
| * | | | | Add missing EIP191 prefix for EIP712Jacob Evans2018-06-0510-50/+279
| | | | | |
* | | | | | Merge pull request #717 from ↵Jacob Evans2018-06-203-17/+15
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | 0xProject/feature/contracts/erc721proxy-always-safeTransferFrom ERC721Proxy Always call safeTransferFrom
| * | | | | Update after rebaseJacob Evans2018-06-201-7/+1
| | | | | |
| * | | | | ERC721Proxy Always call safeTransferFromJacob Evans2018-06-202-10/+14
|/ / / / /
* | | | | Merge pull request #714 from 0xProject/feature/contracts/cancelOrdersUpToSenderAmir Bandeali2018-06-2016-96/+600
|\ \ \ \ \ | | | | | | | | | | | | Make cancelOrdersUpTo compatible with sender abstraction
| * | | | | Rename makerEpoch => orderEpochAmir Bandeali2018-06-2012-110/+113
| | | | | |
| * | | | | Make cancelOrdersUpTo compatible with sender abstractionAmir Bandeali2018-06-2011-47/+548
| | | | | |
* | | | | | Merge pull request #724 from 0xProject/bug/website/onboarding-improvementsFrancesco Agosti2018-06-204-7/+22
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix Wallet appearing over Drawer, and Onboarding not exiting when changing routes
| * \ \ \ \ \ Merge branch 'v2-prototype' of https://github.com/0xProject/0x-monorepo into ↵fragosti2018-06-20114-718/+1373
| |\ \ \ \ \ \ | | | |_|_|/ / | | |/| | | | | | | | | | | bug/website/onboarding-improvements
| * | | | | | Fix onboarding persisting when changing routesfragosti2018-06-192-3/+12
| | | | | | |
| * | | | | | Only elevate wallet zIndex when onboarding is in progressfragosti2018-06-193-4/+10
| | | | | | |
* | | | | | | Merge pull request #727 from ↵Francesco Agosti2018-06-201-3/+7
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | 0xProject/bug/website/fix-fetching-balances-on-wallet-login Fix bug where we do not fetch balances on wallet login
| * | | | | | Fix linting issuefragosti2018-06-191-1/+1
| | | | | | |
| * | | | | | Fix bug where we do fetch balances on wallet loginfragosti2018-06-191-3/+7
| |/ / / / /
* | | | | | Merge pull request #732 from 0xProject/feature/update-pr-templateBrandon Millman2018-06-201-16/+2
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | Update PR template
| * | | | | Update PR templateBrandon Millman2018-06-201-16/+2
| | | | | |
* | | | | | Merge pull request #723 from 0xProject/bug/website/drawer-addressBrandon Millman2018-06-206-48/+82
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Consolidate account state display message logic
| * | | | | Use Image component instead of img tagBrandon Millman2018-06-203-8/+9
| | | | | |
| * | | | | Consolidate account state messaging logicBrandon Millman2018-06-195-47/+80
| |/ / / /
* | | | | Merge pull request #706 from 0xProject/feature/website/portal-v2-analyticsFrancesco Agosti2018-06-208-15/+74
|\ \ \ \ \ | | | | | | | | | | | | Add logging to Portal V2
| * \ \ \ \ Merge branch 'v2-prototype' of https://github.com/0xProject/0x-monorepo into ↵fragosti2018-06-20166-1025/+3150
| |\ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | feature/website/portal-v2-analytics
* | | | | | Merge pull request #726 from 0xProject/bug/website/late-eth-balanceBrandon Millman2018-06-202-47/+48
|\ \ \ \ \ \ | | | | | | | | | | | | | | Check network state immediately instead of waiting for delay
| * | | | | | Check network state immediately instead of waiting for delayBrandon Millman2018-06-192-47/+48
| | | | | | |
* | | | | | | Merge pull request #683 from 0xProject/fix/contracts/memcpyRemco Bloemen2018-06-191-10/+12
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Simplified handling of source < 32 edge case
| * | | | | | | Simplified handling of source < 32 edge caseRemco Bloemen2018-06-111-10/+12
| | | | | | | |
* | | | | | | | Prettier fixFabio Berger2018-06-191-3/+5
| | | | | | | |
* | | | | | | | Merge pull request #701 from feuGeneA/v2-prototypeFabio Berger2018-06-193-5/+16
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Document use of contract_templates (take 2; based on v2-prototype, not master)
| * | | | | | | | change @0xproject/types to ethereum-typesF. Eugene Aumson2018-06-151-1/+1
| | | | | | | | |
| * | | | | | | | remove mistaken commentF. Eugene Aumson2018-06-151-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It must have been left over from when an abi-gen output was modified to be the source of this template.
| * | | | | | | | workaround for TypeScript trailing comma bugF. Eugene Aumson2018-06-151-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | before this change, TypeScript compilation of the generated contract wrapper was giving me the following errors: $ abi-gen --abis 'build/contracts/*.json' --out build/types --template contract_templates/contract.handlebars --partials 'contract_templates/partials/*.handlebars' Found 7 partial templates Found 1 ABI files Processing: Migrations... Created: build/types/migrations.ts $ tsc build/types/migrations.ts(81,23): error TS1013: A rest parameter or binding pattern may not have a trailing comma. build/types/migrations.ts(108,23): error TS1013: A rest parameter or binding pattern may not have a trailing comma. build/types/migrations.ts(130,23): error TS1013: A rest parameter or binding pattern may not have a trailing comma. build/types/migrations.ts(146,25): error TS1013: A rest parameter or binding pattern may not have a trailing comma. build/types/migrations.ts(173,25): error TS1013: A rest parameter or binding pattern may not have a trailing comma. build/types/migrations.ts(195,25): error TS1013: A rest parameter or binding pattern may not have a trailing comma. info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command. Here is the generated code around the first error: 74: public setCompleted = { 75: async sendTransactionAsync( 76: completed: BigNumber, 77: txData: Partial<TxData> = {}, 78: ): Promise<string> { 79: const self = this as any as MigrationsContract; 80: const inputAbi = self._lookupAbi('setCompleted(uint256)').inputs; 81: [completed, 82: ] = BaseContract._formatABIDataItemList(inputAbi, [completed, 83: ], BaseContract._bigNumberToString.bind(self)); All of the other errors are the same, a destructuring assignment with a single element but with a trailing comma. This is legal JavaScript but it is not allowed by the TypeScript compiler, apparently per the bug described at https://github.com/Microsoft/TypeScript/issues/24628 . While awaiting the 3.0 version of TypeScript, it's a simple enough change to have the template not append a trailing comma.
| * | | | | | | | Document contract_templatesF. Eugene Aumson2018-06-151-0/+15
| | |_|_|_|_|_|/ | |/| | | | | |
* | | | | | | | Merge pull request #728 from 0xProject/fix/post-dev-publishFabio Berger2018-06-1985-443/+921
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Update v2-prototype w/ development changes since publish
| * | | | | | | | Fix missed merge conflictsFabio Berger2018-06-193-14/+0
| | | | | | | | |
| * | | | | | | | Fix additional versions and update yarn.lockFabio Berger2018-06-199-97/+75
| | | | | | | | |
| * | | | | | | | Add OrderWatcherConfig type to 0x.js docs pageFabio Berger2018-06-191-0/+1
| | | | | | | | |
| * | | | | | | | Export OrderWatcherConfig from 0x.jsFabio Berger2018-06-191-0/+2
| | | | | | | | |
| * | | | | | | | Update all package versions to match latest published to NPMFabio Berger2018-06-1931-234/+234
| | | | | | | | |
| * | | | | | | | Update CHANGELOG's with publishes performed on development branchFabio Berger2018-06-1950-111/+622
| | |_|/ / / / / | |/| | | | | |
* | | | | | | | Merge pull request #731 from avonian/v2-prototypeLeonid Logvinov2018-06-192-4/+9
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Fix sendRawPaylodAsync for improved JSON-RPC compatibility
| * | | | | | | | Update default params in sendRawPaylodAsync for improved JSON-RPC compatibilityAra Kevonian2018-06-192-4/+9
|/ / / / / / / /
* | | | | | | | Merge pull request #659 from ↵Jacob Evans2018-06-196-129/+206
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | 0xProject/feature/contracts/split-proxy-internal-logic Split transfer implementation and AssetProxyMixin
| * | | | | | | Rebase with latest removing PROXY_ID from transferJacob Evans2018-06-185-95/+111
| | | | | | | |
| * | | | | | | Split transfer impl and AssetProxyMixinJacob Evans2018-06-184-95/+156
| | |_|_|_|_|/ | |/| | | | |
* | | | | | | Increase number of columns in relayer grid for small screensBrandon Millman2018-06-191-1/+1
| |_|_|_|/ / |/| | | | |
* | | | | | Merge pull request #716 from ↵Amir Bandeali2018-06-199-53/+331
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0xProject/feature/contracts/publicSignatureValidator Make isValidSignature public
| * | | | | | Make isValidSignature publicAmir Bandeali2018-06-199-53/+331
|/ / / / / /
* | | | | | Merge pull request #713 from 0xProject/feature/contracts/batchFillReturnAmir Bandeali2018-06-192-10/+28
|\ \ \ \ \ \ | | | | | | | | | | | | | | Make batchFill methods return FIllResults struct
| * | | | | | Make batchFill methods return FIllResults structAmir Bandeali2018-06-192-10/+28
|/ / / / / /
* | | | | | Merge pull request #722 from 0xProject/improvement/publishing-v2Fabio Berger2018-06-199-116/+321
|\ \ \ \ \ \ | | | | | | | | | | | | | | Improvements to pre-publishing checks
| * | | | | | Remove remove_tags scriptFabio Berger2018-06-191-57/+0
| | | | | | |
| * | | | | | Small fixesFabio Berger2018-06-193-10/+4
| | | | | | |
| * | | | | | Use semver package instead of getNextPatchVersionFabio Berger2018-06-192-7/+16
| | | | | | |
| * | | | | | Use semver library instead of semverUtilsFabio Berger2018-06-196-73/+17
| | | | | | |
| * | | | | | small fixesFabio Berger2018-06-191-2/+2
| | | | | | |
| * | | | | | Show all errors of a given kind at once rather then throwing after the first ↵Fabio Berger2018-06-191-10/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | instance is encountered
| * | | | | | fix package.jsonFabio Berger2018-06-191-1/+1
| | | | | | |
| * | | | | | Add more prepublish checksFabio Berger2018-06-198-39/+332
| | | | | | |