# Note for reviewer
Most changes are just syntax changes necessary for the new version.
Most important files should be the ones under the `xcm` folder.
# Description
Added XCMv4.
## Removed `Multi` prefix
The following types have been renamed:
- MultiLocation -> Location
- MultiAsset -> Asset
- MultiAssets -> Assets
- InteriorMultiLocation -> InteriorLocation
- MultiAssetFilter -> AssetFilter
- VersionedMultiAsset -> VersionedAsset
- WildMultiAsset -> WildAsset
- VersionedMultiLocation -> VersionedLocation
In order to fix a name conflict, the `Assets` in `xcm-executor` were
renamed to `HoldingAssets`, as they represent assets in holding.
## Removed `Abstract` asset id
It was not being used anywhere and this simplifies the code.
Now assets are just constructed as follows:
```rust
let asset: Asset = (AssetId(Location::new(1, Here)), 100u128).into();
```
No need for specifying `Concrete` anymore.
## Outcome is now a named fields struct
Instead of
```rust
pub enum Outcome {
Complete(Weight),
Incomplete(Weight, Error),
Error(Error),
}
```
we now have
```rust
pub enum Outcome {
Complete { used: Weight },
Incomplete { used: Weight, error: Error },
Error { error: Error },
}
```
## Added Reanchorable trait
Now both locations and assets implement this trait, making it easier to
reanchor both.
## New syntax for building locations and junctions
Now junctions are built using the following methods:
```rust
let location = Location {
parents: 1,
interior: [Parachain(1000), PalletInstance(50), GeneralIndex(1984)].into()
};
```
or
```rust
let location = Location::new(1, [Parachain(1000), PalletInstance(50), GeneralIndex(1984)]);
```
And they are matched like so:
```rust
match location.unpack() {
(1, [Parachain(id)]) => ...
(0, Here) => ...,
(1, [_]) => ...,
}
```
This syntax is mandatory in v4, and has been also implemented for v2 and
v3 for easier migration.
This was needed to make all sizes smaller.
# TODO
- [x] Scaffold v4
- [x] Port github.com/paritytech/polkadot/pull/7236
- [x] Remove `Multi` prefix
- [x] Remove `Abstract` asset id
---------
Co-authored-by: command-bot <>
Co-authored-by: Keith Yeung <kungfukeith11@gmail.com>
closes https://github.com/paritytech/polkadot-sdk/issues/1882
## Breaking Changes
This PR introduces a new item to `pallet_balances::Config`:
```diff
trait Config {
++ type RuntimeFreezeReasons;
}
```
This value is only used to check it against `type MaxFreeze`. A similar
check has been added for `MaxHolds` against `RuntimeHoldReasons`, which
is already given to `pallet_balances`.
In all contexts, you should pass the real `RuntimeFreezeReasons`
generated by `construct_runtime` to `type RuntimeFreezeReasons`. Passing
`()` would also work, but it would imply that the runtime uses no
freezes at all.
---------
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
* remove candidate when to invulnerable
* fix
* candidates to collators
* make parameters consistent and more reasonable
* add call to kick invulnerable candidates
* factor removal into weight
* fix: use accrue instead of add
* make set_invulnerables non-atomic
* benchmark add_invulnerable to account for candidate removal
* don't remove from candidates with set_invulnerables
* fix bounds on benchmarking
* protect against zero min invulnerables underflow
* extra event and tests
* make candidates/invulnerables self-cleaning on session change
* add integrity test
* unused imports
* make rococo-contracts have 1 collator
* add foreign assets to westmint
* add foreign assets to statemine
* use updated api for ensure origin trait
* Assets/ForeignAssets tests and fixes (#2167)
* Test for create and transfer `TrustBackedAssets` with AssetTransactor
* Test for transfer `local Currency` with AssetTransactor
* Test for create foreign assets (covers foreign relaychain currency)
* Added `ForeignFungiblesTransactor` and test for transfer `ForeignAssets` with AssetTransactor
* Removed unused `pub const Local: MultiLocation`
* Changed `ParaId -> Sibling` for `SiblingParachainConvertsVia`
* Test for create foreign assets (covers local sibling parachain assets)
* Reverted stuff for ForeignCreators from different global consensus (moved to transfer asset branch)
* Refactor `weight_limit` for `execute_xcm`
* Added test for `set_metadata` by ForeignCreator with `xcm::Transact(set_metadata)`
* Renamed `receive_teleported_asset_works` -> `receive_teleported_asset_for_native_asset_works`
* Allow `ForeignCreators` only for sibling parachains
* Unify ReservedDmpWeight/ReservedXcmpWeight usage
* Removed hack - replaced with `MatchedConvertedConcreteId`
* Refactor `ForeignCreators` to assets-common
* Add `ReceiveTeleportedAsset` test
* Change test - `Utility::batch` -> Multiple `xcm::Transact`
* Reusing the same deposits as for TrustBackedAssets
* missing `try_successful_origin` ?
* Finished `ForeignAssets` for westmint (converter, FungiblesApi, tests)
* Refactoring tests - receive_teleported_asset_for_native_asset_works
* ForeignAssets for statemine + refactored `receive_teleported_asset_from_foreign_creator_works`
* Add `ForeignAssets` to statemine `FungiblesApi`
* Add `asset_transactor_transfer_with_local_consensus_currency_works` to all runtimes
* Added `asset_transactor_transfer_with_trust_backed_assets_works` test
* Added `asset_transactor_transfer_with_foreign_assets_works`
* Fix `missing `try_successful_origin` in implementation`
* Added `create_and_manage_foreign_assets_for_local_consensus_parachain_assets_works`
* Added `ExpectTransactStatus` check
* Small rename
* Extended `test_assets_balances_api_works` with ForeignAssets for `statemine`
* PR fixes
* Update parachains/runtimes/assets/test-utils/src/test_cases.rs
---------
Co-authored-by: parity-processbot <>
Co-authored-by: joe petrowski <25483142+joepetrowski@users.noreply.github.com>
* Added `StartsWithExplicitGlobalConsensus` to ignores (#2338)
* Update parachains/runtimes/assets/common/src/lib.rs
Co-authored-by: Gavin Wood <gavin@parity.io>
* include mint and burn in SafeCallFilter
* include mint and burn in SafeCallFilter (statemine)
* clarify doc
* Fix compilation (moved trait `InspectMetadata`)
* Fix test
* Extended test for `teleport` from/to relaychain + `CheckingAccount` (Part I)
* Extended test for `teleport` from/to foreign parachain + `CheckingAccount` (Part II)
* Fixed TODO - `NonLocal` for `ForeignAssets`
* Changed `NonLocal` to `NoChecking`
* Fix weight in test
---------
Co-authored-by: parity-processbot <>
Co-authored-by: muharem <ismailov.m.h@gmail.com>
Co-authored-by: Branislav Kontur <bkontur@gmail.com>
Co-authored-by: Gavin Wood <gavin@parity.io>
* Fix APIs
* Reflect API changes
* Everything builds
* Fixes
* Fixes
* Update Cargo.toml
* Fixes
* Fixes
* No networks use freezes/holds
* update lockfile for {"polkadot", "substrate"}
* Fix test
ED cannot be zero anymore.
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
* Fix test
---------
Signed-off-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
Co-authored-by: parity-processbot <>
Co-authored-by: Oliver Tale-Yazdi <oliver.tale-yazdi@parity.io>
* Use primitives reexported from `polkadot_primitives` crate root
* restart CI
* Fixes after merge
* update lockfile for {"polkadot", "substrate"}
Co-authored-by: parity-processbot <>
Francisco Aguirre