crowetic/protocol
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #1568 from smithki/fix/contract-wrappers/wrong-default-address-for-order-validator-wrapper

Browse Source 
Use correct "orderValidator" address in OrderValidatorWrapper constructor
This commit is contained in:
Amir Bandeali
2019-02-01 16:04:30 -08:00
committed by GitHub
parent 42c13ecafa 9fd372da51
commit 069c9fd4a6
2 changed files with 10 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
packages/contract-wrappers/CHANGELOG.json
View File

@@ -1,4 +1,13 @@
[ [
{
"version": "6.0.1",
"changes": [
{
"note": "Fix OrderValidatorWrapper constructor to use the correct address",
"pr": 1568
}
]
},
{ {
"version": "6.0.0", "version": "6.0.0",
"changes": [ "changes": [

2
packages/contract-wrappers/src/contract_wrappers/order_validator_wrapper.ts
View File

@@ -29,7 +29,7 @@ export class OrderValidatorWrapper extends ContractWrapper {
*/ */
constructor(web3Wrapper: Web3Wrapper, networkId: number, address?: string) { constructor(web3Wrapper: Web3Wrapper, networkId: number, address?: string) {
super(web3Wrapper, networkId); super(web3Wrapper, networkId);
this.address = _.isUndefined(address) ? _getDefaultContractAddresses(networkId).exchange : address; this.address = _.isUndefined(address) ? _getDefaultContractAddresses(networkId).orderValidator : address;
} }
/** /**
* Get an object conforming to OrderAndTraderInfo containing on-chain information of the provided order and address * Get an object conforming to OrderAndTraderInfo containing on-chain information of the provided order and address