Firefox/Sync/Progress Report/2017Q3: Difference between revisions

Jump to navigation Jump to search
Finished updates
(Finished updating objective 2)
(Finished updates)
Line 170: Line 170:
|align="center"|
|align="center"|
|Version-based syncing of bookmarks
|Version-based syncing of bookmarks
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1364644 Version-based syncing of bookmarks]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1364644 Bug 1364644: Version-based syncing of bookmarks]
|Code is functionally complete and tests are being written. Review is pending from rnewman. Timeframe is very tight. Landing in 56 is still feasible, but there will most likely be uplifts if it makes 56. The preference is to deploy this feature in 56, even if this means a bumpier beta than we'd like, than risk having things blow up in 57. If code isn't tested and signed off by July 28th code freeze, Fennec Bookmark Management will most likely have to wait for 57.
|
*Agreed to move bookmark management to 57. Work itself is largely done. Some remaining changes will be done this week.
This work will make the work for 3.2 easier.
|-
|-
| align="left"|3.2 Prevent re-ordering of bookmarks on Android.
| align="left"|3.2 Prevent re-ordering of bookmarks on Android.
Line 185: Line 187:
|align="left"|
|align="left"|
Prevention of bookmark reordering on Android.
Prevention of bookmark reordering on Android.
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1352947 Android writes many root folders with children missing or in the wrong order]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1352947 Bug 1352947: Android writes many root folders with children missing or in the wrong order]
|Haven’t touched yet.
|
*Have made progress on 3.1.
|-
|-
| align="left"|3.3 Analyze and prevent erroneous bookmark deduping on Desktop.
| align="left"|3.3 Analyze and prevent erroneous bookmark deduping on Desktop.
Line 200: Line 203:
|align="left"|
|align="left"|
Feature description(s)
Feature description(s)
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1323333 Dodgy bookmark deduping]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1323333 Bug 1323333: Dodgy bookmark deduping]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1366888 Only consider unsynced local bookmarks for deduping]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1366888 Bug 1366888: Only consider unsynced local bookmarks for deduping]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1305563 Buffering and two-phase application for Desktop]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1305563 Bug 1305563: Buffering and two-phase application for Desktop]
|Reading through ios bookmarks merger and seeing how much we can port to Desktop. Intent is to land two-phase before deduping in 57, but also possible to land existing deduping patches first if we run out of time.
|
*Understand how iOS merge works now. Ported it to desktop. Merged trees can apply back to Places now.
|-
|-
| align="left"|3.4 Batch upload API enabled for all users, with more than 99% of uploads fitting into a single atomic batch.
| align="left"|3.4 Batch upload API enabled for all users, with more than 99% of uploads fitting into a single atomic batch.
Line 214: Line 218:
|align="center"|
|align="center"|
|align="left"|
|align="left"|
Feature description
Batch upload API enabled
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1378567 [meta] Enable batch API v3]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1378567 Bug 1378567: [meta] Enable batch API v3]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1378569 Increase max records per batch so ~99% of sync users are able to post all their bookmarks in a single batch]
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1378569 Bug 1378569: Increase max records per batch so ~99% of sync users are able to post all their bookmarks in a single batch]
|Discussions have started about what needs to be done.
|
*Work is progressing well.
|-
|-
|}
|}

Navigation menu