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 changes only for bookmarks and bookmark folders #48

Closed
ayumi opened this issue Jan 23, 2017 · 0 comments
Closed

Sync changes only for bookmarks and bookmark folders #48

ayumi opened this issue Jan 23, 2017 · 0 comments

Comments

@ayumi
Copy link
Contributor

@ayumi ayumi commented Jan 23, 2017

Similar to 434965a which added SyncRecord.siteSetting.fields to sync only changes.

@ayumi ayumi mentioned this issue Jan 23, 2017
ayumi added a commit to brave/browser-laptop that referenced this issue Jan 24, 2017
ayumi added a commit to brave/browser-laptop that referenced this issue Jan 24, 2017
Fix brave/sync#48

Auditors: @diracdeltas

Test plan:
1. Start Pyramid 0. It should have bookmarks and bookmark folders.
2. Enable Sync on Pyramid 0, and restart to complete setup.
3. Start Pyramid 1. Add it to Pyramid 0's Sync profile and restart.
4. Bookmarks should sync in the same order.
ayumi added a commit to brave/browser-laptop that referenced this issue Jan 24, 2017
Fix brave/sync#48

Auditors: @diracdeltas

Test plan:
1. Start Pyramid 0. It should have bookmarks and bookmark folders.
2. Enable Sync on Pyramid 0, and restart to complete setup.
3. Start Pyramid 1. Add it to Pyramid 0's Sync profile and restart.
4. Bookmarks should sync in the same order.
ayumi added a commit to brave/browser-laptop that referenced this issue Jan 24, 2017
Fix brave/sync#48

Auditors: @diracdeltas

Test plan:
1. Start Pyramid 0. It should have bookmarks and bookmark folders.
2. Enable Sync on Pyramid 0, and restart to complete setup.
3. Start Pyramid 1. Add it to Pyramid 0's Sync profile and restart.
4. Bookmarks should sync in the same order.
ayumi added a commit to brave/browser-laptop that referenced this issue Jan 24, 2017
Fix brave/sync#48

Auditors: @diracdeltas

Test plan:
1. Start Pyramid 0. It should have bookmarks and bookmark folders.
2. Enable Sync on Pyramid 0, and restart to complete setup.
3. Start Pyramid 1. Add it to Pyramid 0's Sync profile and restart.
4. Bookmarks should sync in the same order.
ayumi added a commit to brave/browser-laptop that referenced this issue Jan 27, 2017
Fix brave/sync#48

Test plan:
1. Start Pyramid 0. It should have bookmarks and bookmark folders.
2. Enable Sync on Pyramid 0, and restart to complete setup.
3. Start Pyramid 1. Add it to Pyramid 0's Sync profile and restart.
4. Bookmarks should sync in the same order.
ayumi added a commit to brave/browser-laptop that referenced this issue Jan 27, 2017
Fix brave/sync#48

Test plan:
1. Start Pyramid 0. It should have bookmarks and bookmark folders.
2. Enable Sync on Pyramid 0, and restart to complete setup.
3. Start Pyramid 1. Add it to Pyramid 0's Sync profile and restart.
4. Bookmarks should sync in the same order.
ayumi added a commit to brave/browser-laptop that referenced this issue Feb 1, 2017
Re-introduce SyncRecord.Bookmark.index after Sites Map refactor.

(Re)-Fix brave/sync#48

Auditors: @diracdeltas

Test plan:
1. Start Pyramid 0. It should have bookmarks and bookmark folders.
2. Enable Sync on Pyramid 0, and restart to complete setup.
3. Start Pyramid 1. Add it to Pyramid 0's Sync profile and restart.
4. Bookmarks should sync onto Pyramid 1 in the same order as Pyd 0.
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
1 participant
You can’t perform that action at this time.