Skip to content
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

SuperchainERC20: Factory Spec #10873

Closed
Tracked by #10877
tynes opened this issue Apr 29, 2024 · 3 comments · Fixed by ethereum-optimism/specs#332
Closed
Tracked by #10877

SuperchainERC20: Factory Spec #10873

tynes opened this issue Apr 29, 2024 · 3 comments · Fixed by ethereum-optimism/specs#332
Assignees

Comments

@tynes
Copy link
Contributor

tynes commented Apr 29, 2024

The specs need to be updated with the SuperchainERC20Factory spec. We know some things about this factory but still need to make some decisions. This factory should make it really easy to spin up interoperable tokens across the superchain

  • Is is a predeploy, what is its address?
  • It should be create3 based to prevent coupling to bytecode
  • Where should the bytecode live that ends up being deployed? A predeploy?
  • The salt should commit to the unique erc20 properties - name, symbol, decimals
  • Should the salt also commit to a version? Leaves open to add extensions to the interface in the future
@tynes tynes transferred this issue from another repository Jun 17, 2024
@tynes tynes transferred this issue from ethereum-optimism/temp-export Jun 17, 2024
@tynes tynes added this to the Interop: Devnet 2 milestone Jun 17, 2024
@tynes
Copy link
Contributor Author

tynes commented Jun 17, 2024

There is a big design space here, its possible that we may want to limit the scope to supporting #10899

@skeletor-spaceman skeletor-spaceman self-assigned this Jun 18, 2024
@0xParticle 0xParticle moved this from TO-DO to In progress in Interoperability Jul 25, 2024
@0xParticle
Copy link
Contributor

We are discussing the base design in the following Design Doc PR. We will move into a spec once the Liquidity Migration Spec is merged.

@0xParticle 0xParticle moved this from In progress to In review in Interoperability Aug 14, 2024
@0xParticle
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants