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 widget is shown in NTP when Binance settings is enabled in brave://settings/newTab #9529

Closed
GeetaSarvadnya opened this issue Apr 29, 2020 · 2 comments
Assignees

Comments

@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Apr 29, 2020

Description

When Brave Rewards and Binance settings are set to OFF, Rewards widget is shown in NTP when Binance settings is enabled in brave://settings/newTab

Steps to Reproduce

  1. Set OS locale to Japan
  2. Install Brave nightly 1.8.55
  3. Make sure Binance and Rewards widget is shown
  4. Turn OFF both Binace and Rewards widgets in Dashboard settings
  5. Open an NTP and make sure Binance and Rewards widget isn't shown in NTP
  6. Upgrade to Brave nightly 1.10.15
  7. Navigate to NTP which is already opened in steps 5 and ensure Binance and Rewards widget isn't shown in NTP
  8. Open Dashboard settings and ensure there is no Binace settings and Rewards switch is in OFF state
  9. Navigate to brave://settings/newTab and ensure Binace option is in OFF state
  10. Enable the Binance option
  11. Open a NTP which shows Brave Rewards widget even though Rewards is OFF in both Dashboard and brave://settings/newTab settings

Actual result:

https://drive.google.com/open?id=1LvuPr_KEv4ptRbsMDH0mvSbjmf_SM3JM
Rewards widget is shown in NTP when Binance settings is enabled in brave://settings/newTab
image

Expected result:

Rewards widget should not be shown in NTP when Binance settings is enabled in brave://settings/newTab

Reproduces how often:

Always

Brave version (brave://version info)

Brave 1.10.15 Chromium: 81.0.4044.129 (Official Build) nightly (64-bit)
Revision 3d71af9f5704a40b85806f4d08925db24605ba25-refs/branch-heads/4044@{#979}
OS Windows 10 OS Version 1803 (Build 17134.1006)

Version/Channel Information:

  • Can you reproduce this issue with the current release? NA
  • Can you reproduce this issue with the beta channel? Not sure
  • Can you reproduce this issue with the dev channel? Not sure
  • Can you reproduce this issue with the nightly channel? Yes

Other Additional Information:

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

Miscellaneous Information:

cc: @brave/legacy_qa @ryanml

@ryanml
Copy link
Contributor

ryanml commented May 16, 2020

@GeetaSarvadnya I believe this is actually expected.

There exists this logic to show the widget in two cases, one where Rewards is enabled, or one where there is a sponsored image showing: https://github.com/brave/brave-core/blob/master/components/brave_new_tab_ui/containers/newTab/index.tsx#L485 (Note in your picture the messaging reflecting that they can get paid for this)

So unless the widget shows when toggled off on non-sponsored image NTP's, I think we can close this. Maybe there should be an additional path though that users can take to have it off and still show sponsored images. cc: @rebron @karenkliu

@GeetaSarvadnya
Copy link
Author

GeetaSarvadnya commented May 18, 2020

@ryanml The issue isn't reproducible as the Binance option is removed from brave://settings/newTab settings for Japan locale (#9416 - fixed) The issue was reproducible only when I turn on Binance option from settings. Good to close the issue as it's not reproducible now.

@ryanml Reg Rewards widget, it was shown even in normal NTP including sponsored images NTP's anyways now the issue is not reproducible as Binance option is removed from brave://settings/newTab settings for Japan locale

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

No branches or pull requests

3 participants