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

Polkadot v1.1.0: Account converter changes #1806

Merged
merged 2 commits into from
Apr 11, 2024

Conversation

lemunozm
Copy link
Contributor

@lemunozm lemunozm commented Apr 11, 2024

Description

From this discussion: #1756 (comment)

A refactor of AccountConverter:

  • All methods belong to AccountConverter type.
  • If the method requires the type to have a runtime type, then it uses the wrapper RuntimeAccountConverter type. This should only be used by trait demands in pallet configurations. For simple calls, use always AccountConverter

Are we in a middle point with this PR @mustermeiszer?

@lemunozm lemunozm added the I6-refactoring Code needs refactoring. label Apr 11, 2024
@lemunozm lemunozm self-assigned this Apr 11, 2024
@lemunozm lemunozm mentioned this pull request Apr 11, 2024
58 tasks
Copy link
Collaborator

@mustermeiszer mustermeiszer left a comment

Choose a reason for hiding this comment

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

@lemunozm thanks, thats perfect!! ❤️

@lemunozm lemunozm merged commit d4d58e1 into polkadot-v1.1.0 Apr 11, 2024
10 checks passed
@lemunozm lemunozm deleted the polkadot-v1.1.0-account-converter branch April 11, 2024 12:08
lemunozm added a commit that referenced this pull request Apr 15, 2024
* all dependencies set

* migrate libs/*

* migrate pallet-anchors

* migrate pallet-rewards

* migrate pallet-liquidity-rewards

* migrate pallet-fees pallet-bridge

* migrate pallet-keystore

* migrate restricted-tokens

* migrate restricted-xtokens

* migrate block-rewards

* migrate-interest-accrual

* migrate oracle-feed

* migrate oracle-collection

* migrate collator-allowlist

* migrate transfer-allowlist

* migrate crowdloan pallets

* migrate permissions

* migrate order-book

* migrate pallet-swaps

* migrate pallet-token-mux

* migrate investments

* migrate foreign-investments

* migrate loans

* migrate pool-fees

* migrate liquidity-pools-gateway

* migrate axelar-gateway-precompile

* migrate liquidity-pools-gateway-routers & patch moonbeam deps

* use precompile-utils from moonbeam from now

* simplify mock initialization

* simplify some mocks

* migrate ethereum-transaction

* simplify minor stuff

* disable runtimes by now

* migrate pool-system

* migrate pool-registry

* migrate liquidity-pools

* polish patch section

* dirty fix for unaccesible Pending storage

* minor dependency update

* minor extend comment

* migrate runtime-common

* minor fixes

* Revert "dirty fix for unaccesible Pending storage"

This reverts commit 975060d.

* pointing to the updated pallet-ethereum

* fix pool-system compilation with benchmarks

* fix serde dependencies

* required changes for runtime to work

* development runtime failing because of serde

* patch all required dependencies to make moonbeam works

* fix xcm types

* add serde to no_std

* minor

* migrate altair & centrifuge

* minor fixes

* support runtime-benchmarking in runtimes

* Polkadot v1.1.0: Node required changes (#1789)

* make it compile

* minor change

* prepare changes for modify deprecated code

* fix ExportGenesisState and remove unused warnings

* update benchmark CLI

* remove completelly try-runtime-cli

* minor clean

* fix try-runtime CI job

* Polkadot v1.1.0 Integration Tests (#1776)

* legacy-test migration wip

* main changes

* remove unused dependency

* basic stuff migrated

* uncomment fudge_env

* update fudge

* integration-tests: Fix test envs (#1780)

* main changes

* integration-tests: Fix test envs

* runtime: Implement parachains pallets for runtimes

* runtime: Enable multilocation convert for CurrencyIdConvert

* runtime: Remove extra paras pallets

* integration-tests: Update expected balances

---------

Co-authored-by: lemunozm <[email protected]>

---------

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

* fixed issues compiling with rust 1.78

* fix clippy

* taplo

Signed-off-by: lemunozm <[email protected]>

* reading value from deposit_creating

* remove locked from taplo-cli

* update scripts

* fix docs

* fix integration tests

* add collator-selection migration

* makes try-runtime idempotent checks work

* revert fixed point serialization, remove unused Location code

* comment TODOs and deprecated until 1.3.0

* fix fixed_point serialization for std

* correct features for dev dependency

* apply Cosmin NITs

* remove comments regarding v1.3.0

* revert benchmarking HostFunction

* Polkadot v1.1.0: Account converter changes (#1806)

* Modify AccountConverter types

* fix some imports

* fix: polkadot v1.1.0 client (#1808)

* fix: deprecate parachain CLI

* chore: cleanup deprecated chain specs

* fix: clippy

---------

Signed-off-by: lemunozm <[email protected]>
Co-authored-by: Cosmin Damian <[email protected]>
Co-authored-by: William Freudenberger <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I6-refactoring Code needs refactoring.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants