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

VPN notification not appearing due to Notification permission timing (Android 13+) #30624

Open
kjozwiak opened this issue May 26, 2023 · 2 comments

Comments

@kjozwiak
Copy link
Member

Description

Because we delayed the Notification permission modal on Android 13+, it's possible that a new user buys/enables VPN but won't see the VPN Notification that displays the amount of data that's being routed through the VPN including the current location. Once the user receives the modal and accepts Brave notifications, the VPN entry will appear via Androids notification tray.

Maybe we can improve the timing? Or prompt the user for notification permissions if they're enabling VPN and they still haven't received the modal due to being a new install/user.

Some context, permissions modal was moved to appear later via #27644.

Steps to Reproduce

  1. download/install Brave (new profile)
  2. purchase/enable VPN and you'll notice that the VPN notification won't appear till the user receives the permissions modal

Actual result:

Expected result:

When a user enabled VPN, we should be displaying the VPN notification. If the user hasn't received the permission modal, maybe display it when VPN is being enabled but a permission decision hasn't been made.

Reproduces how often:

100% reproducible using the STR/Cases outlined above.

Brave version (brave://version info)

Brave | 1.52.113 Chromium: 114.0.5735.53 (Official Build) (64-bit)
--- | ---
Revision | c499d7ea22c8b2dba278465a5df7b86a8efa4e64-refs/branch-heads/5735@{#970}
OS | Android 13; Build/TP1A.220624.014; 33; REL

Version/Channel Information:

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

Other Additional Information:

  • 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

Miscellaneous Information:

@kjozwiak kjozwiak added QA/Yes release-notes/include OS/Desktop privacy/permissions privacy features related to limiting, lifetime or other permissions feature/vpn labels May 26, 2023
@deeppandya
Copy link

@kjozwiak after this change : brave/brave-core#18641. without notification, VPN feature should work with out notification. would please confirm ?

@kjozwiak
Copy link
Member Author

@kjozwiak after this change : brave/brave-core#18641. without notification, VPN feature should work with out notification. would please confirm ?

It's not related to brave/brave-core#18641. Basically the issue is if a user enables/restores a VPN before they give Brave notification permissions, users won't see the VPN notification until they Allow. So the above issue is to see if we can possibly improve the timing or trigger the Notification modal when a user enables VPN.

@ShivanKaul ShivanKaul removed the privacy/permissions privacy features related to limiting, lifetime or other permissions label Jun 6, 2023
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

4 participants