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

[Android 13 ] show android 13 modal based update conidition #27032

Closed
sujitacharya2005 opened this issue Nov 29, 2022 · 2 comments · Fixed by brave/brave-core#16128
Closed

[Android 13 ] show android 13 modal based update conidition #27032

sujitacharya2005 opened this issue Nov 29, 2022 · 2 comments · Fixed by brave/brave-core#16128

Comments

@sujitacharya2005
Copy link

sujitacharya2005 commented Nov 29, 2022

Description

Show Rewards warning dialog , privacy warning dialog or combine message dialog.
Conditions

  1. Notification permission is not there
  2. Or particular notification group permission is off (Like Brave Ads or General Group is off)
  3. Then check for rewards enabled or privacy feature enabled
  4. show only one time this dialog for onBoarding
  5. show one time for setting page

Steps to reproduce

This dialog is showing two place.

  1. onBoarding time
  2. Setting page notification click and back
  3. base on above condition.

Actual result

Expected result

Issue reproduces how often

Version/Channel Information:

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

Device details

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

Brave version

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

@sujitacharya2005 sujitacharya2005 added the OS/Android Fixes related to Android browser functionality label Nov 29, 2022
@sujitacharya2005 sujitacharya2005 self-assigned this Nov 29, 2022
@sujitacharya2005 sujitacharya2005 added this to the 1.46.x - Release milestone Nov 29, 2022
@sujitacharya2005 sujitacharya2005 changed the title [Android 13 ] show android 13 modal based on rewards / privacy enabled [Android 13 ] show android 13 modal based update conidition Nov 29, 2022
@kjozwiak kjozwiak removed this from the 1.46.x - Release milestone Nov 30, 2022
@brave-builds brave-builds added this to the 1.48.x - Nightly milestone Dec 10, 2022
@kjozwiak
Copy link
Member

The above requires 1.47.168 or higher for 1.47.x verification. @Uni-verse leave this one for me 👍

@kjozwiak
Copy link
Member

kjozwiak commented Jan 13, 2023

Verification PASSED on Pixel 6 running Android 13 using the following build(s):

