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

handle multiple error cases for sync #971

Merged
merged 1 commit into from
Nov 30, 2018
Merged

Conversation

cezaraugusto
Copy link
Contributor

fix brave/brave-browser#2103

error states added:

  • no internet
  • wrong sync words
  • empty sync words
  • missing device name

Test Plan:

  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 4 - 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. Leave device name empty
  3. Click Set up Sync
  4. Should show the appropriate alert message

@cezaraugusto cezaraugusto self-assigned this Nov 28, 2018
@cezaraugusto cezaraugusto requested a review from darkdh November 28, 2018 02:01
@darkdh
Copy link
Member

darkdh commented Nov 28, 2018

blocked by #973
don't merge this before #973

components/brave_sync/ui/constants/sync_types.ts Outdated Show resolved Hide resolved
components/brave_sync/ui/reducers/sync_reducer.ts Outdated Show resolved Hide resolved
components/brave_sync/ui/reducers/sync_reducer.ts Outdated Show resolved Hide resolved
@cezaraugusto cezaraugusto force-pushed the ca-sync--setup-err branch 2 times, most recently from 6b766be to 6233226 Compare November 28, 2018 22:14
darkdh added a commit that referenced this pull request Nov 29, 2018
fix brave/brave-browser#2103
-
error states added:

- no internet
- wrong sync words
- empty sync words
- missing device name
Copy link
Member

@darkdh darkdh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

++, ERR_SYNC_INIT_FAILED will be handled as follow-up
brave/brave-browser#2316

@cezaraugusto cezaraugusto merged commit 456768c into master Nov 30, 2018
@cezaraugusto cezaraugusto deleted the ca-sync--setup-err branch November 30, 2018 22:27
cezaraugusto added a commit that referenced this pull request Nov 30, 2018
handle multiple error cases for sync
cezaraugusto added a commit that referenced this pull request Nov 30, 2018
handle multiple error cases for sync
@cezaraugusto
Copy link
Contributor Author

master 456768c
0.59.x f5963d8
0.58.x 4cd22cf

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

Successfully merging this pull request may close these issues.

Brave sync show error in UI on sync-setup-error
3 participants