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 to polkadot-sdk commit 4a400dc1866f11707331fb6408df1055d0f42a70 #1943

Open
wants to merge 31 commits into
base: master
Choose a base branch
from

Conversation

cmichi
Copy link
Collaborator

@cmichi cmichi commented Feb 19, 2025

What this PR does:

  • Sync with polkadot-sdk/4a400dc1866f11707331fb6408df1055d0f42a70
  • Remove contract-specific events (CodeStored, Instantiated, etc.), as they were removed in pallet-revive.
  • Derive address of deployed contract in cargo-contract instead of fetching it from an event (no longer possible).
  • Some leftovers: Remove the generic Hash where pallet-revive uses a fixed H256.
  • Remove debug_message, as it was removed in pallet-revive. We'll add the possibility to trace extrinsics in a follow-up, that's the pallet-revive debug functionality which replaces debug_message.

Follow-ups:

  • Display CodeInfoOf for cargo contract upload and cargo contract info (if not already).
  • Add a flag --trace to all sub-commands that issue extrinsics.
  • Add cargo contract map/unmap.

@cmichi cmichi requested a review from ascjones as a code owner February 19, 2025 15:04
@cmichi cmichi removed the request for review from ascjones February 19, 2025 15:04
@cmichi cmichi force-pushed the cmichi-upgrade-to-latest-polkadot-sdk-feb-11 branch from 724f76b to 9a5a24e Compare February 20, 2025 12:56
@cmichi cmichi changed the title Upgrade to polkadot-sdk commit 5f6c8e8fd20f19de0b592ef2413e934e97c9bc68 Upgrade to polkadot-sdk commit b9b73eb2758edc8af140fea60fdc51f890eefc48 Feb 21, 2025
@cmichi cmichi changed the title Upgrade to polkadot-sdk commit b9b73eb2758edc8af140fea60fdc51f890eefc48 Upgrade to polkadot-sdk commit 4a400dc1866f11707331fb6408df1055d0f42a70 Feb 26, 2025
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.

1 participant