Skip to content

Commit

Permalink
Update chronological-issues-from-video.md
Browse files Browse the repository at this point in the history
  • Loading branch information
n4n0b1t3 authored Feb 2, 2022
1 parent d64d413 commit dbd35f5
Showing 1 changed file with 17 additions and 0 deletions.
17 changes: 17 additions & 0 deletions chronological-issues-from-video.md
Original file line number Diff line number Diff line change
Expand Up @@ -73,6 +73,23 @@ transaction = SimpleStorage.constructor().buildTransaction(

![image](https://user-images.githubusercontent.com/2119741/147293025-4dec848b-747b-4da7-9009-3f9174198b54.png)

- [3:55:09](https://youtu.be/M576WGiDBdQ?t=14109) Confusing network ID and chain ID
- In the video the network ID is copied instead of the chain id.
Whenever the terms Network ID and Chain ID are used without distinction, it should be noted that both IDs can be different for a server such as Ganache. As you can see here, Ganache can be using different IDs.

```
>>> from web3 import Web3
>>> w3 = Web3(Web3.HTTPProvider("http://127.0.0.1:8545"))
>>> w3.eth.chain_id
1337
>>> w3.net.version
'5777'
>>>
```

One possibility is setting the chain_id programmatically.
`"chainId": w3.eth.chain_id, `

## Lesson 7
- [8:06:54ish](https://youtu.be/M576WGiDBdQ?t=29214)
- In the video, we use events exclusivly to test our contracts, however, we could have also used `tx.return_value` to get the return value of a function.
Expand Down

0 comments on commit dbd35f5

Please sign in to comment.