-
Notifications
You must be signed in to change notification settings - Fork 1
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
Garage: add multi chain support #879
Conversation
RIG-51 Add support for connecting to multiple chains
The admin page needs to be able to write to tables on filecoin and the contracts that get called for training/piloting/parking/transferring rigs are on eth mainnet, so we need to support for switching the connected provider/wallet between networks depending on the context. This will also be needed for the voting feature since the registry contract & related tables will be deployed to a chain that is not eth mainnet (most likely filecoin?) |
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.
nice! lgtm
we need to wrap the .run()-call in a try/catch so that we can catch exceptions thrown if the user rejects the transaction
16ac3d4
to
7b10568
Compare
sweet! will need a re-approval since the base changed to staging |
Adds support to the garage for being connected to multiple chains. Needed when using the admin page to write to tables that are not on mainnet, and will be needed for voting
NOTE: The base of this PR is #878 and not staging, because it depends on the work in 878. Will change base as soon as that one is merged
Closes RIG-51