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

bump transaction_version due to parachain_system change #748

Merged
merged 1 commit into from
Nov 11, 2021

Conversation

apopiak
Copy link
Contributor

@apopiak apopiak commented Nov 11, 2021

#517 changed the calls of the parachain_system pallet.
--> Bump transaction version.

@apopiak apopiak mentioned this pull request Nov 11, 2021
13 tasks
@chevdor
Copy link
Contributor

chevdor commented Nov 11, 2021

Westmint could use a bump as well (details)

@apopiak
Copy link
Contributor Author

apopiak commented Nov 11, 2021

Westmint could use a bump as well (details)

It is already bumped to 4. I was aware that we're reshuffling all the pallets :-D

@chevdor
Copy link
Contributor

chevdor commented Nov 11, 2021

What about rococo ?

@apopiak
Copy link
Contributor Author

apopiak commented Nov 11, 2021

It's not live anywhere, is it?
We don't really have to bump spec or transaction versions for chains that are not live (at least as a testnet)

Copy link
Contributor

@chevdor chevdor left a comment

Choose a reason for hiding this comment

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

LGTM

@apopiak apopiak merged commit 472da73 into release-statemine-v6 Nov 11, 2021
@apopiak apopiak deleted the apopiak/bump-tx-version branch November 11, 2021 16:38
bkchr pushed a commit that referenced this pull request Jan 18, 2022
Co-authored-by: Alexander Popiak <alexander.popiak@parity.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants