substrate icon indicating copy to clipboard operation
substrate copied to clipboard

nomination-pools: fix usage of `stash` and `controller` around `StakingInterface`

Open kianenigma opened this issue 2 years ago • 3 comments

currently it is a bit of a mess. It should be aligned with the dispatch origin of the corresponding call. For example, the controller can call bond_extra, so it should be called controller.

kianenigma avatar May 17 '22 09:05 kianenigma

Hello again, can I pick this task instead? Is there any additional information I might need?

kris524 avatar May 22 '22 16:05 kris524

We should instead try and drop the controller account logic imo, and just rely on proxies. Seems better than continually iterating on an obsolete api

shawntabrizi avatar May 23 '22 05:05 shawntabrizi

cc @ruseinov

kianenigma avatar Oct 10 '22 10:10 kianenigma

@ruseinov I believe this is closed by #12424 ?

kianenigma avatar Nov 07 '22 13:11 kianenigma