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

Approving transaction without internet does not remove submitting / loading overlay #38375

Closed
StephenHeaps opened this issue May 16, 2024 · 1 comment · Fixed by brave/brave-core#23706
Assignees
Labels
bug feature/web3/wallet Integrating Ethereum+ wallet support front-end-change This task is a front end task and doesn't need any C++ changes OS/iOS Fixes related to iOS browser functionality QA/Yes release-notes/exclude
Milestone

Comments

@StephenHeaps
Copy link

Description:

On iOS there is a transaction status overlay on Transaction Confirmation after a user taps Approve, until they tap 'ok' to either dismiss or show the next transaction in queue. More details: brave/brave-ios#6861

While a transaction is moving from unapproved to submitted/error/dropped status, there is currently no way to hide this overlay (without dismissing entire transaction confirmation modal).

When approving a transaction without an internet connection, this 'Submitting' overlay is not removed, but if you dismiss entire Transaction Confirmation and check Activity tab you can see the transaction is in error status.

Steps to Reproduce

  1. Create a transaction on any network. Used ethereum for testing but should work for any transaction.
  2. In Transaction Confirmation panel, wait for data to load
  3. Once data has loaded, disable network connection
  4. Tap approve
  5. Loading spinner will not be removed from transaction. Dismiss transaction confirmation and open activity tab, observe transaction has error status

Actual result:

RPReplay_Final1715871159.MP4

Expected result:

Transaction should show error status after the loading spinner screen in Transaction Confirmation

Reproduces how often:

Easily reproduced

Brave Version:

  • Can you reproduce this issue with the most recent build from TestFlight?
  • Can you reproduce this issue with the previous version of the current build from TestFlight?
  • Can you reproduce this issue with the current build from AppStore?

Device details:

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields?
  • Is the issue reproducible on the latest version of Mobile Safari?

Additional Information

CC @srirambv thanks for reporting 👍

@kjozwiak
Copy link
Member

The above requires 1.67.98 or higher for 1.67.x verification 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/web3/wallet Integrating Ethereum+ wallet support front-end-change This task is a front end task and doesn't need any C++ changes OS/iOS Fixes related to iOS browser functionality QA/Yes release-notes/exclude
Projects
Status: Done
3 participants