question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

do not show "Your wallet is verified!" message when reconnecting after natural disconnect

See original GitHub issue

Description

Per discussion in https://bravesoftware.slack.com/archives/CB0KCRCQN/p1627659102017500, we should not show the “Your wallet is verified!” message/notification when a user is reconnecting after natural disconnect.

Steps to Reproduce

  1. Clean profile 1.29.x, staging env
  2. Enable Rewards
  3. Link KYC’d, verified Uphold account
  4. Simulate “natural” disconnect by revoking Brave-Browser app authorization on Uphold site
  5. Confirm UI shows “Disconnected”/no balance
  6. Click on “Disconnected” and reauthorize Uphold
  7. You are successful in re-auth and you get this notification

Also reproduces in this scenario:

  1. Using current release version, 1.27.109, clean profile, staging env, enable Rewards, link KYC’d Uphold wallet
  2. Simulate “natural” disconnect by revoking Brave-Browser app authorization on Uphold site
  3. Close browser
  4. “Upgrade” (rename profile to Brave-Browser-Nightly) to 1.29.x
  5. Confirm your verified, KYC’d wallet is still in Disconnected status after upgrade (appropriate UI, no Uphold balance shown)
  6. Click on “Disconnected” and re-auth with Uphold

Actual result:

Your wallet is verified notification: Screen Shot 2021-07-30 at 11 26 04 AM

Expected result:

No notification in this case

Reproduces how often:

easily

Brave version (brave://version info)

Brave 1.29.40 Chromium: 92.0.4515.115 (Official Build) nightly (x86_64)
Revision 48cb2f4029b84b003719740a6cf9ca73f374a857-refs/branch-heads/4515_105@{#4}
OS macOS Version 11.5.1 (Build 20G80)

Version/Channel Information:

  • Can you reproduce this issue with the current release? n/a
  • Can you reproduce this issue with the beta channel? n/a
  • Can you reproduce this issue with the nightly channel? yes

Other Additional Information:

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

Miscellaneous Information:

cc @Miyayes @szilardszaloki

Issue Analytics

  • State:closed
  • Created 2 years ago
  • Comments:6 (3 by maintainers)

github_iconTop GitHub Comments

1reaction
szilardszalokicommented, Sep 1, 2021

@GeetaSarvadnya That’s odd. Just took a look with 1.31.10 and it was working fine. Let me rebase and try that again!

1reaction
szilardszalokicommented, Aug 13, 2021

Hey @GeetaSarvadnya! FYI: https://github.com/brave/brave-core/pull/9738 is not getting uplifted to 1.29.x.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Wallet Address Not Confirmed - Coinmama Support
Coinmama will not send your coins until you have confirmed your wallet address by clicking the link in the wallet confirmation email you...
Read more >
How to check if Metamask is connected after page refreshing
I finally found a possible solution and it turned out to be as simple as it should be. There is an eth_accounts method...
Read more >
Electrum Documentation - Read the Docs
In short, not really. The Electrum client never sends private keys to the servers. In addition, it verifies the information.
Read more >
Web3 Dapp Developer Guide: React Hooks for Ethereum
With Web3 Onboard's Ethereum react hooks, any developer can quickly set up their dapp to interact with the best web3 wallets.
Read more >
ID document verification | Coinbase Help
ID verification is required to prevent fraud and keep the community safe. It also adds an extra layer of security by ensuring no...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found