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

Duplicate entries added after entering correct code words #2130

Closed
srirambv opened this issue Nov 14, 2018 · 5 comments
Closed

Duplicate entries added after entering correct code words #2130

srirambv opened this issue Nov 14, 2018 · 5 comments

Comments

@srirambv
Copy link
Contributor

Description

Duplicate entries added after entering correct code words

Steps to Reproduce

  1. Launch brave beta with --enable-brave-sync
  2. Create sync chain on profile, copy code words
  3. Remove profile completely
  4. Create a new profile with --enable-brave-sync
  5. Paste the code from step 2 but misspell a word
  6. Click on Create Sync, no error message is shown for incorrect words typed in
  7. Correct misspell word and click on Create Sync, doesn't show sync chain created instantly, need to keep clicking on create sync
  8. Once sync chain is created shows the device listed from step 5

Actual result:

https://youtu.be/7UC3_ZoCdUY

Expected result:

  1. Should show warning message when wrong code words is entered
  2. Should have a waiter message when sync chain is being created
  3. Should not have previous attempts added as devices on the sync chain

Reproduces how often:

Easy

Brave version (brave://version info)

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

Reproducible on current release:

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

Website problems only:

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

Additional Information

cc: @brave/legacy_qa to verify on other platforms as well
cc: @darkdh @AlexeyBarabash @SergeyZhukovsky

@darkdh
Copy link
Member

darkdh commented Nov 15, 2018

I'm confused by duplicate entries. In your video, device list contains "Linux" and "Test" is expected. Removing profile doesn't mean it will be removed from sync chain

@AlexeyBarabash
Copy link
Contributor

Agree with @darkdh .
Expected pt.3 Should not have previous attempts added as devices on the sync chain could not be implemented because sync seed and device id are stored in profile.

@AlexeyBarabash AlexeyBarabash self-assigned this Nov 22, 2018
@AlexeyBarabash
Copy link
Contributor

Expected pt.1 Should show warning message when wrong code words is entered has own issue assigned
#2103

@AlexeyBarabash
Copy link
Contributor

Splitted expected result points by issues:
Should show warning message when wrong code words is entered
#2103
#2128
Should have a waiter message when sync chain is being created
#2227
Should not have previous attempts added as devices on the sync chain
it cannot be fixed

@bbondy bbondy modified the milestones: 0.57.x - Beta, 1.x Backlog Nov 27, 2018
@darkdh
Copy link
Member

darkdh commented Dec 5, 2018

as #2130 (comment) stated,
some phenomenons are duplicated of other issues and duplicate devices scenario is invalid
so close the issue here

@darkdh darkdh closed this as completed Dec 5, 2018
@darkdh darkdh modified the milestones: 1.x Backlog, 0.58.x - Beta, Dupe / Invalid / Not actionable Dec 5, 2018
@darkdh darkdh added closed/duplicate Issue has already been reported and removed QA/Yes labels Dec 5, 2018
@bbondy bbondy removed this from the Dupe / Invalid / Not actionable milestone May 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants