From 5ed97fd59ac9b956e11dfd41068383ef2fc83b63 Mon Sep 17 00:00:00 2001 From: Shawn Tabrizi Date: Tue, 14 Jun 2022 13:13:11 -0400 Subject: [PATCH] Lower all Assets / NFT deposits by 10x (#1332) * lower all deposits by 10x * undo rococo stuff * Apply suggestions from code review * update asset deposits * align statemint deposit ratios Co-authored-by: joepetrowski --- .../parachains/runtimes/assets/statemine/src/constants.rs | 4 ++-- cumulus/parachains/runtimes/assets/statemine/src/lib.rs | 6 +++--- .../parachains/runtimes/assets/statemint/src/constants.rs | 4 ++-- cumulus/parachains/runtimes/assets/statemint/src/lib.rs | 6 +++--- .../parachains/runtimes/assets/westmint/src/constants.rs | 4 ++-- cumulus/parachains/runtimes/assets/westmint/src/lib.rs | 8 ++++---- .../runtimes/contracts/contracts-rococo/src/constants.rs | 4 ++-- 7 files changed, 18 insertions(+), 18 deletions(-) diff --git a/cumulus/parachains/runtimes/assets/statemine/src/constants.rs b/cumulus/parachains/runtimes/assets/statemine/src/constants.rs index 8661ec78a3..fabc09d017 100644 --- a/cumulus/parachains/runtimes/assets/statemine/src/constants.rs +++ b/cumulus/parachains/runtimes/assets/statemine/src/constants.rs @@ -26,8 +26,8 @@ pub mod currency { pub const MILLICENTS: Balance = constants::currency::MILLICENTS; pub const fn deposit(items: u32, bytes: u32) -> Balance { - // map to 1/10 of what the kusama relay chain charges (v9020) - constants::currency::deposit(items, bytes) / 10 + // map to 1/100 of what the kusama relay chain charges (v9020) + constants::currency::deposit(items, bytes) / 100 } } diff --git a/cumulus/parachains/runtimes/assets/statemine/src/lib.rs b/cumulus/parachains/runtimes/assets/statemine/src/lib.rs index b9e739934c..49834bcd98 100644 --- a/cumulus/parachains/runtimes/assets/statemine/src/lib.rs +++ b/cumulus/parachains/runtimes/assets/statemine/src/lib.rs @@ -210,7 +210,7 @@ impl pallet_transaction_payment::Config for Runtime { } parameter_types! { - pub const AssetDeposit: Balance = UNITS; // 1 UNIT deposit to create asset + pub const AssetDeposit: Balance = UNITS / 10; // 1 / 10 UNITS deposit to create asset pub const AssetAccountDeposit: Balance = deposit(1, 16); pub const ApprovalDeposit: Balance = EXISTENTIAL_DEPOSIT; pub const AssetsStringLimit: u32 = 50; @@ -507,8 +507,8 @@ impl pallet_asset_tx_payment::Config for Runtime { } parameter_types! { - pub const CollectionDeposit: Balance = UNITS; // 1 UNIT deposit to create asset class - pub const ItemDeposit: Balance = UNITS / 100; // 1/100 UNIT deposit to create asset instance + pub const CollectionDeposit: Balance = UNITS / 10; // 1 / 10 UNIT deposit to create asset class + pub const ItemDeposit: Balance = UNITS / 1_000; // 1 / 1000 UNIT deposit to create asset instance pub const KeyLimit: u32 = 32; // Max 32 bytes per key pub const ValueLimit: u32 = 64; // Max 64 bytes per value pub const UniquesMetadataDepositBase: Balance = deposit(1, 129); diff --git a/cumulus/parachains/runtimes/assets/statemint/src/constants.rs b/cumulus/parachains/runtimes/assets/statemint/src/constants.rs index e6886fcb75..17624fded9 100644 --- a/cumulus/parachains/runtimes/assets/statemint/src/constants.rs +++ b/cumulus/parachains/runtimes/assets/statemint/src/constants.rs @@ -26,8 +26,8 @@ pub mod currency { pub const MILLICENTS: Balance = constants::currency::MILLICENTS; pub const fn deposit(items: u32, bytes: u32) -> Balance { - // 1/10 of Polkadot v9010 - constants::currency::deposit(items, bytes) / 10 + // 1/100 of Polkadot v9010 + constants::currency::deposit(items, bytes) / 100 } } diff --git a/cumulus/parachains/runtimes/assets/statemint/src/lib.rs b/cumulus/parachains/runtimes/assets/statemint/src/lib.rs index cebdce8701..e7100dfb88 100644 --- a/cumulus/parachains/runtimes/assets/statemint/src/lib.rs +++ b/cumulus/parachains/runtimes/assets/statemint/src/lib.rs @@ -240,7 +240,7 @@ impl pallet_transaction_payment::Config for Runtime { } parameter_types! { - pub const AssetDeposit: Balance = 100 * DOLLARS; // 100 DOLLARS deposit to create asset + pub const AssetDeposit: Balance = 10 * UNITS; // 10 UNITS deposit to create fungible asset class pub const AssetAccountDeposit: Balance = deposit(1, 16); pub const ApprovalDeposit: Balance = EXISTENTIAL_DEPOSIT; pub const AssetsStringLimit: u32 = 50; @@ -537,8 +537,8 @@ impl pallet_asset_tx_payment::Config for Runtime { } parameter_types! { - pub const CollectionDeposit: Balance = UNITS; // 1 UNIT deposit to create asset class - pub const ItemDeposit: Balance = UNITS / 100; // 1/100 UNIT deposit to create asset instance + pub const CollectionDeposit: Balance = 10 * UNITS; // 10 UNIT deposit to create uniques class + pub const ItemDeposit: Balance = UNITS / 100; // 1 / 100 UNIT deposit to create uniques instance pub const KeyLimit: u32 = 32; // Max 32 bytes per key pub const ValueLimit: u32 = 64; // Max 64 bytes per value pub const UniquesMetadataDepositBase: Balance = deposit(1, 129); diff --git a/cumulus/parachains/runtimes/assets/westmint/src/constants.rs b/cumulus/parachains/runtimes/assets/westmint/src/constants.rs index f4f5054ce4..032ed5c1f4 100644 --- a/cumulus/parachains/runtimes/assets/westmint/src/constants.rs +++ b/cumulus/parachains/runtimes/assets/westmint/src/constants.rs @@ -26,8 +26,8 @@ pub mod currency { pub const GRAND: Balance = constants::currency::GRAND; pub const fn deposit(items: u32, bytes: u32) -> Balance { - // 1/10 of Westend testnet - constants::currency::deposit(items, bytes) / 10 + // 1/100 of Westend testnet + constants::currency::deposit(items, bytes) / 100 } } diff --git a/cumulus/parachains/runtimes/assets/westmint/src/lib.rs b/cumulus/parachains/runtimes/assets/westmint/src/lib.rs index 44293376f0..c8946668e6 100644 --- a/cumulus/parachains/runtimes/assets/westmint/src/lib.rs +++ b/cumulus/parachains/runtimes/assets/westmint/src/lib.rs @@ -208,8 +208,8 @@ impl pallet_transaction_payment::Config for Runtime { } parameter_types! { - pub const AssetDeposit: Balance = 1 * UNITS; // 1 WND deposit to create asset - pub const AssetAccountDeposit: Balance = 1 * UNITS; // 1 WND for an asset account + pub const AssetDeposit: Balance = UNITS / 10; // 1 / 10 WND deposit to create asset + pub const AssetAccountDeposit: Balance = deposit(1, 16); pub const ApprovalDeposit: Balance = EXISTENTIAL_DEPOSIT; pub const AssetsStringLimit: u32 = 50; /// Key = 32 bytes, Value = 36 bytes (32+1+1+1+1) @@ -497,8 +497,8 @@ impl pallet_asset_tx_payment::Config for Runtime { } parameter_types! { - pub const CollectionDeposit: Balance = UNITS; // 1 UNIT deposit to create asset class - pub const ItemDeposit: Balance = UNITS / 100; // 1/100 UNIT deposit to create asset instance + pub const CollectionDeposit: Balance = UNITS / 10; // 1 / 10 UNIT deposit to create asset class + pub const ItemDeposit: Balance = UNITS / 1_000; // 1 / 1000 UNIT deposit to create asset instance pub const KeyLimit: u32 = 32; // Max 32 bytes per key pub const ValueLimit: u32 = 64; // Max 64 bytes per value pub const UniquesMetadataDepositBase: Balance = deposit(1, 129); diff --git a/cumulus/parachains/runtimes/contracts/contracts-rococo/src/constants.rs b/cumulus/parachains/runtimes/contracts/contracts-rococo/src/constants.rs index 8661ec78a3..fabc09d017 100644 --- a/cumulus/parachains/runtimes/contracts/contracts-rococo/src/constants.rs +++ b/cumulus/parachains/runtimes/contracts/contracts-rococo/src/constants.rs @@ -26,8 +26,8 @@ pub mod currency { pub const MILLICENTS: Balance = constants::currency::MILLICENTS; pub const fn deposit(items: u32, bytes: u32) -> Balance { - // map to 1/10 of what the kusama relay chain charges (v9020) - constants::currency::deposit(items, bytes) / 10 + // map to 1/100 of what the kusama relay chain charges (v9020) + constants::currency::deposit(items, bytes) / 100 } }