-
Notifications
You must be signed in to change notification settings - Fork 25
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
Add deploy flag to Customize Your Own dApp #78
Conversation
docs/recipes/DevelopingOnFork.md
Outdated
``` | ||
|
||
- If you wanna use events, **MAKE SURE** the fromBlock : is > the number in console + 1, otherwise the events data will always be empty. i.e. do not capture the first block of the fork | ||
 |
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.
why dont we use embedded code here rather than a picture ?
docs/recipes/DevelopingOnFork.md
Outdated
```typescript | ||
const scaffoldConfig = { | ||
targetNetworks: [chains.mainnetFork], | ||
isFork: true, // handles forked network validation | ||
isFork: true, // handles forked network validation |
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.
this doesnt exist anymore
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.
Gm, so need to remove isFork: true, right sir ?
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.
yes
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.
please remove this file
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.
hello sir. i just push a commit that remove this file
Add deploy flag to Customize Your Own dApp & Add note for yarn chain fork
Types of change
Comments (optional)