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

Upgrade subxt to 0.32.0 #1352

Merged
merged 22 commits into from
Oct 6, 2023
Merged

Upgrade subxt to 0.32.0 #1352

merged 22 commits into from
Oct 6, 2023

Conversation

ascjones
Copy link
Collaborator

@ascjones ascjones commented Sep 29, 2023

Summary

  • [n] y/n | Does it introduce breaking changes?
  • [n] y/n | Is it dependent on the specific version of ink or pallet-contracts?

Update to latest subxt release paritytech/subxt#1178

Description

Main changes are to the storage api for fetching all contract infos, and using the legacy rpc client to fetch system properties for token metadata.

best block vs finalized block

The latest release of subxt has changed the default of at_latest for reading storage changes to the latest finalized block, where previously it was the best block. Because cargo-contract extrinsics feature is intended primarily for use during development, and because we use substrate-contracts-node for testing (which does not have finalization), in this PR I have changed all storage reads to read for the latest best block to maintain the current behaviour.

In the future we probably want to add the ability to read from the latest finalized flag, either by default or by way of a flag, see also #672

@ascjones ascjones marked this pull request as ready for review October 2, 2023 17:22
@ascjones ascjones changed the title Bump subxt to 0.32.0 Upgrade subxt to 0.32.0 Oct 3, 2023
crates/extrinsics/src/lib.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/lib.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/lib.rs Show resolved Hide resolved
crates/extrinsics/src/call.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/call.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/lib.rs Outdated Show resolved Hide resolved
crates/cargo-contract/src/cmd/info.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/call.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/lib.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/lib.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/lib.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/lib.rs Outdated Show resolved Hide resolved
crates/extrinsics/src/lib.rs Outdated Show resolved Hide resolved
ascjones and others added 2 commits October 5, 2023 17:26
Bumps [substrate-build-script-utils](https://github.com/paritytech/polkadot-sdk) from 7.0.0 to 8.0.0.
- [Release notes](https://github.com/paritytech/polkadot-sdk/releases)
- [Commits](https://github.com/paritytech/polkadot-sdk/commits)

---
updated-dependencies:
- dependency-name: substrate-build-script-utils
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@ascjones ascjones mentioned this pull request Oct 6, 2023
5 tasks
@ascjones ascjones merged commit 3448659 into master Oct 6, 2023
11 checks passed
@ascjones ascjones deleted the aj/update-subxt branch October 6, 2023 09:43
@smiasojed smiasojed mentioned this pull request Nov 30, 2023
@smiasojed smiasojed mentioned this pull request Mar 4, 2024
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

Successfully merging this pull request may close these issues.

2 participants