Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Swap transaction shows incorrect token symbol in submitted state. Updates correctly when Swap is completed #30447

Closed
srirambv opened this issue May 19, 2023 · 4 comments · Fixed by brave/brave-core#18379

Comments

@srirambv
Copy link
Contributor

Description

Swap transaction shows incorrect token symbol in submitted state. Updates correctly when Swap is completed

Steps to Reproduce

  1. Initiate a Swap from a ERC20 token to ETH
  2. Go to activity tab
  3. Shows swap for ETH -> ETH
  4. Wait for the transaction to complete
  5. Activity tab updates correctly ERC20 -> ETH

Actual result:

image

Expected result:

image

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 1.52.102 Chromium: 113.0.5672.126 (Official Build) beta (64-bit)
Revision c541687b21a73452ab403e2dced7033ddc97ee9d-refs/branch-heads/5672@{#1202}
OS Windows 11 Version 22H2 (Build 22621.1702)

Version/Channel Information:

  • Can you reproduce this issue with the current release? NA
  • Can you reproduce this issue with the beta channel? Yes
  • Can you reproduce this issue with the nightly channel? Not checked

Other Additional Information:

  • Does the issue resolve itself when disabling Brave Shields? NA
  • Does the issue resolve itself when disabling Brave Rewards? NA
  • Is the issue reproducible on the latest version of Chrome? NA

Miscellaneous Information:

cc: @onyb

@srirambv
Copy link
Contributor Author

Also noticed Approve transaction doesn't show the token name when the swap transaction is in progress and only shows up the token symbol when swap is completed

@josheleonard
Copy link

@srirambv , I can no longer reproduce on master. Was most-likely fixed by: brave/brave-core#18379

Screen.Recording.2023-06-05.at.5.52.40.PM.mov

@josheleonard josheleonard moved this from Backlog to Done in Web3 Jun 7, 2023
@josheleonard josheleonard added this to the 1.54.x - Nightly milestone Jun 8, 2023
@josheleonard
Copy link

Resolved by: brave/brave-core#18379

@srirambv
Copy link
Contributor Author

Verification passed on

Brave 1.57.18 Chromium: 115.0.5790.98 (Official Build) beta (64-bit)
Revision d515871a6c6df7d2843e755c8b7dda9b744f26d6
OS Windows 11 Version 22H2 (Build 22621.1992)
  • Verified steps from issue description
  • Verified correct token symbol is shown in activity tab for Swap transactions
  • Verified correct token symbol is shown for Unapproved -> Approved -> Submitted -> Completed state
30447.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants