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

App crashes after attempting to send assets with minimum value #19610

Closed
Tracked by #19466
VolodLytvynenko opened this issue Apr 11, 2024 · 6 comments · Fixed by status-im/status-go#5178
Closed
Tracked by #19466

App crashes after attempting to send assets with minimum value #19610

VolodLytvynenko opened this issue Apr 11, 2024 · 6 comments · Fixed by status-im/status-go#5178
Assignees
@briansztamfater
Labels
E:Mobile Bug MVP E:Mobile Wallet MVP Design system components, initial research and page wireframes E:Mobile Wallet Send Page MVP all issues for the send page of the wallet high-priority status-go

Comments

@VolodLytvynenko
Copy link
Contributor

Unfortunately, there are no exact steps to reproduce this issue. in my case, the issue usually happens for the particular user after attempting to send WETH available on the L2 network

Steps:

  1. Recover the account with available assets (in my case it is always reproducible with WETH assets)
  2. Go to 'send to' page
  3. Enter a minimum value to send assets

Actual result:

App crashes after attempting to send assets with minimum value

crash.mp4

OS:

IOS, Android

Devices:

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

Logs

Logs:

Status app logs logs (13).zip
geth logs logs (2).txt
@VolodLytvynenko VolodLytvynenko mentioned this issue Apr 11, 2024
Closed
57 tasks
@mmilad75 mmilad75 assigned mmilad75 and unassigned mmilad75 Apr 22, 2024
@VolodLytvynenko VolodLytvynenko added E:Mobile Wallet Send Page MVP all issues for the send page of the wallet E:Mobile Wallet MVP Design system components, initial research and page wireframes high-priority labels Apr 26, 2024
@OmarBasem
Copy link
Member

@VolodLytvynenko I tried to reproduce this issue with WETH but did not crash. Are you still able to reproduce the issue?

@VolodLytvynenko
Copy link
Contributor Author

@VolodLytvynenko I tried to reproduce this issue with WETH but did not crash. Are you still able to reproduce the issue?

Hi @OmarBasem, unfortunately, I can't check it because I'm always facing the issue described in #19999, which might overlap with the current crash

@OmarBasem
Copy link
Member

@VolodLytvynenko I tried to reproduce this issue with WETH but did not crash. Are you still able to reproduce the issue?

Hi @OmarBasem, unfortunately, I can't check it because I'm always facing the issue described in #19999, which might overlap with the current crash

Is that issue recent?

@VolodLytvynenko
Copy link
Contributor Author

@VolodLytvynenko I tried to reproduce this issue with WETH but did not crash. Are you still able to reproduce the issue?

Hi @OmarBasem, unfortunately, I can't check it because I'm always facing the issue described in #19999, which might overlap with the current crash

Is that issue recent?

@OmarBasem yes. Just yesterday encountered it

@briansztamfater briansztamfater self-assigned this May 17, 2024
@briansztamfater
Copy link
Member

briansztamfater commented May 17, 2024 •
edited

I was able to reproduce this. Trying to figure out the cause, because I coudln't find anything in the logs or in debug terminal. I suspect this might be a status-go bug, but not sure, I'll continue debugging. Assigning this to myself and see if I can find something or fix it.

@briansztamfater
Copy link
Member

@VolodLytvynenko Indeed, I tried the same on Status Desktop, and it crashed. So it is for sure a status-go bug.

Attaching error report:
reportstatusdesktop.txt

Please can you make this bug visible for the desktop team as well so they can propertly fix in on the status-go side?

@IvanBelyakoff IvanBelyakoff mentioned this issue May 17, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@briansztamfater briansztamfater

Labels
E:Mobile Bug MVP E:Mobile Wallet MVP Design system components, initial research and page wireframes E:Mobile Wallet Send Page MVP all issues for the send page of the wallet high-priority status-go
Projects
Do not use - tasks moved to unified Desktop / Mobile Board
Status: Done
Status Desktop/Mobile Board
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(wallet)_: fix crash on cbridge fee calculation status-im/status-go
4 participants
@briansztamfater @OmarBasem @VolodLytvynenko @mmilad75