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

The max fee estimation on mobile is not matched to the fee on Desktop for bridge transactions #21620

Open
VolodLytvynenko opened this issue Nov 13, 2024 · 3 comments
Assignees
Labels
mobile-core wallet: Send all issues for the send page of the wallet wallet-core Issues for mobile wallet team
Milestone

Comments

@VolodLytvynenko
Copy link
Contributor

VolodLytvynenko commented Nov 13, 2024

Steps:

  1. Go to the bridge page
  2. Build the following bridge routes

Actual result:

Bridge routes are estimated incorrectly

example:
L2 arb -> L2 opt
or
L2 arb -> L2 opt

Max fee is estimated as <0.01 USD

Image

Expected result:

The bridge estimation should be matched to the Desktop
Image

Additional info:

It seems mobile is not including the "bridge fee" and "approval fee" as desktop does Image
Image

Devices:

  • Pixel 7a, Android 13
  • iPhone 11 Pro Max, IOS 17

ENV:

Nightly 13 Now

@VolodLytvynenko VolodLytvynenko added this to the 2.32.0 milestone Nov 13, 2024
@VolodLytvynenko VolodLytvynenko added mobile-core wallet: Send all issues for the send page of the wallet wallet-core Issues for mobile wallet team labels Nov 13, 2024
@churik churik moved this from Backlog to Next in Status Desktop/Mobile Board Nov 14, 2024
@alwx alwx self-assigned this Nov 21, 2024
@alwx
Copy link
Contributor

alwx commented Nov 21, 2024

Should be fixed by this: #21541

@VolodLytvynenko
Copy link
Contributor Author

VolodLytvynenko commented Nov 21, 2024

issue is not fixed within #21541. Fee estimation still incorrect

@alwx
Copy link
Contributor

alwx commented Nov 21, 2024

@VolodLytvynenko Ok, you're right because the fee estimation is still done separately and might be done differently on desktop. Will take a further look at it but for now it's not an issue that's linked to #21541

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mobile-core wallet: Send all issues for the send page of the wallet wallet-core Issues for mobile wallet team
Projects
Status: Next
Development

No branches or pull requests

2 participants