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

Migration to CosmWasm v2 #71

Closed
CyberHoward opened this issue Sep 4, 2024 · 1 comment
Closed

Migration to CosmWasm v2 #71

CyberHoward opened this issue Sep 4, 2024 · 1 comment

Comments

@CyberHoward
Copy link

Description

CosmWasm 2.0 has been slowly gaining adoption throughout the ecosystem. As critical infrastructure, Polytone should be up-to-date with this latest version, allowing devs who are using CW 2.0 to make use of Polytone.

Issue Scope

While working to upgrade our own contracts we've identified two major changes that should be made to Polytone.

  1. SubMsgResponse's data field is deprecated.
    This field is currently used to retrieve and cache execution results before returning them in the Ack.
  2. Use Reply's Payload instead of state-caching.
    A new Payload field was added to the SubMsg which allows developers to pass data to the reply endpoint without needing to store it to state. Applying this to Polytone would reduce the gas usage and simplify the contract's implementation. It can also be used to cache the execution responses, further reducing gas-costs.

Any other issues encountered over the software's life should preferably also be addressed in this update.

Update Rollout

Polytone's canonical deployments are non-migratable. As a result rolling out this update will require a coordinated governance proposal on each deployed chain to migrate the contracts.

Open questions

  • Will migrating one side of the polytone channel cause the channel to break until the other side is migrated?
  • Are there any other issues that should be linked to this upgrade?
@CyberHoward
Copy link
Author

Closed in favor of cosmos/ibc-apps#220

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

No branches or pull requests

1 participant