Matt, Park
Matt, Park
Let's merge this upon reverting excluding the `Require()` :) !
@catShaark answered! I think ur understanding there is correct!
Closing this issue per finishing all tasks
good point! Agreed 😎
Oh good catch! Thanks for this issue, we should be solving this by the next upgrade. Meanwhile, querying via correct arguments should be able to be done via stargate queries...
> I am going to tag @ValarDragon for his opinion on potential attack surface this would provide. I am not concerned about the number of validators an account can set....
> > Left second round of reviews, overall looking good, I think we just need test cases for the community pool funding for val set creation fee > > Thank...
Feel free to re-request for review once ur in a comfortable state for the review
@hieuvubk please go ahead! That'd be great! Might be almost impossible to do this in a single PR since the scope is too big, might want to do it by...
hey @alexanderbez are you stabbing this rn? If not, I'd like to PR this change with the other audit items!