Onboarding updates #226
Onboarding updates #226
Conversation
👷 Deploy request for dapper-sundae-ae0873 pending review.Visit the deploys page to approve it
|
| @@ -0,0 +1 @@ | |||
| stable | |||
There was a problem hiding this comment.
There is a minimum version of node for NextJS and I was below minimum. With this file running nvm use fixes the situation and is generally safe.
|
|
||
| export const anvilConfig = getConfig(31337); | ||
|
|
||
| export const anvilAccounts: Array<Hex> = [ |
| "0xa0ee7a142d267c1f36714e4a8f75612f20a79720", | ||
| ]; | ||
| // Wrapped into a function so we do not initialize on import, before all contracts are deployed | ||
| export function getAnvilConfig() { |
There was a problem hiding this comment.
I was running into a problem: importing chainConfig.ts file required certain contracts are already deployed. Since when we run preDeploy those contracts do not exist I created "getters" for config so that importing chainConfig.ts does not blow things up.
There was a problem hiding this comment.
I see what was happening now 🙂
There was a problem hiding this comment.
@ivanvpan nice - this looks good to me. i cloned it locally and went through the steps, all seems to work good altho i had some issues with dependencies, noted below.
when ive cloned this fork and run thru the setup steps i have to use pnpm exec next dev instead of just pnpm run dev due to an issue with the next dependency i havent been able to replicate nor resolve. i suspect this is something about my local setup, so not suggesting any changes to the PR but felt worth noting here.
|
|
||
| Because of how the distributor contract works the developer wallet needs to hold some bread before we deploy the contracts for us to be able to vote. The setup script takes care of this as well as funding the wallet with LP tokens which is needed for the LP locking feature. | ||
| 3. Because of how the distributor contract works the developer wallet needs to hold some bread before we deploy the contracts for us to be able to vote. The setup script takes care of this as well as funding the wallet with LP tokens which is needed for the LP locking feature. | ||
|
|
||
| ```sh | ||
| $ pnpm run chain:setup |
There was a problem hiding this comment.
i cloned your fork to try this whole setup from a fresh start and noticed this setup step fails if tsx is not installed globally. so a suggestion to add a note here or above in the list of general dependencies, for installing tsx
There was a problem hiding this comment.
Weird that was happening for you as tsx is added as a dev dependency 🤔
No description provided.