This repository has been archived by the owner on Jul 9, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 465
[asset-buyer] Create new AssetBuyer class #1037
Merged
Merged
Changes from 27 commits
Commits
Show all changes
43 commits
Select commit
Hold shift + click to select a range
8687b95
Add getForwarderHelperForMakerAssetDataAsync method to forwarderHelpe…
BMillman19 8d61327
Remove taker address as param and filter out non-open orders
BMillman19 c0a14a4
Catch standard relayer api errors
BMillman19 90674d9
Various clean up
BMillman19 91702bb
Move packages/forwarder-helper into packages/asset-buyer
BMillman19 8da7d39
Update readme, changelog, and package.json with rename
BMillman19 7b46cef
Create initial AssetBuyer class
BMillman19 f1a22e9
Flesh out the AssetBuyer class
BMillman19 190bf25
Implement StandardRelayerAPIOrderFetcher
BMillman19 4e59be9
Implement ProvidedOrderFetcher
BMillman19 dfe58e4
Delete old test
BMillman19 c692115
Fix lint errors
BMillman19 d57619b
Export the order fetchers
BMillman19 af40989
Add factory method on AssetBuyer for provided orders
BMillman19 9ec2b5a
Add factory method on AssetBuyer for specific token address
BMillman19 a22d2dc
Add factory method for specific assetData to buy and add comments
BMillman19 f5dbf21
Merge branch 'development' of https://github.com/0xProject/0x-monorep…
fragosti 65f709d
Move all constants to own file
fragosti 38afd10
Fix order expired bug
fragosti b947609
Restrict orders field of AssetBuyerOrdersAndFillableAmounts to Signed…
fragosti 0003666
make the slippage percentage customizable by integrator
fragosti fd4b4f8
Make ForwaderWrapper take in a number for feePercentage instead of Bi…
fragosti 93f7e33
Add isValidPercentage assert
fragosti 60e2dfd
Calculate min and max rates in buy quote
fragosti 14345ab
Rename forceOrderRefresh to shouldForceOrderRefresh
fragosti a03b1d4
Move ObjectMap to shared types
fragosti a44f77a
Implement StandardRelayerAPIAssetBuyerManager
fragosti 3238925
Add better description to README
fragosti d23487b
Update descriptions for methods on AssetBuyer
fragosti 03e18ff
Rename OrderFetcher to OrderProvider and other small improvements
fragosti 35c324f
Add utility to get assetDatas
fragosti fcca63a
Move some logic to order utils
fragosti 97150cf
Add labels to TODOs
fragosti 8dea48e
Renamed orderFetcherResponseProcessor to orderProviderResponseProcessoer
fragosti c64dcec
More renaming
fragosti c48cf3a
Make asset buyer map private
fragosti 1bfaefb
Add order provider response validation
fragosti 5e84e96
Small typos
fragosti 7a53766
Merge branch 'development' of https://github.com/0xProject/0x-monorep…
fragosti e4b664b
Add expiry buffer
fragosti 7a43e19
Improve README
fragosti f49b231
Merge branch 'development' of https://github.com/0xProject/0x-monorep…
fragosti d8d1c98
Upgrade all deps
fragosti File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
File renamed without changes.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
[ | ||
{ | ||
"version": "1.0.0-rc.1", | ||
"changes": [ | ||
{ | ||
"note": "Init" | ||
} | ||
] | ||
} | ||
] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,25 +1,25 @@ | ||
## @0xproject/forwarder-helper | ||
## @0xproject/asset-buyer | ||
|
||
Provides convenience objects to help work with the Forwarder Contract | ||
Convenience package for buying assets | ||
|
||
### Read the [Documentation](https://0xproject.com/docs/forwarder-helper). | ||
### Read the [Documentation](https://0xproject.com/docs/asset-buyer). | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Will there really be a documentation page for this package? If "eventually" is the answer, let's remove this line for now. |
||
|
||
## Installation | ||
|
||
```bash | ||
yarn add @0xproject/forwarder-helper | ||
yarn add @0xproject/asset-buyer | ||
``` | ||
|
||
**Import** | ||
|
||
```typescript | ||
import { forwarderHelperFactory } from '@0xproject/forwarder-helper'; | ||
import { AssetBuyer } from '@0xproject/asset-buyer'; | ||
``` | ||
|
||
or | ||
|
||
```javascript | ||
var forwarderHelperFactory = require('@0xproject/forwarder-helper').forwarderHelperFactory; | ||
var AssetBuyer = require('@0xproject/asset-buyer').AssetBuyer; | ||
``` | ||
|
||
If your project is in [TypeScript](https://www.typescriptlang.org/), add the following to your `tsconfig.json`: | ||
|
@@ -55,13 +55,13 @@ yarn install | |
To build this package and all other monorepo packages that it depends on, run the following from the monorepo root directory: | ||
|
||
```bash | ||
PKG=@0xproject/forwarder-helper yarn build | ||
PKG=@0xproject/asset-buyer yarn build | ||
``` | ||
|
||
Or continuously rebuild on change: | ||
|
||
```bash | ||
PKG=@0xproject/forwarder-helper yarn watch | ||
PKG=@0xproject/asset-buyer yarn watch | ||
``` | ||
|
||
### Clean | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,10 +1,10 @@ | ||
{ | ||
"name": "@0xproject/forwarder-helper", | ||
"version": "1.0.2", | ||
"name": "@0xproject/asset-buyer", | ||
"version": "1.0.0-rc.1", | ||
"engines": { | ||
"node": ">=6.12" | ||
}, | ||
"description": "Convenience object for working with the forwarder contract", | ||
"description": "Convenience package for buying assets", | ||
"main": "lib/src/index.js", | ||
"types": "lib/src/index.d.ts", | ||
"scripts": { | ||
|
@@ -34,21 +34,26 @@ | |
"bugs": { | ||
"url": "https://github.com/0xProject/0x-monorepo/issues" | ||
}, | ||
"homepage": "https://github.com/0xProject/0x-monorepo/packages/forwarder-helper/README.md", | ||
"homepage": "https://github.com/0xProject/0x-monorepo/packages/asset-buyer/README.md", | ||
"dependencies": { | ||
"@0xproject/assert": "^1.0.8", | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There has been a publish in the mean-time. This should be |
||
"@0xproject/connect": "^2.0.0", | ||
"@0xproject/contract-wrappers": "^1.0.1", | ||
"@0xproject/json-schemas": "^1.0.1", | ||
"@0xproject/order-utils": "^1.0.1", | ||
"@0xproject/subproviders": "^2.0.2", | ||
"@0xproject/types": "^1.0.1", | ||
"@0xproject/typescript-typings": "^2.0.0", | ||
"@0xproject/utils": "^1.0.8", | ||
"@types/node": "^8.0.53", | ||
"@0xproject/web3-wrapper": "^2.0.2", | ||
"ethereum-types": "^1.0.6", | ||
"lodash": "^4.17.10" | ||
}, | ||
"devDependencies": { | ||
"@0xproject/tslint-config": "^1.0.7", | ||
"@types/lodash": "^4.14.116", | ||
"@types/mocha": "^2.2.42", | ||
"@types/node": "^8.0.53", | ||
"chai": "^4.0.1", | ||
"chai-as-promised": "^7.1.0", | ||
"chai-bignumber": "^2.0.1", | ||
|
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please provide a more elaborate description, this is too vague.