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: move bookmark into folder not reflected #2512

Closed
AlexeyBarabash opened this issue Dec 12, 2018 · 2 comments · Fixed by brave/brave-core#1079
Closed

Sync: move bookmark into folder not reflected #2512

AlexeyBarabash opened this issue Dec 12, 2018 · 2 comments · Fixed by brave/brave-core#1079

Comments

@AlexeyBarabash
Copy link
Contributor

From slack messages from @darkdh and @petemill

Description

Move bookmark into folder on device A is not reflected on synced device B

Steps to Reproduce

  1. Create sync chain with device A
  2. Connect device B to sync chain
  3. On device A create bookmark a.com in a root of Bookmarks bar
  4. On device A create folder F1 in a root of Bookmarks bar
  5. Wait on device B for entities get synced
  6. On device A move a.com into F1

Actual result:

7. On device B a.com is not inside of F1

Expected result:

7. On device B a.com is inside of F1

Reproduces how often:

Easy

Brave version (brave://version info)

Brave 0.61.0 Chromium: 71.0.3578.80 (Developer Build) (64-bit)
Revision 2ac50e7249fbd55e6f517a28131605c9fb9fe897-refs/branch-heads/3578@{#860}
OS Linux
@bbondy
Copy link
Member

bbondy commented Dec 18, 2018

Please remember to update issue milestones to match where the associated PR landed. Thanks!

@LaurenWags
Copy link
Member

LaurenWags commented Dec 18, 2018

Verified passed with

Brave 0.58.14 Chromium: 71.0.3578.98 (Official Build) (64-bit)
Revision 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS Mac OS X
  • Verified STR in description

Verification passed on

Brave 0.58.14 Chromium: 71.0.3578.98 (Official Build) (64-bit)
Revision 15234034d19b85dcd9a03b164ae89d04145d8368-refs/branch-heads/3578@{#897}
OS Windows 7

Used STR from OP

Verification passed on

Brave 0.59.14 Chromium: 72.0.3626.28 (Official Build) beta(64-bit)
Revision 997b1040b63bac324e815797ba52be0cd8f616ed-refs/branch-heads/3626@{#461}
OS Linux

Used STR from OP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment