Repository navigation
chore: remove insuficient asset ED and fee fallback to xyk pools - #1546
enthusiastmartin wants to merge 6 commits into
Conversation
There was a problem hiding this comment.
Claude Code Review
Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.
Tip: disable this comment in your organization's Code Review settings.
Weight Diff Report0 extrinsic(s) changed across 0 pallet(s). New: 0. Removed: 3. Removed extrinsics (3)
Threshold: ±10%. Base |
|
Crate versions that have not been updated:
Crate versions that have been updated:
Crate versions that have been downgraded:
Runtime version has not been increased. |
There was a problem hiding this comment.
Claude Code Review
Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.
Tip: disable this comment in your organization's Code Review settings.
Removes the paths that let non-whitelisted assets pay fees, and the insufficient-asset ED toll.
Changes
AcceptedCurrencies‚ the XYK(X, DOT)tier is gone.set_currency,get_currency_price,account_currency, EVM permit gas andNativePriceOracleall apply the same whitelist check. XCM execution fees (Trader,XcmPaymentApi) are now limited to accepted currencies too.AssetRegistry::register_external(call index 4) is deleted; external assets are onboarded through governanceregisteronly.SufficiencyCheck‚BannedAssetCheck;ban_assetis now the spam control. Legacysufficientsrefs are kept on purpose (not released).asset_inisn't an accepted currency, the fee is taken from the owner's free balance in their fee currency instead of swappingasset_into DOT.Migrations
PurgeUnsupportedFeeCurrencies(multi-block) ‚ removesAccountCurrencyMapentries for non-accepted assets.RetireInsufficientEdPool(single-block) ‚ removes the treasurySUFFICIENCY_LOCKandExistentialDepositCounter. Note that treasury keeps it, not refunds. currently around 100k