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

clean up the text that warns users that their device time is wrong when syncing #4011

Closed
kjozwiak opened this issue Apr 4, 2019 · 1 comment
Closed

Comments

@kjozwiak
Copy link
Member

@kjozwiak kjozwiak commented Apr 4, 2019

Description

We'll need to clean up the following text before this warning makes it out from nightly --> dev. It's currently in 0.65.

You can find the current text in the following example:

brave/brave-core#2155 (comment)

We should get this done sooner than later as this resolves a common support issues where users are receiving errors due to their times being wrong.

Steps to Reproduce

  1. #3962

Actual result:

Example of current text: brave/brave-core#2155 (comment)

Expected result:

We'll need text from @bradleyrichter's team.

Reproduces how often:

100% reproducible using the STR mentioned above.

Brave version (brave://version info)

Brave 0.65.17 Chromium: 74.0.3729.40 (Official Build) nightly(64-bit)
Revision 278d6a0d856d8314f36fb9a68b6e1d85cd0d14e7-refs/branch-heads/3729@{#470}
OS Mac OS X

Reproducible on current release:

N/A

  • Does it reproduce on brave-browser dev/beta builds? N/A. Change is currently only in the nightly build.

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

CCing @bradleyrichter @rebron @AlexeyBarabash

@kjozwiak kjozwiak added this to Untriaged backlog in Sync via automation Apr 4, 2019
@kjozwiak kjozwiak mentioned this issue Apr 4, 2019
7 of 19 tasks complete
@rebron rebron removed this from Untriaged backlog in Sync Apr 5, 2019
@rebron
Copy link
Collaborator

@rebron rebron commented Apr 5, 2019

Close. Text review is in the pr.

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

Successfully merging a pull request may close this issue.

None yet
5 participants
You can’t perform that action at this time.