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.

Sign up for GitHub

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

Jump to bottom

Max value calculates 3 networks in bridge flow instead of 2 #20440

Closed
VolodLytvynenko opened this issue Jun 12, 2024 · 0 comments · Fixed by #20604
Closed

Max value calculates 3 networks in bridge flow instead of 2 #20440

VolodLytvynenko opened this issue Jun 12, 2024 · 0 comments · Fixed by #20604
Assignees
@J-Son89
Labels
E:Mobile Bug MVP E:Mobile Wallet Networks and Routes T:mobile-wallet-core issues for mobile wallet core team 🔢 medium prio
Milestone
2.30.0 Beta

Comments

@VolodLytvynenko
Copy link
Contributor

VolodLytvynenko commented Jun 12, 2024 •
edited
Loading

Steps:

  1. Go to bridge flow
  2. Select a network on the 'Bridge to' page to which you are going to bridge assets
  3. Go to bridge routes generation
  4. Check max value

Actual result:

The sum of max value consists of 3 networks

Expected result:

The sum of max value consists of 2 networks, excluding the network selected in 'Bridge to'

image

https://www.figma.com/design/xLs1KYmF4e6WwRTZVJKeUK/Descoped-Wallet?node-id=15417-73808&t=vW0pAveMDn6J8cax-0

OS:

IOS, Android

Devices:

  • Pixel 7a, Android 13
  • iPhone 11 Pro Max, IOS 17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@J-Son89 J-Son89

Labels
E:Mobile Bug MVP E:Mobile Wallet Networks and Routes T:mobile-wallet-core issues for mobile wallet core team 🔢 medium prio
Projects
Status Desktop/Mobile Board
Status: Done
Milestone
2.30.0 Beta
Development

Successfully merging a pull request may close this issue.

fix(wallet): adjust max amount to not include network being bridged to status-im/status-mobile
3 participants
@churik @J-Son89 @VolodLytvynenko