-
Notifications
You must be signed in to change notification settings - Fork 79
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
removed a setup action initializer #147
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
ab8ac47
to
844bcaf
Compare
We absolutely need multicall here. It's used by the team to setup mints on the contract. |
We could move this logic to the factory and set the initial admin as the factory then create the correct admin roles and remove the factory role at the end? or allow the factory contract to override checks for permissions with an override branch. |
this is a draft and was demoed as part of the graphite workshop...will delete |
* feat: Boosted Minter factory and implementation * remove rpc endpoints (#203) * feat: Deploy Scripts (#128) * feat: Withdraw Gas for frame minter (#132) * multichain deploy scripts (#143) * feat: upgradeable factory (#144) feat: upgradeable factory [fix] fees multichain deploy * update fee and upgrade (#145) * fix: upgrade script (#147) * Update upgrades to standard pattern (#154) Update deployment scripts to use standard multichain patterns. --------- Co-authored-by: Dan Oved <[email protected]> Co-authored-by: Iain Nash <[email protected]> Co-authored-by: Rohan Kulkarni <[email protected]>
No description provided.