Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: update cosmwasm to 2.0 #NTRN-302 #147

Merged
merged 9 commits into from
Jul 3, 2024
Merged

Conversation

joldie777
Copy link
Contributor

@joldie777 joldie777 commented Jun 7, 2024

Task: https://hadronlabs.atlassian.net/browse/NTRN-302

Related PRs:
neutron-org/neutron-dev-contracts#54

List of changes:

  1. Due to this update, any uses of the backtraces feature have been removed.
  2. Due to this update, new is now explicitly called to create Binary instance from an inner value.
  3. Due to this update, newly exposed module is used to access the update_staking function directly.
  4. Due to this update, calls to mock_info has been replaced with message_info.
  5. Due to non_exhaustive attribute, the Delegation struct is no longer accessible from the outside of the crate where it defined. So StdDelegation struct was created as the copy of the cosmwasm Delegation to be used instead.
  6. Due to deprecation, the uses of the data field have been replaced with msg_responses. Since msg_responses contains protobuf-encoded messages, the JSON deserialization methods have been replaced by decode_message_response with corresponding replacement of the response types: MsgIbcTransferResponse for MsgTransferResponse and MsgSubmitTxResponse for MsgSubmitTxResponse

@joldie777 joldie777 changed the base branch from feat/sdk-50 to main June 13, 2024 10:09
@joldie777 joldie777 changed the title feat: update cosmwasm to 2.0 feat: update cosmwasm to 2.0 #NTRN-302 Jun 13, 2024
contracts/ibc_transfer/src/contract.rs Outdated Show resolved Hide resolved
contracts/ibc_transfer/src/contract.rs Outdated Show resolved Hide resolved
contracts/neutron_interchain_txs/src/contract.rs Outdated Show resolved Hide resolved
@quasisamurai
Copy link
Contributor

please merge main, i see diff locally

pr0n00gler
pr0n00gler previously approved these changes Jun 26, 2024
sotnikov-s
sotnikov-s previously approved these changes Jul 1, 2024
Copy link
Contributor

@sotnikov-s sotnikov-s left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove several backtraces occurrences in the project. the rest LGTM

@joldie777 joldie777 dismissed stale reviews from sotnikov-s and pr0n00gler via ffad52e July 1, 2024 21:41
sotnikov-s
sotnikov-s previously approved these changes Jul 2, 2024
@pr0n00gler pr0n00gler merged commit 9aac376 into main Jul 3, 2024
10 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants