Thomas
Thomas
Hey there, any chance to have a review for this PR plz ?
I can be wrong but I think it's the inverse actually, `r/users` is the old one, and `r/demo/users` is the current one.
Thanks for raising this, I checked on my side and I noticed an other issue related to `PackageValue.IsRealm()` method. If we fix it, maybe we won't need this extra `std.TestSetPrevRealm`...
@r3v4s I was able to fix the issue I had, can you compile `gno` from this PR #896 and see if you still have the issue on your side ?...
~~My take on this PR stays the same, considering #1048, we don't need those extra `TestSetPrev*` functions. But since #1048 is blocked, do what you think is best.~~ See comment...
> My take on this PR stays the same, considering #1048, we don't need those extra `TestSetPrev*` functions. But since #1048 is blocked, do what you think is best. So...
> Instead of overwriting the previous frames, we should add a new frame that marks the PrevRealm I don't think it's a good idea to manually manipulate the frames. The...
> Indeed, it would be very interesting if we could rely on the fact that valsets are shared for ICS (and hence implicit) and maybe implement an "implicit" IBC that...
OK, so it's similar to Replicated Security, I actually thought it was something else.
Given recent discussions on this topic, some key aspects of ICS1.5 have raised. (Disclaimer: The following items provided are not necessarily exact and/or may change in the future, they are...