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

Feat/upgrade1020 misc 1 #1529

Merged
merged 8 commits into from
Sep 5, 2023
Merged

Feat/upgrade1020 misc 1 #1529

merged 8 commits into from
Sep 5, 2023

Conversation

mustermeiszer
Copy link
Collaborator

@mustermeiszer mustermeiszer commented Sep 4, 2023

Description

Miscellaneous changes for the next release

Changes and Descriptions

  • pallet-order-book implements trait TokenSwaps without min_buy_amount checks
  • axelar-gateway-precompile in Altair uses EnsureRootOr<HalfOfCouncil>
  • provides a Multilocation based account conversion runtime API

Checklist:

  • I have added Rust doc comments to structs, enums, traits and functions
  • I have made corresponding changes to the documentation
  • I have performed a self-review of my code
  • I have added tests that prove my fix is effective or that my feature works

@mustermeiszer
Copy link
Collaborator Author

Bit wild kind of changes. But I did not wanted to wait for CI with 3 little PRs. Sorry...

@mustermeiszer
Copy link
Collaborator Author

Fixed lints and checked locally. We can merge this without CI going through. All other checks where green.

Copy link
Contributor

@cdamian cdamian left a comment

Choose a reason for hiding this comment

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

Not so familiar with the order-book, the rest of the changes seem OK.

Copy link
Contributor

@wischli wischli left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the OrderBook fix 🚀

@@ -91,6 +90,6 @@ impl pallet_ethereum_transaction::Config for Runtime {
}

impl axelar_gateway_precompile::Config for Runtime {
type AdminOrigin = EnsureRoot<AccountId>;
type AdminOrigin = EnsureRootOr<HalfOfCouncil>;
Copy link
Contributor

Choose a reason for hiding this comment

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

Out of curiosity: Do want to propose limiting to Root long-term?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I guess gov v2 will anyways make Council obsolete, right?

@mustermeiszer mustermeiszer merged commit 178e1d6 into upgrade1020 Sep 5, 2023
mustermeiszer added a commit that referenced this pull request Sep 6, 2023
* wip xcm v3 asset-registry migrations

* fix: xcm v3 altair, centrifuge

* feat: add defensive SafeCallFilter for transact

* clean up asset addition for catalyst & centrifuge

* add Altair + Algol migration

* Add Liquidity Pools related pallets to centrifuge runtime (#1523)

* centrifuge: Add liquidity pools pallets to runtime

* centrifuge: Add LP OutboundQueue wrapper to runtime

* taplo: Obey

* runtime: Move OutboundQueue import

* centrifuge: Allow both root and half of council as admin origins for LP pallets

* centrifuge: Add Axelar Gateway precompile

* centrifuge: Use correct HRMP encoder, update HRMP fee to 1 DOT, use XCM V3 imports

* centrifuge: Update error message for InboundQueue

* clippy: Fix warnings

* centrifuge: Rename outbound queue, fix migration import for orml asset registry

* Upgrade1020 LP follow-ups (#1525)

* centrifuge: Enable LP gateway XCM origin converter

* centrifuge: Update LP pallet config to use Quantity for BalanceRatio

* runtime: Update XCM origin converter, use Quantity in LP message, add OriginRecovery to LP gateway

* feat: safe XCM version migration + extend SafeCallFilter (#1526)

* fix: allow gateway process msg for xcm

* feat: add safe xcm version migration

* fix: Allow process_msg for xcm transact

* refactor: remove spec_version check

* refactor: stricter SafeCallFilter

* Upgrade1020 LP follow-ups (#1525)

* centrifuge: Enable LP gateway XCM origin converter

* centrifuge: Update LP pallet config to use Quantity for BalanceRatio

* runtime: Update XCM origin converter, use Quantity in LP message, add OriginRecovery to LP gateway

* fix: rm OrderBook safe call cause missing

* fix: OrderBook missing in both runtimes

* fix: clippy

---------

Co-authored-by: Cosmin Damian <[email protected]>

* Feat/upgrade1020 misc 1 (#1529)

* fix: asset metadata xcm v3 migration cleanup other migrations (#1528)

* fix: centrifuge asset migration

* Apply suggestions from code review

Co-authored-by: Frederik Gartenmeister <[email protected]>

* chore: append cfg assets to catalyst

* fix: asset registry migration improvements

* feat: apply asset metadata migration to altair runtime

* fix: cleanup altair/algol migrations

* fix: less aggressive nuke pre-upgrade

* fix: finalize migrations, add context

* fix: clippy + cleanup imports

* fix: native currency altair, algol

* fix: Algol native currency name

---------

Co-authored-by: Frederik Gartenmeister <[email protected]>

* fmt

* fixup

* Upgrade1020 gateway changes (#1531)

* gateway: Use BoundedVec instead of EVMChain enum

* gateway: Add Sender type to LP gateway config

* gateway: Rename EVM chain size const, emit event when submitting message, adjust gateway sender provider

* Feat: upgrade1020 misc 2 (#1532)

* feat: account ensuring origin

* feat: bumb runtime version develpment

* fix: allow council to control gatway on Altair based chains

* fix: use EnsureSigned, make use of type and add admin

* feat: overestimated weights for lp logic

* fix: taplooo

* fix: tests and comment

* chore: bump weights for 1020 (#1533)

* chore: update centrifuge, altair weights

* feat: bump and improve dev weights

* fix: transfer_allowlist weights

* feat: normalize SafeCallFilter across runtimes

* fix: native asset key in migration

---------

Co-authored-by: William Freudenberger <[email protected]>
Co-authored-by: Cosmin Damian <[email protected]>
Co-authored-by: Frederik Gartenmeister <[email protected]>
Co-authored-by: nuno <[email protected]>
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.

3 participants