feat: migrate transfers form #1555
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
The fees shouldn't have to match the asset being transferred. I think the client defaults to USDC right now and should be displayed as such until we provide the ability for users to use native gas token (This will come a lot later after we fully deprecate old withdraw/deposit flow since Abacus only supports USDC gas atm) |
|
oh wow it actually does use dv4tnt as the gas token. Me thinks we should prob still use USDC so that it is consistent and we can control which gas token to use in the future |
|
Label for the gas token changes as I fill out the amount input Screen.Recording.2025-03-07.at.8.40.24.AM.mov |
| }; | ||
| } | ||
|
|
||
| class TransferFormValidationErrors { |
There was a problem hiding this comment.
i think we should also validate for withdrawGatingInfo and possible withdrawCapacity limits.
There was a problem hiding this comment.
these are only necessary for USDC transfer tho
There was a problem hiding this comment.
this is handled by the useWithdrawalInfo I think. It pops up a window if things are blocked. I talked to teddy and I THINK he said withdrawalCapacity isn't relevant for transfers.
There was a problem hiding this comment.
but why wouldn't withdrawalCapacity matter? It does a withdrawSubaccountMsg which is the same that we do on actual withdrawals?
There was a problem hiding this comment.
teddy said it only applies to the noble thingy, so only if you're taking usdc out via that noble whatchamacallit
No description provided.