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

RemovePallet migration utility struct #13835

Merged
merged 5 commits into from
Apr 6, 2023

Conversation

liamaharon
Copy link
Contributor

Migration utility struct that allows easily removing pallet storage on the next runtime upgrade.

Useful for things like removing Gov V1 pallet storage after the switch to OpenGov.

Related: paritytech/polkadot-sdk#485, paritytech/polkadot#6977

@liamaharon liamaharon added A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit B1-note_worthy Changes should be noted in the release notes labels Apr 6, 2023
@liamaharon liamaharon added the T1-runtime This PR/Issue is related to the topic “runtime”. label Apr 6, 2023
frame/support/src/remove_pallet.rs Outdated Show resolved Hide resolved
frame/support/src/remove_pallet.rs Outdated Show resolved Hide resolved
liamaharon and others added 3 commits April 6, 2023 19:11
Co-authored-by: Bastian Köcher <git@kchr.de>
Co-authored-by: Bastian Köcher <git@kchr.de>
@liamaharon liamaharon merged commit 9a07e9d into master Apr 6, 2023
@liamaharon liamaharon deleted the liam-remove-pallet-utility-struct branch April 6, 2023 14:27
@Polkadot-Forum
Copy link

This pull request has been mentioned on Polkadot Forum. There might be relevant details there:

https://forum.polkadot.network/t/april-updates-for-substrate-and-polkadot-devs/2764/1

nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
* remove pallet utility struct

* Update frame/support/src/remove_pallet.rs

Co-authored-by: Bastian Köcher <git@kchr.de>

* Update frame/support/src/remove_pallet.rs

Co-authored-by: Bastian Köcher <git@kchr.de>

* make removepallet a module of migrations

* fix rust doc lint

---------

Co-authored-by: Bastian Köcher <git@kchr.de>
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. B1-note_worthy Changes should be noted in the release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit T1-runtime This PR/Issue is related to the topic “runtime”.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants