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

[Follow up to #7860] "Default browser prompt" must not be shown after 14 days since installation #7977

Closed
hffvld opened this issue Aug 28, 2023 · 3 comments
Assignees
Milestone

Comments

@hffvld
Copy link
Collaborator

hffvld commented Aug 28, 2023

Description:

Follow-up to #7860

Steps to Reproduce

  1. Fresh install and launch Brave
  2. Start onboarding flow > Don't set Brave as the default browser and select Not now
  3. Tap Done > Wait until landed on NTP > Confirm there is no Default browser prompt is shown
  4. Device Settings > Update the date to be <7 days from today
  5. Relaunch Brave > Confirm there is no Default browser prompt on NTP
  6. Now update the date between 7 and 14 days from today
  7. Relaunch Brave > Confirm that Default browser prompt is shown on NTP
  8. Now update the date to be>14 days from today
  9. Relaunch Brave > Observe, Default browser prompt still shown

Actual result:

Observed that Default browser prompt is still shown after 14 days since installation, and the prompt actually removed only on the 18th day after installation.

default_14days.mov

Expected result:

Default browser prompt must be removed after 14 days since Brave installation.

Reproduces how often: [Easily reproduced, Intermittent Issue]

Easily reproduced

Brave Version:

Brave build: 1.57 (23.8.28.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? NA
  • 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

@soner-yuksel
Copy link
Contributor

@vlad-hoff There was typo and 17 days was used instead of 14 but it should be fixed with #7969, it can be great if you can re-stest.
Also while testing better to count 7 days after and 14 days after cause we are not checking this fully perfect tike diffrence here(it is not necessary only days difference)

RPReplay_Final1694612715.MP4

@soner-yuksel soner-yuksel added the blocked: needs info Needs further information before work can commence label Aug 29, 2023
@iccub
Copy link
Contributor

iccub commented Aug 29, 2023

Please retest on the latest build 1.57 (23.8.29.17)

@iccub iccub closed this as completed Aug 29, 2023
@iccub iccub added this to the 1.57 milestone Aug 29, 2023
@hffvld
Copy link
Collaborator Author

hffvld commented Aug 30, 2023

Verified on iPhone 14 using version(s):

Device/OS: iPhone 14 [iOS 17.0]
Brave build: 1.57 (23.8.29.22)
BraveCore: 1.57.53 (116.0.5845.114)

STEPS:
Followed the steps from #7977 (comment)

ACTUAL RESULTS:

  • Verified that Default browser prompt is not shown on NTP after 14 days since installation

Default_fixed.mov

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

No branches or pull requests

4 participants