search

celo-org / optimism

Optimism is Ethereum, scaled.
https://optimism.io
MIT License
3 stars 2 forks source link

Add CallProxies for mainnet addresses on alfajores #152

Closed palango closed 3 weeks ago

palango commented 3 weeks ago

Ref https://github.com/celo-org/optimism/issues/141

Moving state accounts including storage turned out to be not supported. As we currently only need this for testnets, this offers an alternative solution instead.

This PR creates a new proxy which used call to forward calls to the implementation contract. It should only be used for proxying mainnet addresses to the respective (normal) proxy on alfajores.

@pahor167 Please give me some early feedback on this approach 🙏

pahor167 commented 3 weeks ago

Ref #141

Moving state accounts including storage turned out to be not supported. As we currently only need this for testnets, this offers an alternative solution instead.

This PR creates a new proxy which used call to forward calls to the implementation contract. It should only be used for proxying mainnet addresses to the respective (normal) proxy on alfajores.

@pahor167 Please give me some early feedback on this approach 🙏

This is a good workaround

palango commented 3 weeks ago

This doesn't work, the approach fails for payable contracts.