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

Rewards panel not showing "verified" status in some cases #17669

Closed
Brave-Matt opened this issue Aug 24, 2021 · 5 comments · Fixed by brave/brave-core#9854
Closed

Rewards panel not showing "verified" status in some cases #17669

Brave-Matt opened this issue Aug 24, 2021 · 5 comments · Fixed by brave/brave-core#9854
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated.

Comments

@Brave-Matt
Copy link
Collaborator

Description

For some users, the Rewards panel is not displaying Twitter and/or Youtube channel verification status in the Rewards panel. Strangely, other websites do appear to show site verified status.
(Expected display on left, issue on right)
image image

Issue appears to be intermittent and seems to "fix" itself temporarily, although what fixes it is still unclear. For users who are experiencing this, it appears to be broken consistently (as opposed to intermittent) and are unable to resolve the issue after:

  • Relaunching/restarting browser
  • Clearing cache/cookies
  • Updating to latest build
  • Visiting both .es and/or .com
  • Changing language settings for both sites as well as language settings in the browser

Steps to Reproduce/ Reproduces how often:

Steps to reproduce the issue are unclear. At the time of writing this, the issue appears to show up most consistently in Nightly build, version 1.30.41 (at the time of writing this).

Desktop Brave version:

In addition to Nightly build, user's are seeing this issue in Brave stable v1.28.150 and 1.28.106.

Version/Channel Information:

Stable/Nightly

  • Can you reproduce this issue with the current release? Yes
  • Can you reproduce this issue with the beta channel? No
  • Can you reproduce this issue with the nightly channel? Yes

Miscellaneous Information:

Internal Brave conversation here:
https://bravesoftware.slack.com/archives/CB0KCRCQN/p1629303796208800
Community thread w/some users experiencing the issue:
https://community.brave.com/t/idk-if-its-a-bug-please-read-and-help-me/275583/30

@Brave-Matt Brave-Matt added feature/rewards OS/Android Fixes related to Android browser functionality OS/Desktop labels Aug 24, 2021
@emerick emerick added this to New Issues in Brave Rewards via automation Aug 24, 2021
@kjozwiak
Copy link
Member

kjozwiak commented Aug 24, 2021

I can manage to reproduce the above almost 100% on Win 10 x64 using the following build:

Brave | 1.30.43 Chromium: 93.0.4577.51 (Official Build) nightly (64-bit)
-- | --
Revision | 762d21050e2da59930c784c09b134d0b0b148188-refs/branch-heads/4577@{#915}
OS | Windows 10 OS Version 2009 (Build 19042.1165)

Both @GeetaSarvadnya @LaurenWags & @StephenBass also reproduced the issue using the STR/Cases mentioned below:

STR:

  • launch 1.30.43 Chromium: 93.0.4577.51
  • enable rewards via the rewards panel
  • visit https://www.youtube.com
  • search hot ones and click on the First We Feast channel
  • click on Videos within the First We Feast channel
  • pick a random video from the list
  • once that video loads, click on the rewards panel

I also get the following whenever I click on the rewards panel on YT (not sure if related):

[61996:72356:0824/134303.039:INFO:CONSOLE(1)] "Greaselion script loaded: youtubeBase.ts", source: chrome-extension://feipjgjfhmnfhhmbkclfopokbcgnpnnd/youtubeBase.bundle.js (1)
[61996:72356:0824/134303.040:WARNING:CONSOLE(1)] "Received external message from a sender who is not allowed to send messages to this background. Origin: https://www.youtube.com, URL: https://www.youtube.com/", source: chrome-extension://mnojpmjdmbbfmejpflffifhffcmidifd/out/brave_extension_background.bundle.js (1)
[61996:72356:0824/134303.160:ERROR:CONSOLE(1)] "Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'ruleExceptions')", source: chrome-extension://mnojpmjdmbbfmejpflffifhffcmidifd/out/brave_extension_background.bundle.js (1)
Example (Existing profile) Example (New profile)
image image (1)

Also happening on reddit as per the following:

image

@emerick
Copy link
Contributor

emerick commented Aug 24, 2021

I can repro following the steps @kjozwiak provided. I see the following error in the console log for the Rewards extension:

Uncaught TypeError: Cannot read properties of undefined (reading 'catch')
    at Module.359 (brave_rewards_panel_background.bundle.js:1)
    at r (brave_rewards_panel_background.bundle.js:1)
    at brave_rewards_panel_background.bundle.js:1
    at brave_rewards_panel_background.bundle.js:1

@zenparsing zenparsing added the priority/P2 A bad problem. We might uplift this to the next planned release. label Aug 25, 2021
@zenparsing
Copy link
Collaborator

The Rewards extension still uses extension manifest v2, which does not support promise return values on chrome.* APIs. We should probably remove the call to .catch(...) and add a tslint ignore comment if TS generates an error.

@mkarolin
Copy link
Collaborator

I am a bit concerned that the users also reporting seeing the issue on Beta/Release channels. The change to using .catch(...) has not been uplifted beyond Nightly.

@Miyayes
Copy link
Collaborator

Miyayes commented Oct 5, 2021

I think we can close this, as it is superseded and resolved by #14602

@Miyayes Miyayes closed this as completed Oct 5, 2021
Brave Rewards automation moved this from On deck to Done Oct 5, 2021
@kjozwiak kjozwiak added closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. and removed feature/rewards priority/P2 A bad problem. We might uplift this to the next planned release. QA/Yes release-notes/include OS/Android Fixes related to Android browser functionality OS/Desktop labels Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated.
Projects
Brave Rewards
  
Done
6 participants