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

chore: upgrade to polkadot 2409 #965

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

Conversation

Roznovjak
Copy link
Contributor

@Roznovjak Roznovjak commented Dec 16, 2024

Notable changes:

Changes in config params:

  • Removed ApproveOrigin and OnSlash from the treasury pallet.
  • Removed deprecated calls propose_spend, reject_proposal, approve_proposal from the treasury pallet.
  • Add OnSlash config parameter to the bounties and tips pallets.

TODO:

  • testing

@Roznovjak Roznovjak self-assigned this Dec 16, 2024
Copy link

github-actions bot commented Dec 30, 2024

Crate versions that have not been updated:

  • runtime-integration-tests: v1.32.1
  • pallet-broadcast: v1.0.1
  • pallet-dca: v1.8.1
  • pallet-omnipool: v4.5.2
  • pallet-route-executor: v2.7.1
  • hydradx-adapters: v1.3.11

Crate versions that have been updated:

  • hydra-dx-math: v8.3.0 -> v8.3.1
  • hydradx: v14.0.2 -> v14.1.0
  • pallet-asset-registry: v3.3.0 -> v3.3.1
  • pallet-bonds: v2.2.4 -> v2.2.5
  • pallet-circuit-breaker: v1.1.28 -> v1.1.29
  • pallet-claims: v3.4.12 -> v3.4.13
  • pallet-collator-rewards: v1.0.9 -> v1.0.10
  • pallet-currencies: v3.0.0 -> v3.0.1
  • pallet-democracy: v4.3.2 -> v4.3.3
  • pallet-duster: v3.2.7 -> v3.2.8
  • pallet-dynamic-evm-fee: v1.0.5 -> v1.0.6
  • pallet-dynamic-fees: v1.0.6 -> v1.0.7
  • pallet-ema-oracle: v1.3.4 -> v1.3.5
  • pallet-evm-accounts: v1.2.1 -> v1.2.2
  • pallet-genesis-history: v2.1.5 -> v2.1.6
  • pallet-lbp: v4.10.0 -> v4.10.1
  • pallet-liquidation: v1.0.2 -> v1.0.3
  • pallet-liquidity-mining: v4.4.5 -> v4.4.6
  • pallet-nft: v7.1.5 -> v7.1.6
  • pallet-omnipool-liquidity-mining: v2.6.1 -> v2.6.2
  • pallet-otc: v2.1.0 -> v2.1.1
  • pallet-otc-settlements: v1.1.1 -> v1.1.2
  • pallet-referrals: v1.2.8 -> v1.2.9
  • pallet-relaychain-info: v0.3.5 -> v0.3.6
  • pallet-stableswap: v4.3.0 -> v4.3.1
  • pallet-staking: v4.1.0 -> v4.1.1
  • pallet-transaction-multi-payment: v10.1.4 -> v10.1.5
  • pallet-transaction-pause: v1.1.0 -> v1.1.1
  • pallet-xyk: v6.7.0 -> v6.7.1
  • pallet-xyk-liquidity-mining: v1.4.1 -> v1.4.2
  • pallet-evm-precompile-call-permit: v0.1.3 -> v0.1.4
  • precompile-utils: v0.1.3 -> v0.1.4
  • primitives: v6.1.0 -> v6.1.1
  • runtime-mock: v1.0.3 -> v1.0.4
  • hydradx-runtime: v285.0.0 -> v286.0.0

Runtime version has been increased.

@Roznovjak Roznovjak marked this pull request as ready for review January 20, 2025 13:03
integration-tests/src/xyk_liquidity_mining.rs Outdated Show resolved Hide resolved
let deps = crate::rpc::FullDeps {
client: client.clone(),
pool: transaction_pool.clone(),
deny_unsafe,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where this moved to?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was removed. You can find more info here paritytech/polkadot-sdk#4792

pallets/omnipool-liquidity-mining/src/lib.rs Show resolved Hide resolved
pallets/route-executor/src/lib.rs Show resolved Hide resolved
@@ -782,6 +784,19 @@ impl<T: Config> Pallet<T> {
Ok(amount_in_and_outs)
}

pub fn calculate_expected_amount_in(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why was this added?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a new method in the AssetExchange trait and I needed it in the implementation of this methd.

pallet_route_executor::Pallet::<Runtime>::calculate_expected_amount_in(&route, amount.into()).ok()?;

runtime/hydradx/src/evm/mod.rs Show resolved Hide resolved
@@ -146,13 +146,8 @@ impl frame_support::traits::tokens::Pay for PayFromTreasuryAccount {

impl pallet_treasury::Config for Runtime {
type Currency = Balances;
type ApproveOrigin = EitherOf<EnsureRoot<AccountId>, Treasurer>;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where did this go?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ApproveOrigin is used only in one extrinsic, approve_proposal. The extrinsic was deprecated and replaced by spend extrinsic and SpendOrigin.

@Roznovjak Roznovjak requested a review from mrq1911 February 4, 2025 09:33
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.

2 participants