Skip to content
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

Fixes to TypeScript definitions #1131

Merged
merged 11 commits into from
Oct 24, 2017
Merged

Fixes to TypeScript definitions #1131

merged 11 commits into from
Oct 24, 2017

Conversation

naddison36
Copy link
Contributor

Added Eth.signTransaction
Renamed accounts.privateToAccount to accounts.privateKeyToAccount
Added missing types in Contract.options
Added status to TransactionReceipt

@coveralls
Copy link

coveralls commented Oct 21, 2017

Coverage Status

Coverage remained the same at 85.538% when pulling 3daf969 on naddison36:1.0 into 037b116 on ethereum:1.0.

Copy link
Contributor

@frozeman frozeman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assuming that correct, as I don’t use type script

@frozeman frozeman merged commit 006b394 into web3:1.0 Oct 24, 2017
nachomazzara pushed a commit to nachomazzara/web3.js that referenced this pull request Jun 4, 2020
* removed privateKey from web3.eth.signTransaction and web3.eth.accounts.signTransaction documentation

* removed privateKey from web3.eth.signTransaction and web3.eth.accounts.signTransaction documentation

* added vscode to gitignore and minor doco change

* Added test to show privateKey can not be passed to accounts.signTransaction

* removed passing of private key

* Added Eth.signTransaction function
renamed accounts.privateToAccount to accounts.privateKeyToAccount
Added missing Contract.options
Added status to TransactionReceipt

* reversing changes so my forked 1.0 branch is in sync with the upstream 1.0 branch
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants