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

nomination-pools: clear pool metadata after a pool is deleted. #12128

Closed
wants to merge 5 commits into from

Conversation

tifecool
Copy link
Contributor

PR for: #12077

@tifecool
Copy link
Contributor Author

tifecool commented Aug 27, 2022

I'm aware a test would most likely be necessary, but I don't have experience with tests.

Would appreciate some guidance

Copy link
Contributor

@kianenigma kianenigma left a comment

Choose a reason for hiding this comment

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

Not quire correct.

To begin with, you should make sure dissolve_pool also clears the metadata, as the issue is asking for.

Then, you should make sure this is tested. You can just add relevant assertions to some tests that already end up dissolving a pool.

The migration can be dealt with later.

This PR is rather time-sensitive for the next release. If not able to attend to it ASAP, let me know and I will do it myself.

@tifecool
Copy link
Contributor Author

To begin with, you should make sure dissolve_pool also clears the metadata, as the issue is asking for.

The metadata of the bonded pool or all the metadata?

This PR is rather time-sensitive for the next release. If not able to attend to it ASAP, let me know and I will do it myself.

I'll try running the assertions now. I should be able to finish off today, if I'm unable to I'll drop it,

@kianenigma kianenigma added this to the Polkadot v0.9.29 milestone Aug 29, 2022
@kianenigma
Copy link
Contributor

The metadata of the bonded pool or all the metadata?

The metadata of the pool, what's stored in Metadata::<T>::get(pool_id).

@kianenigma kianenigma added the A3-in_progress Pull request is in progress. No review needed at this stage. label Aug 29, 2022
@ruseinov
Copy link
Contributor

I'll take over this in a new PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A3-in_progress Pull request is in progress. No review needed at this stage.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants