-
Notifications
You must be signed in to change notification settings - Fork 732
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
Deprecate pallet-xcm send and execute in favor of new ones #1798
Conversation
bot bench polkadot-pallet --runtime westend --subcommand xcm --pallet pallet_xcm |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3891807 was started for your command Comment |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3891808 was started for your command Comment |
@franciscoaguirre Command |
@franciscoaguirre Command |
bot bench polkadot-pallet --runtime westend --subcommand xcm --pallet pallet_xcm |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3907164 was started for your command Comment |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3907165 was started for your command Comment |
@franciscoaguirre Command |
@franciscoaguirre Command |
bot bench polkadot-pallet --runtime westend --subcommand xcm --pallet pallet_xcm |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3907686 was started for your command Comment |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3907687 was started for your command Comment |
@franciscoaguirre Command |
@franciscoaguirre Command |
bot bench polkadot-pallet --runtime westend --subcommand xcm --pallet pallet_xcm |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3920591 was started for your command Comment |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3920592 was started for your command Comment |
What is the reason behind deprecating the old extrinsics? Putting the encoded blob makes it a bit harder to use from tools like polkadot-js apps |
@franciscoaguirre Command |
@franciscoaguirre Command |
…stend --target_dir=polkadot --pallet=pallet_xcm
@franciscoaguirre Command |
bot bench polkadot-pallet --runtime westend --pallet pallet_xcm |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3946535 was started for your command Comment |
…e=westend --target_dir=polkadot --pallet=pallet_xcm
@franciscoaguirre Command |
bot bench polkadot-pallet --runtime rococo --pallet pallet_xcm |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3950317 was started for your command Comment |
…e=rococo --target_dir=polkadot --pallet=pallet_xcm
@franciscoaguirre Command |
bot help |
Here's a link to docs |
bot bench cumulus-assets --runtime asset-hub-westend --pallet pallet_xcm |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3951419 was started for your command Comment |
…=asset-hub-westend --runtime_dir=assets --target_dir=cumulus --pallet=pallet_xcm
@franciscoaguirre Command |
bot bench cumulus-assets --runtime asset-hub-kusama --pallet pallet_xcm |
@franciscoaguirre https://gitlab.parity.io/parity/mirrors/polkadot-sdk/-/jobs/3957299 was started for your command Comment |
…=asset-hub-kusama --runtime_dir=assets --target_dir=cumulus --pallet=pallet_xcm
@franciscoaguirre Command |
Closing in favor of #3749 |
… and `send_blob` (#3749) `execute` and `send` try to decode the xcm in the parameters before reaching the filter line. The new extrinsics decode only after the filter line. These should be used instead of the old ones. ## TODO - [x] Tests - [x] Generate weights - [x] Deprecation issue -> #3771 - [x] PRDoc - [x] Handle error in pallet-contracts This would make writing XCMs in PJS Apps more difficult, but here's the fix for that: polkadot-js/apps#10350. Already deployed! https://polkadot.js.org/apps/#/utilities/xcm Supersedes #1798 --------- Co-authored-by: PG Herveou <[email protected]> Co-authored-by: command-bot <> Co-authored-by: Adrian Catangiu <[email protected]>
… and `send_blob` (paritytech#3749) `execute` and `send` try to decode the xcm in the parameters before reaching the filter line. The new extrinsics decode only after the filter line. These should be used instead of the old ones. ## TODO - [x] Tests - [x] Generate weights - [x] Deprecation issue -> paritytech#3771 - [x] PRDoc - [x] Handle error in pallet-contracts This would make writing XCMs in PJS Apps more difficult, but here's the fix for that: polkadot-js/apps#10350. Already deployed! https://polkadot.js.org/apps/#/utilities/xcm Supersedes paritytech#1798 --------- Co-authored-by: PG Herveou <[email protected]> Co-authored-by: command-bot <> Co-authored-by: Adrian Catangiu <[email protected]>
Description
This PR starts the deprecation process for pallet-xcm's
send
andexecute
extrinsics, in favor of new ones.