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

enabling/disabling Brave News via setting puts it into a weird state until users taps on Show Brave News via NTP #21493

Closed
kjozwiak opened this issue Mar 6, 2022 · 1 comment · Fixed by brave/brave-core#12508
Assignees
Labels
bug feature/brave-news formerly brave-today OS/Android Fixes related to Android browser functionality QA Pass - Android ARM QA/Yes release-notes/include

Comments

@kjozwiak
Copy link
Member

kjozwiak commented Mar 6, 2022

Description

When a user enables Brave News via Hamburger -> Brave News after brave/brave-core#12489 has been merged and then goes back to the Brave News settings, they'll notice that Show Brave News toggle is disabled even though the on-boarding card was already enabled. Enabling the setting once again creates a weird state where it looks like Brave News is completely enabled but the on-boarding card still appears via NTP. Sometimes the fee will load and sometimes you'll get the on-boarding card. To fix this situation/state, a user needs to tap on Show Brave News via the on-boarding card.

My guess is that when you disable/enable multiple times, Brave News thinks it's still enabled but the user only enabled the card and never specifically toggled Show Brave News. But this part is missing from the Settings page if you enable/disable.

image

Steps to reproduce

  1. download/install 1.38.25 Chromium: 99.0.4884.51
  2. tap on Hamburger Menu -> Brave News and toggle Enable Brave News and close the settings page via x
  3. should see the Brave news on-boarding card enabled/visible via NTP
  4. tap on Hamburger Menu -> Brave News and notice that the toggle is disabled
  5. enable the toggle once again and you'll notice that this time, it's missing the Show Brave News isn't being displayed
  6. close the Brave News setting page via x and you'll notice the news feed is visible
  7. open a NTP again and you'll notice that the on-boarding card is being displayed rather than the news feed

Pressing the X on the on-boarding card or tapping on Show Brave News will remove the user from the weird state.

Actual result

Example of the issue --> https://youtu.be/l2lls1mU4II

Expected result

Users shouldn't be getting into a state where the feed is supposed to be appearing but they're still getting the on-boarding card.

Issue reproduces how often

100% reproducible using the STR/Cases mentioned above.

Version/Channel Information:

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

Once brave/brave-core#12489 is uplifted into BETA/Release, all channels will have the above issue.

Device details

  • Install type (ARM, x86): ARM
  • Device type (Phone, Tablet, Phablet): Pixel 6
  • Android version: Android 12

Brave version

1.38.25 Chromium: 99.0.4884.51

Website problems only

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

Additional information

CCing @alexsafe @bsclifton @srirambv @Uni-verse @anthonypkeane

@srirambv
Copy link
Contributor

Verification passed on Oppo Reno 5 with Android 12 running 1.36.115 x64 build

  • Verified steps from issue description
  • Verified enabling/disabling Brave News via settings doesn't put it into a weird state like before
  • Verified enabling and not opting-in to news and closing the active NTP doesn't cause any issue
  • Logged #21659

avinassh pushed a commit to avinassh/brave-browser-hardening that referenced this issue May 29, 2022
- Fixed Brave News onboarding card still appearing under New Tab Page in certain cases when Brave News has already been enabled via settings. ([#21493](brave/brave-browser#21493))
- Upgraded Chromium to 99.0.4844.74. ([#21693](brave/brave-browser#21693)) ([Changelog for 99.0.4844.74](https://chromium.googlesource.com/chromium/src/+log/99.0.4844.51..99.0.4844.74?pretty=fuller&n=1000))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feature/brave-news formerly brave-today OS/Android Fixes related to Android browser functionality QA Pass - Android ARM QA/Yes release-notes/include
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants