-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat: ui skeleton, deployment scripts and addr of dummy factory on goerli-optimism #5
Conversation
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.
was able to deploy via local ui!!
next steps i see:
- grab config from deployed module
- display current config
- alter current config
- connect using actual safe
- deploy module and
enableModule
on safe
@@ -5,6 +5,7 @@ | |||
"scripts": { | |||
"anvil": "source .env && anvil --fork-url $ANVIL_FORK_URL --chain-id $ANVIL_CHAIN_ID", | |||
"deploy": "source .env && forge create contracts/src/ModuleFactory.sol:ModuleFactory --rpc-url $FORGE_RPC_URL --private-key $FORGE_PRIVATE_KEY --etherscan-api-key $ETHERSCAN_API_KEY --verify", | |||
"deploy:module": "source .env && forge create contracts/src/modules/DummyModule.sol:DummyModule --constructor-args 0x99999315a1AE71633E95fB35601B59180eC8D61d 604800 --rpc-url $FORGE_RPC_URL --private-key $FORGE_PRIVATE_KEY --etherscan-api-key $ETHERSCAN_API_KEY --verify", |
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.
would suggest not giving the singleton module actual address or cadence values
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.
added that cmd to have one deployed to allow all the one that we get deployed to be auto-verified, the args does not affect that process
README.md
Outdated
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.
maybe keep/write our own readme?
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.
updated 161491e. left only a ref to the starter package for now
Goerli Opti dummy factory addr is plugged here
Test:
npm run dev
Sample of txs thru ui: