search

sygmaprotocol / sygma-widget

Transfer widget for the sygmaprotocol
5 stars 2 forks source link

Refresh MAX and Amount to transfer field when changing Source #118

Closed LyonSsS closed 3 months ago

LyonSsS commented 4 months ago

Severity: Low | Priority: Low

Context: After selecting a relationship Source-Dest and filling in some "Amount to transfer", if the user then switches to another source, the context of Max and Amount to transfer is not refreshed.

STR:

  1. Select From as "Mumbai" and connect metmask wallet to the correct network.
  2. Select To as "Sepolia" and then select ERC20RLTest token.
  3. After balance is updated, input an amount <= Max value for the "Amount to transfer" field.
  4. Change From value now to "Cronos".

Exp: The context to be resseted for the token type to be transferred, max amount, and amount to be transferred. Act: The only context that is resseted is for the token type. image

wainola commented 4 months ago

this issue is related to #111

MakMuftic commented 4 months ago

Hey team! Please add your planning poker estimate with Zenhub @Lykhoyda @mpetrunic @wainola

LyonSsS commented 4 months ago

Tested based on fix https://github.com/sygmaprotocol/sygma-widget/pull/127 The "Amount to transfer", still persist

Amount_To_Transfer.gif
LyonSsS commented 3 months ago

Retested on https://github.com/sygmaprotocol/sygma-widget/pull/127 - the amount issue still persists

LyonSsS commented 3 months ago

retested and fixed https://github.com/sygmaprotocol/sygma-widget/pull/127