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

Nightly on Android does not turn off Rewards icon #20633

Closed
pratyakshm opened this issue Jan 22, 2022 · 5 comments · Fixed by brave/brave-core#12182
Closed

Nightly on Android does not turn off Rewards icon #20633

pratyakshm opened this issue Jan 22, 2022 · 5 comments · Fixed by brave/brave-core#12182
Assignees
Labels
bug feature/rewards OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA Pass - Android Tab QA/Yes release-notes/exclude

Comments

@pratyakshm
Copy link

Description

Steps to reproduce

  1. Go to Brave Nightly Settings > Appearance.
  2. Turn Show Brave Rewards icon in address bar* off.
  3. It prompts to restart app for changes to take effect, hence tap on Relaunch now.

Actual result

Brave Rewards icon is not disabled from the toolbar.

Expected result

Brave Rewards icon is disabled.

Issue reproduces how often

Easily reproduced

Version/Channel Information:

  • Can you reproduce this issue with the current Play Store version? No
  • Can you reproduce this issue with the current Play Store Beta version? No response
  • Can you reproduce this issue with the current Play Store Nightly version? Yes

Device details

  • Install type (ARM, x86): ARM64
  • Device type (Phone, Tablet, Phablet): Phablet
  • Android version: 11

Brave version

1.36.58

Website problems only

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

Additional information

@pratyakshm pratyakshm added the OS/Android Fixes related to Android browser functionality label Jan 22, 2022
@Miyayes
Copy link
Collaborator

Miyayes commented Jan 23, 2022

I wonder if recent change #19695 for this setting on desktop is not propagating nicely over into Android? cc: @emerick

@emerick
Copy link
Contributor

emerick commented Jan 24, 2022

Yes, I think we'll need to do a real preference migration on Android. @deeppandya I may ping you for some pointers...

@Uni-verse
Copy link
Contributor

Uni-verse commented Feb 17, 2022

I am still able to repro this issue on 1.36.96, chromium 98.0.4758.87 - https://github.com/brave/brave-browser/releases/tag/v1.36.96

After the browser relaunches, changes do not take affect.

@kjozwiak
Copy link
Member

Adding QA/Blocked for Android till #21183 is addressed/fixed. The above can also be tested alongside #19695 & #20646 when #21183 is fixed.

@srirambv
Copy link
Contributor

Verification passed as part of #21183 (comment).

  • Verified on both clean install and upgrade scenario rewards button is hidden or shown based on the setting

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/rewards OS/Android Fixes related to Android browser functionality priority/P2 A bad problem. We might uplift this to the next planned release. QA Pass - Android ARM QA Pass - Android Tab QA/Yes release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants