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

Merge pull request #1227 from WyseNynja/patch-1

Browse Source 
[web3-wrapper] start jsonRpcRequestId at 1, not 0
This commit is contained in:
Fabio B 2018-11-12 20:43:00 +01:00 committed by GitHub
commit add1cfd261
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
packages/order-watcher/CHANGELOG.json
View File

@ -1,4 +1,13 @@
[ [
{
"version": "2.2.3",
"changes": [
{
"note": "Start jsonRpcRequestId at 1, not 0 as 0 breaks the web3.js websocket RPC provider",
"pr": 1227
}
]
},
{ {
"timestamp": 1542028948, "timestamp": 1542028948,
"version": "2.2.2", "version": "2.2.2",

2
packages/web3-wrapper/src/web3_wrapper.ts
View File

@ -157,7 +157,7 @@ export class Web3Wrapper {
this.abiDecoder = new AbiDecoder([]); this.abiDecoder = new AbiDecoder([]);
this._provider = provider; this._provider = provider;
this._txDefaults = txDefaults || {}; this._txDefaults = txDefaults || {};
this._jsonRpcRequestId = 0; this._jsonRpcRequestId = 1;
} }
/** /**
* Get the contract defaults set to the Web3Wrapper instance * Get the contract defaults set to the Web3Wrapper instance