telosnetwork / telos-bridge

The Telos LayerZero bridge enables seamless asset transfers between Telos and other blockchains
https://bridge.telos.net
5 stars 3 forks source link

Fix Bridge Token Allowance Process #100

Closed pmjanus closed 4 months ago

pmjanus commented 4 months ago

Describe the bug

To bridge USDT from ETH to TLOS, ensure you have enough USDT and ETH in your account. Approve the bridge to handle the amount of USDT you want to transfer. You can edit this amount during approval.

For example, if you set the approval amount to 1 USDT and later decide to bridge a higher amount, the bridge should prompt you for a new approval. However, it fails to do so, leading to a gas estimation error with the message "execution reverted" during the transaction.

To Reproduce Try to bridge a coin, set allowance to a number less than what you want to send, then try to send.

Expected behavior Bridge should always let user set allowance to the level they need to get the bridge done if the allowance is not sufficient.

rozzaswap commented 4 months ago

To be addressed in https://github.com/telosnetwork/telos-bridge/issues/101