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

Merge pull request #303 from ERCdEX/ercdex/send_transaction_return_txhash

Browse Source 
_sendTransactionAsync should return txHash string
This commit is contained in:
Leonid 2018-01-03 11:53:37 +01:00 committed by GitHub
commit 1c542c3e0d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/subproviders/src/subproviders/ledger.ts
View File

@ -238,7 +238,7 @@ export class LedgerSubprovider extends Subprovider {
this._ledgerClientIfExists = undefined;
this._connectionLock.signal();
}
private async _sendTransactionAsync(txParams: PartialTxParams): Promise<Web3.JSONRPCResponsePayload> {
private async _sendTransactionAsync(txParams: PartialTxParams): Promise<string> {
await this._nonceLock.wait();
try {
// fill in the extras
@ -252,7 +252,7 @@ export class LedgerSubprovider extends Subprovider {
};
const result = await this.emitPayloadAsync(payload);
this._nonceLock.signal();
return result;
return result.result;
} catch (err) {
this._nonceLock.signal();
throw err;