Brave | 1.47.171 Chromium: 109.0.5414.87 (Official Build) (32-bit)
--- | ---
Revision | 2dc18eb511c56e012081b4abc9e38c81c885f7d4-refs/branch-heads/5414@{#1241}
OS | Android 13; Build/T2B1.221118.00

Test Case #1 (Generic Modal/Warning) - Both Rewards & Privacy Report disabled

  • launch 1.47.171 Chromium: 109.0.5414.87 and run through onboarding
  • tap on Continue via the Brave notifications make it easier modal
    • also ran through this case by tapping on Not now and ensuring notifications weren't being enabled with this method
  • tap on Dont Allow via the Allow Brave to send you notifications modal
  • go into Settings -> Notifications and notice that Notifications are disabled as expected
  • tap on <- (back) to go back into Settings and ensure that you receive a generic warning re: notifications being disabled
  • ensured that taping on Not now dismisses the modal without any issues
  • ensured that tapping on Continue takes you back into the Notifications screen without an issues
    • ensured that going back into Settings didn't display the generic warning modal re: notifications being disabled
  • ensured that once either Not now or Continue has been tapped, the generic warning modal doesn't appear again
    • also ensured that the generic modal doesn't re-appear if the browser was closed while the modal was still opened
Example Example Example Example Example
Screenshot_20230112-150754 Screenshot_20230112-150758 Screenshot_20230112-150809 Screenshot_20230112-150812 Screenshot_20230112-150834

Test Case #2 (Generic Modal/Warning) - Rewards enabled & Privacy Report disabled

  • launch 1.47.171 Chromium: 109.0.5414.87 and run through onboarding
  • tap on Continue via the Brave notifications make it easier modal
  • tap on Allow via the Allow Brave to send you notifications modal to enable notifications
  • enable rewards via the rewards panel (run through country selection/onboarding)
  • once rewards has been enabled, go into Settings -> Notifications
  • toggle/disable All Brave - notifications from the Notifications page
  • tap on <- (back) to return into Settings and ensure that you don't get any warning modals
  • close the Settings page
  • go back into Settings -> Notifications
  • tap on <- (back) and ensure that you receive a generic warning re: notifications being disabled
  • ensured that taping on Not now dismisses the modal without any issues
  • ensured that tapping on Continue takes you back into the Notifications screen without an issues
    • ensured that going back into Settings didn't display the generic warning modal re: notifications being disabled
  • ensured that once either Not now or Continue has been tapped, the generic warning modal doesn't appear again
    • also ensured that the generic modal doesn't re-appear if the browser was closed while the modal was still opened
Example Example Example Example Example Example
Screenshot_20230112-205633 Screenshot_20230112-205647 Screenshot_20230112-205708 Screenshot_20230112-205720 Screenshot_20230112-210404 Screenshot_20230112-205736

Test Case #3 (Generic Modal/Warning) - Rewards disabled & Privacy Report enabled

  • launch 1.47.171 Chromium: 109.0.5414.87 and run through onboarding
  • tap on Continue via the Brave notifications make it easier modal
  • tap on Allow via the Allow Brave to send you notifications modal to enable notifications
  • tap on the Brave Stats via NTP and run through the Privacy Report onboarding (ensure you enable Privacy Report)
  • visit Settings -> Notifications and toggle/disable All Brave - notifications from the Notifications page
  • tap on <- (back) to return into Settings and ensure that you don't get any warning modals
  • close the Settings page
  • go back into Settings -> Notifications
  • tap on <- (back) and ensure that you receive a generic warning re: notifications being disabled
  • ensured that taping on Not now dismisses the modal without any issues
  • ensured that tapping on Continue takes you back into the Notifications screen without an issues
    • ensured that going back into Settings didn't display the generic warning modal re: notifications being disabled
  • ensured that once either Not now or Continue has been tapped, the generic warning modal doesn't appear again
    • also ensured that the generic modal doesn't re-appear if the browser was closed while the modal was still opened
  • ensured that there's a banner warning the user that Notifications are disabled within the Privacy Report
    • ensured that tapping on Turn on notifications redirects you to Settings -> Notifications
    • ensured that tapping on <- (back) within Notifications takes you back to the Privacy Report
    • ensured that warning re-appears if the user dismissed via X and then restarted the browser
Example Example Example Example Example Example Example Example
Screenshot_20230112-210742 Screenshot_20230112-210745 Screenshot_20230112-210932 Screenshot_20230112-210938 Screenshot_20230112-210950 Screenshot_20230112-210953 Screenshot_20230112-211001 Screenshot_20230112-211015

Test Case #4 (Generic Modal/Warning) - Rewards enabled & Privacy Report enabled

With the following case, basically follow Test Case #2 & Test Case #3 in terms of enabling both features and then disabling All Brave - notifications and ensuring you get a the generic warning modal as per the following:

Example Example Example Example Example Example Example
Screenshot_20230113-012456 Screenshot_20230113-012508 Screenshot_20230113-012525 Screenshot_20230113-012533 Screenshot_20230113-012542 Screenshot_20230113-012546 Screenshot_20230113-012559

Test Case #5 (Reward/Warning) - Rewards enabled & Privacy Report disabled

  • launch 1.47.171 Chromium: 109.0.5414.87 and run through onboarding
  • tap on Continue via the Brave notifications make it easier modal
  • tap on Allow via the Allow Brave to send you notifications modal to enable notifications
  • enable rewards via the rewards panel (run through country selection/onboarding)
  • once rewards has been enabled, go into Settings -> Notifications
  • toggle/disable All "Brave Ads" notifications from the Notifications page
  • tap on <- (back) to return into Settings and ensure that you get the You're not earning Brave Rewards warning modal
    • ensure that tapping on X or Got it dismiss the modal without any issues
    • ensured that the same modal never appears again when going back from Notifications -> Settings again
  • close the Settings page and restart the browser
  • ensure that you get the You're not earning Brave Rewards warning modal
  • ensured that taping on Not now dismisses the modal without any issues
  • ensured that tapping on Turn on Brave notifications takes you to the Settings -> Notifications page within Brave
  • ensured that once either Not now or Turn on Brave notifications has been tapped, the warning modal never appears again
    • also ensured that the rewards warning modal doesn't re-appear if the browser was closed while the modal was still opened
Example Example Example Example Example Example Example Example
Screenshot_20230113-012456 Screenshot_20230113-012508 Screenshot_20230113-012525 Screenshot_20230113-012533 Screenshot_20230113-012542 Screenshot_20230113-012546 Screenshot_20230113-012559 Screenshot_20230113-013052

Test Case #6 (Privacy Report/Warning) - Rewards disabled & Privacy Report enabled

  • launch 1.47.171 Chromium: 109.0.5414.87 and run through onboarding
  • tap on Continue via the Brave notifications make it easier modal
  • tap on Allow via the Allow Brave to send you notifications modal to enable notifications
  • enable Privacy Report by tapping on the Brave Stats and running through onboarding
  • once Privacy Report has been enabled, go into Settings -> Notifications
  • toggle/disable All "General" notifications from the Notifications page
  • tap on <- (back) to return into Settings and ensure that you get the You're not receiving Privacy Reports warning modal
    • ensure that tapping on X or Got it dismiss the modal without any issues
    • ensured that the same modal never appears again when going back from Notifications -> Settings again
  • close the Settings page and restart the browser
  • ensure that you get the You're not receiving Privacy Reports warning modal
  • ensured that taping on Not now dismisses the modal without any issues
  • ensured that tapping on Turn on Brave notifications takes you to the Settings -> Notifications page within Brave
  • ensured that once either Not now or Turn on Brave notifications has been tapped, the warning modal never appears again
    • also ensured that the privacy report warning modal doesn't re-appear if the browser was closed while the modal was still opened
Example Example Example Example Example Example Example
Screenshot_20230113-013935 Screenshot_20230113-013927 Screenshot_20230113-013924 Screenshot_20230113-013921 Screenshot_20230113-013912 Screenshot_20230113-013902 Screenshot_20230113-013858

Test Case #7 (Reward & Privacy Report/Warning) - Rewards enabled & Privacy Report enabled

With the following case, basically follow Test Case #5 & Test Case #6 in terms of enabling both features and then disabling All "Brave Ads" notifications & All "General" notifications and ensuring you get the combined warning modal as per the following:

Example Example Example Example Example Example
Screenshot_20230113-014457 Screenshot_20230113-014500 Screenshot_20230113-014510 Screenshot_20230113-014518 Screenshot_20230113-014533 Screenshot_20230113-014542

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

Successfully merging a pull request may close this issue.

4 participants