zho
zho
These feature changes for moving table are not yet merged with master right?
I'll paste your quote from closed PR here @wraeth-eth > Another problem I just noticed is that when you have amount set, you cannot set anything higher than the amount...
@wraeth-eth I've found a much easier workaround for all of this which I've tested on every scenario, I'm gonna push it soon.
> Hey @NakedFool sorry i've been away for the last couple of days. Let me know when this is ready for a re-review :) No worries! It took me time...
> Hey @NakedFool - I think we probably need an overhaul of the entire system. It is likely we are employing the wrong solution for the problem here which would...
yoo @wraeth-eth ! I've pushed commits that are responsible for og bugs, I have a few notes that I would love to let you know. ### v1 vs v2 I'll...
Pushed the latest changes, and things should be good now! Discussion - https://discord.com/channels/939317843059679252/1011161626747088956/1013710657046200340
I've applied the logic that we talked about above, feel free to review @wraeth-eth or @johnnyd-eth EDIT: I've applied the same logic to `V2MintModal`
> Hey @NakedFool, > > This was a great catch. > > We have this problem though out the app, I've been working on fixing a bunch of them. Definitely...
> My bad! I sent you the wrong link of the pattern we use today. > > This is the pattern that we use today: https://github.com/jbx-protocol/juice-interface/blob/1ee4f8db276c80610652ed919a55a385989672ab/src/hooks/v2/transactor/TransferProjectOwnershipTx.ts > > Moving forward,...