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

Buy/Send/Swap not opening from Asset Details #36266

Closed
Tracked by #8775
StephenHeaps opened this issue Feb 21, 2024 · 4 comments · Fixed by brave/brave-core#22229
Closed
Tracked by #8775

Buy/Send/Swap not opening from Asset Details #36266

StephenHeaps opened this issue Feb 21, 2024 · 4 comments · Fixed by brave/brave-core#22229
Assignees
Labels
bug feature/web3/wallet Integrating Ethereum+ wallet support OS/iOS Fixes related to iOS browser functionality QA/No release-notes/exclude

Comments

@StephenHeaps
Copy link

StephenHeaps commented Feb 21, 2024

Description:

Buy, send and swap are not opening for some assets when tapped in Asset Details. The bug seems to occur on any asset with no account balance.

Steps to Reproduce

  1. Open Wallet
  2. Scroll Portfolio to find ETH on Optimism.
    • Seems to occur on any asset where user does not have any account balance
  3. Tap 'Buy'
  4. Observe buy does not open
    • Buy/send/swap do not open either
    • Some other assets do open as expected. Needs investigation.

Actual result:

Asset.Details.MP4

Expected result:

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easily reproduced with ETH on Optimism, not all assets failing to open.

Brave Version:

v1.62 App Store

  • 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:

iPhone 14 Pro iOS 17.3.1

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

Community report:
https://community.brave.com/t/i-cannot-send-or-swap-crypto/532738/3

@kjozwiak
Copy link
Member

Closing as the above landed in master via brave/brave-core#22229.

@kjozwiak
Copy link
Member

The above requires 1.63.164 or higher for 1.63.x verification 👍 @srirambv @StephenHeaps assuming this is a regression hence being uplifted into 1.63.x? Should we add the release-notes/include label?

@StephenHeaps
Copy link
Author

@kjozwiak yes regression, I think good to include in release notes as it was reported by community.

@kjozwiak
Copy link
Member

Labelling as QA/No & release-notes/exclude as per brave/brave-ios#8776 (comment).

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 OS/iOS Fixes related to iOS browser functionality QA/No release-notes/exclude
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants