Skip to content
This repository has been archived by the owner on May 10, 2024. It is now read-only.

VPN Reset Configuration doesn't have a clear confirmation if the reset was done already #8087

Closed
hffvld opened this issue Sep 18, 2023 · 1 comment · Fixed by #8122
Closed

Comments

@hffvld
Copy link
Collaborator

hffvld commented Sep 18, 2023

Description:

VPN Reset Configuration doesn't have a clear confirmation if the reset was done already.

Steps to Reproduce

  1. Launch Brave
  2. Three-dot menu > VPN
  3. Buy a subscription and Install VPN profile
  4. Three-dot menu > Settings > Brave Firewall + VPN
  5. Reset Configuration > Reset > Observe

Actual result:

There is no clear confirmation shown when resetting VPN configuration.

221121.mov

Expected result:

When resetting VPN configuration user must clearly see that the reset is working.

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easily reproduced

Brave Version:

Brave build: 1.57.1 (23.9.18.13)
BraveCore: 1.57.53 (116.0.5845.114)

  • Can you reproduce this issue with the most recent build from TestFlight? YES
  • Can you reproduce this issue with the previous version of the current build from TestFlight? YES
  • Can you reproduce this issue with the current build from AppStore? NA

Device details:

iPhone 14 [iOS 17.0]

Website problems only:

  • did you check with Brave Shields down?
  • did you check in Safari/Firefox (WkWebView-based browsers)?

Additional Information

@hffvld
Copy link
Collaborator Author

hffvld commented Nov 6, 2023

Verified on iPhone 14 using version(s):

Device/OS: iPhone 14 [iOS 17.1]
Brave build: 1.60 (23.11.4.14)
BraveCore: 1.60.105 (119.0.6045.59)

STEPS:

  1. Followed the steps from VPN Reset Configuration doesn't have a clear confirmation if the reset was done already #8087 (comment)

ACTUAL RESULTS:

  • Verified that the confirmation message is shown when successfully Reset Configuration

1 2
1 2
2023-11-06_15-22-51.mp4

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

Successfully merging a pull request may close this issue.

3 participants