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

Brave sync show error in UI on sync-setup-error #2103

Closed
AlexeyBarabash opened this issue Nov 12, 2018 · 6 comments · Fixed by brave/brave-core#971
Closed

Brave sync show error in UI on sync-setup-error #2103

AlexeyBarabash opened this issue Nov 12, 2018 · 6 comments · Fixed by brave/brave-core#971
Assignees
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/sync priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/exclude

Comments

@AlexeyBarabash
Copy link
Contributor

AlexeyBarabash commented Nov 12, 2018

Description

Error from sync-setup-error is displayed only in logs, need to make to show it in UI.

Examples of the situations when error is occurred:

  1. No internet connection
  2. Wrong code phrase
  3. Empty code phrase
  4. No device name (added here -- will be removed in 0.58.x)

Steps to Reproduce

  1. Run Brave enabling Sync via brave --enable-brave-sync
  2. Open page brave://bravesync/
  3. Click I have an existing sync code

Case 1 - No internet connection

  1. Open Brave devtools, go to network tab -> click the "offline" toggle
  2. Click Set up Sync
  3. Should show the appropriate alert message

Case 2 - Wrong code phrase

  1. Enter wrong words noise elite faculty between enforce soul happy talk nut anxiety minute flower follow cradle act road hollow lazy unlock glove brush pepper buyer wrongword
  2. Click Set up Sync
  3. Should show the appropriate alert message

Case 3 - Empty Code Phrase

  1. Leave the Code Phrase empty
  2. Type something for device name
  3. Click Set up Sync
  4. Should show the appropriate alert message

Case 3 - No device name

  1. Enter right words `borrow other exile curve inflict monster jacket piece shrug spider shock clip art describe task fancy odor such bargain legal embody sibling course labor
  2. Type something for device name
  3. Click Set up Sync
  4. Should show the appropriate alert message

Actual result:

No error in UI. Only console log has message wrong code words

Expected result:

Error displayed in UI.

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

0.59.1 Chromium: 71.0.3578.44 (Developer Build)

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    yes
@bbondy bbondy added this to the 1.x Backlog milestone Nov 13, 2018
@LaurenWags
Copy link
Member

Reproduced with below build, except I do not see 'wrong code words' in terminal. There is no indication in the UI of what I did wrong.

Brave 0.57.6 Chromium: 71.0.3578.31 (Official Build) beta(64-bit)
Revision c88fdf2a4ce19a713615ca4fbde7a0d0b5fe2363-refs/branch-heads/3578@{#427}
OS Mac OS X

@cezaraugusto
Copy link
Contributor

cc @bradleyrichter @rossmoody

@cezaraugusto cezaraugusto self-assigned this Nov 15, 2018
@rebron rebron added design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. priority/P4 Planned work. We expect to get to it "soon". priority/P3 The next thing for us to work on. It'll ride the trains. and removed priority/P4 Planned work. We expect to get to it "soon". labels Nov 16, 2018
@darkdh darkdh modified the milestones: 1.x Backlog, 0.57.x - Beta Nov 27, 2018
@cezaraugusto
Copy link
Contributor

heads-up I edited this issue description with states that the fixing PR address. The one that doesn't is referenced in #2262 and will be addressed separately.

@bbondy
Copy link
Member

bbondy commented Nov 29, 2018

Pls no tracking open issues inside milestones. Moving out.
We use project boards per category now to triage importance.

@bbondy bbondy modified the milestones: 0.57.x - Beta, 1.x Backlog Nov 29, 2018
cezaraugusto added a commit to brave/brave-core that referenced this issue Nov 30, 2018
fix brave/brave-browser#2103
-
error states added:

- no internet
- wrong sync words
- empty sync words
- missing device name
@cezaraugusto cezaraugusto modified the milestones: 1.x Backlog, 0.58.x - Beta Nov 30, 2018
@srirambv
Copy link
Contributor

srirambv commented Dec 12, 2018

Verification passed on

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta(64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Linux

Invalid sync code( Wrong code phrase)

  • Tried without entering sync code
  • Tried with incorrect number of sync code words
  • Tried with spelling mistakes
    image

Device name is required

  • Verified without entering a device name
    image

No internet connection (With network disabled)
screenshot from 2018-12-12 17-17-27

Verification passed on

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Windows

Invalid sync code( Wrong code phrase)

  • Tried without entering sync code
  • Tried with incorrect number of sync code words
  • Tried with spelling mistakes

Without entering sync code
image

Incorrect number of sync code
image

Code word with spelling mistake

image

Entered Wrong Code

  • Entered wrong code and got the below alert message to enter device name
    image
  • Verified after entering the device name, alert message is displayed as invalid code
    image

Correct sync code, but network is disconnected
image

Verified passed with

Brave 0.58.11 Chromium: 71.0.3578.80 (Official Build) beta(64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Mac OS X

No internet connection
screen shot 2018-12-12 at 4 46 19 pm
screen shot 2018-12-12 at 4 47 26 pm

Wrong code phrase
screen shot 2018-12-12 at 4 48 01 pm

Empty code phrase
screen shot 2018-12-12 at 4 48 19 pm

No device name
screen shot 2018-12-12 at 4 51 43 pm

@khezsowavy
Copy link

Description

Error from sync-setup-error is displayed only in logs, need to make to show it in UI.

Examples of the situations when error is occurred:

  1. No internet connection

  2. Wrong code phrase

  3. Empty code phrase

  4. No device name (added here -- will be removed in 0.58.x)

Steps to Reproduce

  1. Run Brave enabling Sync via brave --enable-brave-sync

  2. Open page brave://bravesync/

  3. Click I have an existing sync code

Case 1 - No internet connection

  1. Open Brave devtools, go to network tab -> click the "offline" toggle

  2. Click Set up Sync

  3. Should show the appropriate alert message

Case 2 - Wrong code phrase

  1. Enter wrong words noise elite faculty between enforce soul happy talk nut anxiety minute flower follow cradle act road hollow lazy unlock glove brush pepper buyer wrongword

  2. Click Set up Sync

  3. Should show the appropriate alert message

Case 3 - Empty Code Phrase

  1. Leave the Code Phrase empty

  2. Type something for device name

  3. Click Set up Sync

  4. Should show the appropriate alert message

Case 3 - No device name

  1. Enter right words `borrow other exile curve inflict monster jacket piece shrug spider shock clip art describe task fancy odor such bargain legal embody sibling course labor

  2. Type something for device name

  3. Click Set up Sync

  4. Should show the appropriate alert message

Actual result:

No error in UI. Only console log has message wrong code words

Expected result:

Error displayed in UI.

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

0.59.1 Chromium: 71.0.3578.44 (Developer Build)

Reproducible on current release:

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

yes

issue is unresolved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. feature/sync priority/P3 The next thing for us to work on. It'll ride the trains. QA Pass-Linux QA Pass-macOS QA Pass-Win64 QA/Test-Plan-Specified QA/Yes release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants