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

Fixed typo in example for interacting with contract #1615

Closed
wants to merge 1 commit into from
Closed

Fixed typo in example for interacting with contract #1615

wants to merge 1 commit into from

Conversation

cryptopapi997
Copy link

@cryptopapi997 cryptopapi997 commented Mar 28, 2020

What was wrong?

The web3 instance is called w3 in the example, which means fetching the nonce should be done using w3.eth.getTransactionCount('Your_Wallet_Address'). In the example, however, it is web3.eth.getTransactionCount('Your_Wallet_Address') which would try fetching it with the name of the imported library (which doesn't work).

Related to Issue #

How was it fixed?

Changed the name of the variable

Todo:

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

The web3 instance is called w3 in the example, which means fetching the nonce should be done using w3.eth.getTransactionCount('Your_Wallet_Address'). In the example, however, it is web3.eth.getTransactionCount('Your_Wallet_Address') which would try fetching it with the name of the imported library.
Copy link
Collaborator

@kclowes kclowes left a comment

Choose a reason for hiding this comment

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

👍 Nice catch, thank you!

@kclowes
Copy link
Collaborator

kclowes commented Mar 30, 2020

A quick glance didn't reveal a way for me to update this PR to add the newsfragment since it looks like the fork was deleted. Closing this PR in favor of #1616

@kclowes kclowes closed this Mar 30, 2020
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.

2 participants