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

Sync available in Settings - follow up to 2609 #2642

Closed
LaurenWags opened this issue Dec 19, 2018 · 3 comments
Closed

Sync available in Settings - follow up to 2609 #2642

LaurenWags opened this issue Dec 19, 2018 · 3 comments

Comments

@LaurenWags
Copy link
Member

Description

Follow up to #2609

Build 0.58.15 should have Sync disabled by default. However, when navigating to brave://settings, Sync is still available in Settings.

Steps to Reproduce

  1. Install 0.58.15
  2. Navigate to brave://settings
  3. Search for Sync

Actual result:

Sync is displayed in settings:
screen shot 2018-12-19 at 10 46 59 am

Expected result:

Since sync is disabled by default due to #2609, it should not be displayed in settings.

Reproduces how often:

easily

Brave version (brave://version info)

Brave 0.58.15 Chromium: 71.0.3578.98 (Official Build) (64-bit)
Revision 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds? n/a

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Additional Information

As expected, sync is not available from hamburger menu and navigating to brave://sync results in an error.

@darkdh
Copy link
Member

darkdh commented Dec 19, 2018

For channels other than 0.58.x, we need to check if --disable-brave-sync hides sync in settings

@darkdh darkdh self-assigned this Dec 19, 2018
@darkdh darkdh modified the milestones: 1.x Backlog, 0.58.x - Release Dec 19, 2018
@darkdh
Copy link
Member

darkdh commented Dec 19, 2018

For channels other than 0.58.x, we need to check if --disable-brave-sync hides sync in settings

will be addressed in #2650

closed by brave/brave-core#1146

@darkdh darkdh closed this as completed Dec 19, 2018
Sync automation moved this from Untriaged backlog to Completed Dec 19, 2018
@LaurenWags
Copy link
Member Author

LaurenWags commented Dec 19, 2018

Verified passed with

Brave 0.58.16 Chromium: 71.0.3578.98 (Official Build) (64-bit)
Revision 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS Mac OS X

Verified passed with

Brave 0.58.16 Chromium: 71.0.3578.98 (Official Build) (64-bit)
Revision 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS Linux

Verification passed on

Brave 0.58.16 Chromium: 71.0.3578.98 (Official Build) (64-bit)
Revision 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS Windows 7

Verification passed on

Brave 0.58.16 Chromium: 71.0.3578.98 (Official Build) (64-bit)
Revision 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS Windows 10

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