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

Fix for typo in constructor and gas price to apply to meta transaction.

Browse Source
This commit is contained in:
Joshua Richardson 2019-12-01 16:14:31 +10:30
parent 3642e96154
commit 5bded1946e
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
contracts/coordinator/src/client/index.ts
View File

@ -103,7 +103,7 @@ export class CoordinatorClient {
this.chainId = chainId; this.chainId = chainId;
const contractAddresses = getContractAddressesForChainOrThrow(this.chainId); const contractAddresses = getContractAddressesForChainOrThrow(this.chainId);
this.address = address === undefined ? contractAddresses.coordinator : address; this.address = address === undefined ? contractAddresses.coordinator : address;
this.exchangeAddress = exchangeAddress === undefined ? contractAddresses.coordinator : exchangeAddress; this.exchangeAddress = exchangeAddress === undefined ? contractAddresses.exchange : exchangeAddress;
this.registryAddress = registryAddress === undefined ? contractAddresses.coordinatorRegistry : registryAddress; this.registryAddress = registryAddress === undefined ? contractAddresses.coordinatorRegistry : registryAddress;
this._web3Wrapper = new Web3Wrapper(provider); this._web3Wrapper = new Web3Wrapper(provider);
this._txDefaults = { ...txDefaults, ...DEFAULT_TX_DATA }; this._txDefaults = { ...txDefaults, ...DEFAULT_TX_DATA };
@ -624,7 +624,7 @@ export class CoordinatorClient {
const data = (this._exchangeInstance as any)[exchangeFn](...args).getABIEncodedTransactionData(); const data = (this._exchangeInstance as any)[exchangeFn](...args).getABIEncodedTransactionData();
// generate and sign a ZeroExTransaction // generate and sign a ZeroExTransaction
const signedZrxTx = await this._generateSignedZeroExTransactionAsync(data, txData.from); const signedZrxTx = await this._generateSignedZeroExTransactionAsync(data, txData.from, txData.gasPrice);
// get approval signatures from registered coordinator operators // get approval signatures from registered coordinator operators
const approvalSignatures = await this._getApprovalsAsync(signedZrxTx, ordersNeedingApprovals, txData.from); const approvalSignatures = await this._getApprovalsAsync(signedZrxTx, ordersNeedingApprovals, txData.from);
@ -643,6 +643,7 @@ export class CoordinatorClient {
private async _generateSignedZeroExTransactionAsync( private async _generateSignedZeroExTransactionAsync(
data: string, data: string,
signerAddress: string, signerAddress: string,
gasPrice?: BigNumber | string | number,
): Promise<SignedZeroExTransaction> { ): Promise<SignedZeroExTransaction> {
const transaction: ZeroExTransaction = { const transaction: ZeroExTransaction = {
salt: generatePseudoRandomSalt(), salt: generatePseudoRandomSalt(),
@ -657,7 +658,7 @@ export class CoordinatorClient {
DEFAULT_APPROVAL_EXPIRATION_TIME_SECONDS - DEFAULT_APPROVAL_EXPIRATION_TIME_SECONDS -
DEFAULT_EXPIRATION_TIME_BUFFER_SECONDS, DEFAULT_EXPIRATION_TIME_BUFFER_SECONDS,
), ),
gasPrice: new BigNumber(1), gasPrice: gasPrice ? new BigNumber(gasPrice) : new BigNumber(1),
}; };
const signedZrxTx = await signatureUtils.ecSignTransactionAsync( const signedZrxTx = await signatureUtils.ecSignTransactionAsync(
this._web3Wrapper.getProvider(), this._web3Wrapper.getProvider(),