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

clicking on an in-line tip icon does not open the panel to show rewards onboarding #26109

Closed
LaurenWags opened this issue Oct 18, 2022 · 4 comments · Fixed by brave/brave-core#15541

Comments

@LaurenWags
Copy link
Member

LaurenWags commented Oct 18, 2022

Description

Found while testing #14277

Prior to opting into Brave Rewards, if you click on an in-line tip icon (on twitter, reddit, gh), the panel should open and show onboarding. This is not occurring with 1.46.x. When you click on the in-line tip button nothing happens in the UI, so the button appears broken.

Not sure if this is related to the changes from #14277.

Note, confirmed that the panel opens as expected to offer onboarding with 1.45.x via #24538 (comment).

Steps to Reproduce

  1. Fresh profile, relaunch to pull griffin seed
  2. Go to twitter.com/emerick (or reddit.com/u/cryptojennie or open this GH issue)
  3. Click on an in-line tip button

Note to @brave/qa-team - when retesting this issue after it's fixed, Case 4 from #14277 (comment) needs to be checked.

Actual result:

Nothing happens

Expected result:

Rewards panel opens to show onboarding

Screen Shot 2022-10-18 at 4 40 44 PM

Reproduces how often:

easily

Brave version (brave://version info)

Brave | 1.46.66 Chromium: 107.0.5304.36 (Official Build) nightly (x86_64)
-- | --
Revision | 2f9c7a5a1fe357d87e9bc07c65cca9136b3651c5-refs/branch-heads/5304@{#671}
OS | macOS Version 12.6 (Build 21G115)

Version/Channel Information:

  • Can you reproduce this issue with the current release? 1.44.x n/a
  • Can you reproduce this issue with the beta channel? 1.45.x no
  • Can you reproduce this issue with the nightly channel? 1.46.x 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 @mkarolin

@LaurenWags
Copy link
Member Author

Added release/blocking per discussion with @Miyayes

@LaurenWags
Copy link
Member Author

Uplift required, adding QA/Blocked for now.

@kjozwiak
Copy link
Member

Removing QA/Blocked. The above requires 1.46.80 or higher for 1.46.x verification 👍

@GeetaSarvadnya GeetaSarvadnya added the QA/In-Progress Indicates that QA is currently in progress for that particular issue label Oct 28, 2022
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Oct 28, 2022

Verification PASSED on

Brave | 1.46.81 Chromium: 107.0.5304.68 (Official Build) beta (64-bit)
-- | --
Revision | a4e93e89d3b3df1be22214603fba846ad0183ca5-refs/branch-heads/5304@{#991}
OS | Windows 10 Version 21H2 (Build 19044.2130)

Twitter, Reddit and GH - PASSED
  1. Fresh profile, relaunch to pull griffin seed
  2. Go to twitter.com/emerick and reddit.com/u/cryptojennie and open this GH issue
  3. Click on an in-line tip button somewhere on the page
  4. Confirmed panel opened to show onboarding for Twitter, Reddit and GH
Twitter Reddit GH
image image image

Verified Case 4 from #14277 (comment) for the in-line tip buttons:

Case 4 - In-line tip button - PASSED
  1. Fresh profile, relaunch to pull griffin seed
  2. Go to twitter.com/emerick
  3. Click on an in-line tip button
  4. Panel is displayed, but do not opt in to rewards. Close panel.
  5. Check for calls to *.rewards.brave.com
  6. Confirm no calls to *.rewards.brave.com
  7. Go to github.com/miyayes
  8. Click on an in-line tip button
  9. Panel is displayed, but do not opt in to rewards. Close panel.
  10. Check for calls to *.rewards.brave.com
  11. Confirm no calls to *.rewards.brave.com
  12. Go to reddit.com/u/cryptojennie
  13. Click on an in-line tip button
  14. Panel is displayed, but do not opt in to rewards. Close panel.
  15. Check for calls to *.rewards.brave.com
  16. Confirm no calls to *.rewards.brave.com
  17. Open NTP
  18. Check for calls to *.rewards.brave.com
  19. Confirm no calls to *.rewards.brave.com
  20. Open browser Task Manager and confirm Ledger service is not running
Step 6 Step 11 Step 16 Step 19 Step 20
image image image image image

@GeetaSarvadnya GeetaSarvadnya added QA Pass-Win64 and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Brave Rewards
  
Done
Development

Successfully merging a pull request may close this issue.

5 participants