search

sygmaprotocol / sygma-widget

Transfer widget for the sygmaprotocol
5 stars 2 forks source link

Switching to a different TOKEN mapping for "From - TO" relationship is not updated in UI if amount is inserted #117

Closed LyonSsS closed 3 months ago

LyonSsS commented 4 months ago

Severity: Low | Priority: Medium

Context: When switching to another "From - To" mapping, that needs to update the available compatible ERC20 token vs previous selection, UI is not updated.

STR:

  1. Login with metmask and select from as Sepolia.
  2. Select to as Mumbai and select drop down to ERC20RLTest tokens.
  3. After balance is updated, insert an amount <= MAX in the "Amount to transfer" field.
  4. Select TO destination to Rhala, which has the only mapping to ERC20 token sygUSDC.

Exp: UI to update the correct erc20 mapping. ACT: Nothing happened. The same ERC20RLTest token remain.

image

MakMuftic commented 4 months ago

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

saadjhk commented 3 months ago

@LyonSsS I tested this functionality by pulling latest commits from main and it seems to be fixed, can you please validate the functionality once?

LyonSsS commented 3 months ago

I did retested it on latest build from https://github.com/sygmaprotocol/sygma-widget/pull/60 and I could reproduce it. IncorrectMapping

LyonSsS commented 2 months ago

Retested on CoreReleaseBranch - this now works fine