-
Notifications
You must be signed in to change notification settings - Fork 311
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 Ultron mainnet and testnet deployments #274
Conversation
Please make sure that the contracts are deployed deterministically. This repository only allows deterministic deployments. Your addresses should equal 1 of the other two addresses in each file. |
@mmv08 Hello! We have deployed as described in Readme file using clean wallets. ![]() P.S. The networks do not support verifications for the moment. And updated the @safe-global/safe-singleton-factory": to the "1.0.15" version (only in this version our addresses do exist) I'll add more screenshots of deployments/ dir bellow for both chains (custom = 1230, custom2 = 1231) Thank you! |
Did you make sure that you're using version 1.3.0? |
@mmv08 using this release |
The documentation has a section on how to fix the error. Please read it. |
@mmv08
![]()
Cause after these steps we have got some weird results. Any thoughts or additional steps we missed?
|
no, you need to set an environment variable |
@mmv08 Thank you so much, we definitely missed that parameter in older Readme 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.
LGTM
@mmv08 Hello! Could you please also update the package? |
@softwareultron we were validating the shortnames on the packages where we are EIP-3770 compatible and we found that there may be a typo on the shortname here: https://github.com/ethereum-lists/chains/blob/master/_data/chains/eip155-1231.json It says |
Includes:
#265 and #273