Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Use Extensible Multiaddress in all Polkadot Runtimes #2126

Merged
2 commits merged into from
Jan 27, 2021

Conversation

shawntabrizi
Copy link
Member

This PR is a breaking change for the transaction version. cc @s3krit @joepetrowski

This PR introduces the Multiaddress type into the Polkadot, Kusama, Westend, Cumulus, and "Test" runtimes.

You can learn more about the Multiaddress type here: paritytech/substrate#7380

With this PR, all of the Polkadot runtimes are now able to more easily adopt future address formats like those provided by pallet-evm or even a future name-service pallet, or those we have not even thought of.

@shawntabrizi shawntabrizi added A0-please_review Pull request needs code review. B7-runtimenoteworthy C1-low PR touches the given topic and has a low impact on builders. labels Dec 16, 2020
@shawntabrizi
Copy link
Member Author

Note that I have decided to create this PR now since #2025 will already force us to bump tx version, so two birds with one stone.

@gavofyork
Copy link
Member

bot merge

@ghost
Copy link

ghost commented Jan 27, 2021

Trying merge.

@ghost ghost merged commit 917dcb1 into master Jan 27, 2021
@ghost ghost deleted the shawntabrizi-multiaddress branch January 27, 2021 08:00
s3krit pushed a commit that referenced this pull request Jan 27, 2021
* Use extensible multiaddress

* copy pasta
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants