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

Editing bookmark name on laptop is not propagated to Android #119

Closed
srirambv opened this issue Jun 21, 2017 · 3 comments
Closed

Editing bookmark name on laptop is not propagated to Android #119

srirambv opened this issue Jun 21, 2017 · 3 comments
Milestone

Comments

@srirambv
Copy link

@srirambv srirambv commented Jun 21, 2017

  • Did you search for similar issues before submitting this one?
    Yes

  • Describe the issue you encountered:
    Editing bookmark name on laptop is not propagated to Android

  • Platform (Win7, 8, 10? macOS? Linux distro?):
    Pyramid 0 - Laptop
    Pyramid 1 - Android

  • Brave Version (revision SHA):
    Pyramid 0 - Laptop (0.17.4)
    Pyramid 1 - Android (1.0.24)

  • Steps to reproduce:

    1. Sync Android with laptop with QR code
    2. Import bookmarks on laptop with nested bookmark folder
    3. Edit any nested folder name on Android, changes propagate to laptop
    4. Edit any nested folder name on laptop, changes are not propagate to Android
  • Actual result:
    Editing bookmark name on laptop is not propagated to Android

  • Expected result:
    Change propagation should be bi-directional

  • Will the steps above reproduce in a fresh profile? If not what other info can be added?
    Yes

  • Is this an issue in the currently released version?
    N/A

  • Can this issue be consistently reproduced?
    Yes

  • Extra QA steps:
    1.
    2.
    3.

  • Screenshot if needed:
    image
    image

  • Any related issues:
    #111

@alexwykoff alexwykoff added this to the pyramid 2 milestone Jun 29, 2017
@diracdeltas
Copy link
Member

@diracdeltas diracdeltas commented Jun 29, 2017

could you re-try with latest 0.17.x? if it's broken there, please try master @srirambv

@ayumi
Copy link
Contributor

@ayumi ayumi commented Jul 6, 2017

@srirambv we did a sync lib update recently; is this issue still happening? cc @SergeyZhukovsky

@srirambv
Copy link
Author

@srirambv srirambv commented Jul 10, 2017

Verified works fine with 0.17.14 and 1.0.24 Android

@srirambv srirambv closed this Jul 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.