Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

Transaction lists may open block explorer with incorrect url #7455

Closed
StephenHeaps opened this issue May 15, 2023 · 1 comment · Fixed by #7456
Closed

Transaction lists may open block explorer with incorrect url #7455

StephenHeaps opened this issue May 15, 2023 · 1 comment · Fixed by #7456

Comments

@StephenHeaps
Copy link
Contributor

StephenHeaps commented May 15, 2023

Description:

Transactions lists in Wallet no longer are limited to the selected network only, but in a few cases we are using selected network to build the block explorer URLs.

This applies to:

Steps to Reproduce

  1. Create Transaction on Goerli and confirm it so it's on the blockchain
  2. Open Uniswap in a tab
  3. Change network to any other Ethereum network
  4. Open Transactions list
  5. Tap and hold to open context menu and open on blockchain explorer
  6. Observe incorrect blockchain explorer used

Actual result:

Screen.Recording.2023-05-15.at.2.34.17.PM.mov

Expected result:

Goerli blockchain explorer url is used

Reproduces how often:

Easily reproduced

Brave Version:

Only applicable to Brave iOS versions created May 15th 2023 or later.

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

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

@nuo-xu nuo-xu self-assigned this May 15, 2023
@StephenHeaps StephenHeaps changed the title Transaction list may open block explorer with incorrect url Transaction lists may open block explorer with incorrect url May 15, 2023
@iccub iccub added this to the 1.52 milestone May 24, 2023
@srirambv
Copy link
Contributor

srirambv commented Jun 5, 2023

Verification passed on iPhone 13 with iOS 16.5 running 1.52 (23.6.1.16)

  • Verified steps from issue description and #7456
  • Verified correct block explorer link is opened irrespective of which network is set as active
  • Verified transaction list from panel loads the correct block explorer
  • Verified transaction from the account page loads the correct block explorer irrespective of which is set as active
  • Verified add token screen links to the correct token page
7455-1.MP4
7455-2.MP4

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants