-
Notifications
You must be signed in to change notification settings - Fork 5.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Release - 1.2.2 #3147
Release - 1.2.2 #3147
Conversation
@cgewecke It would be great if you could check the draft release notes I've created. I have written some phrases about all important additions and improvements. |
Looks good, just a thought for next time, it probably would of been better to of merged every PR into the Anyway looks good to merge 👍 |
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.
Sweet! Release notes LGTM.
* versions manually updated * branches to contribute too updated in root readme * build pipeline changes reverted * karmaconf added to jshintignore * release commit
Description
This release contains mostly back-ports from 2.x and some other improvements.
Added
eth.getChainId
method (Add eth.getChainId method to 1.x #3113)connected
event on subscription creation (subscription.js Emit 'connected' event with subscription.id #3028)ethereum-bloom-filters
#3137)Fixed
0
as a validfromBlock
ortoBlock
filter param (fromBlock: 0
defaults tolatest
,fromBlock: 1
works correctly ineth.getPastLogs
#1100)null
(Possible incorrect parsing of receipt status #2183)net_version
when signing txs (Invalid chainId used to sign transactions #2378)tx
object passed to functionsignTransaction
(Function 'sendSignedTransaction' changes the value of the 'to' field from 'null' to '0x' #2190)Compare View
v1.2.1 -> v1.2.2
Type of change
Checklist:
npm run build
in the root folder and tested it in the browser and with node.