Jeancarlo Barrios

Results 12 comments of Jeancarlo Barrios

Is there any news on this subject

> @marbar3778 : @JeancarloBarrios might hop into this PR to fix some stuff in the meantime while you're out. I will keep the progress in this PR #13031

> These values should be set in the app.toml for node operators to change. Or is it already set somewhere Can you re-check if that is what you mean

Failing simapp test will be solved by #14036

> redundant > nameOfMap This is a really good point thanks @odeke-em !!!

> Nice work and thank you @JeancarloBarrios! LGTM from a security, performance and Go review perspective. I've added some extra comments though. One thing though is ensure we document to...

> > > Nice work and thank you @JeancarloBarrios! LGTM from a security, performance and Go review perspective. I've added some extra comments though. One thing though is ensure we...

I add a small draft that just returns an error when max tx is reached let me know if that makes sense @kocubinski @tac0turtle if so I can also add...

> Yes, agree that needing the capital in advance could be a barrier to entry, and there could be other solutions for anti-spam fee. But we wondered if these things...

> * governance has to decide what to do with the fee pool of non-REGEN tokens I believe we can do that already. with authz itself