nomination-pools: clear pool metadata after a pool is deleted.#12128
nomination-pools: clear pool metadata after a pool is deleted.#12128tifecool wants to merge 5 commits intoparitytech:masterfrom
Conversation
|
I'm aware a test would most likely be necessary, but I don't have experience with tests. Would appreciate some guidance |
kianenigma
left a comment
There was a problem hiding this comment.
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.
The metadata of the bonded pool or all the metadata?
I'll try running the assertions now. I should be able to finish off today, if I'm unable to I'll drop it, |
The metadata of the pool, what's stored in |
|
I'll take over this in a new PR. |
PR for: #12077