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: deleted node becomes folder in Other bookmarks #2449

Closed
AlexeyBarabash opened this issue Dec 11, 2018 · 1 comment · Fixed by brave/brave-core#1061
Closed

Sync: deleted node becomes folder in Other bookmarks #2449

AlexeyBarabash opened this issue Dec 11, 2018 · 1 comment · Fixed by brave/brave-core#1061

Comments

@AlexeyBarabash
Copy link
Contributor

Description

Deleted node becomes folder in Other bookmarks

Steps to Reproduce

  1. Create sync chain, device BC-A;
  2. Connect to sync chain with device BC-B with code words;
  3. On device BC-A create in Bookmarks bar bookmarks A.com, B.com, C.com
  4. Delete bookmark B.com on device BC-A fast after pt.3
  5. On device BC-A create in Bookmarks bar folders F1, F2, F3
  6. Delete folder F2 on device BC-A fast after pt.5

Actual result:

  1. See folders B.com and F2 in Other bookmarks on device BC-B

Expected result:

  1. folders B.com and F2 in Other bookmarks on device BC-B are not created.

Reproduces how often:

Easily reproduced

Brave version (brave://version info)

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

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds?
    yes
@AlexeyBarabash AlexeyBarabash self-assigned this Dec 11, 2018
@AlexeyBarabash AlexeyBarabash added this to Untriaged backlog in Sync Dec 11, 2018
@AlexeyBarabash AlexeyBarabash moved this from Untriaged backlog to Inprogress in Sync Dec 11, 2018
Sync automation moved this from Inprogress to Completed Dec 12, 2018
@AlexeyBarabash AlexeyBarabash added this to the 0.60.x - Dev milestone Dec 13, 2018
@darkdh darkdh modified the milestones: 0.60.x - Dev, 0.59.x - Beta Dec 20, 2018
@GeetaSarvadnya
Copy link

GeetaSarvadnya commented Dec 27, 2018

Verification passed on

Brave 0.59.12 Chromium: 72.0.3626.17 (Official Build) beta (64-bit)
Revision 08d2140079f8db14c9eb885a803d231b1a30e817-refs/branch-heads/3626@{#315}
OS Windows
  • Verified the STR mentioned in the issue.
    image

Verification passed on

Brave 0.59.12 Chromium: 72.0.3626.17 (Official Build) beta (64-bit)
Revision 08d2140079f8db14c9eb885a803d231b1a30e817-refs/branch-heads/3626@{#315}
OS Linux

  • Verified the STR mentioned in the issue.
    image

Verified passed with

Brave 0.59.14 Chromium: 72.0.3626.28 (Official Build) beta(64-bit)
Revision 997b1040b63bac324e815797ba52be0cd8f616ed-refs/branch-heads/3626@{#461}
OS Mac OS X
  • Verified STR from description

@rebron rebron removed this from Completed in Sync Mar 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment