670
edits
Firefox/Sync/Progress Report/2017Q3: Difference between revisions
Jump to navigation
Jump to search
Progress updates for July 25th
(completed adding objective 2) |
(Progress updates for July 25th) |
||
| Line 1: | Line 1: | ||
__NOTOC__ | __NOTOC__ | ||
<big><big>'''July | <big><big>'''July 25, 2017'''</big></big> | ||
== OKR Progress == | == OKR Progress == | ||
| Line 38: | Line 36: | ||
Bookmarks are Sync'd from iOS to desktop. | Bookmarks are Sync'd from iOS to desktop. | ||
* [https://bugzilla.mozilla.org/show_bug.cgi?id=1380060 Sync new local bookmarks into Mobile Bookmarks] | * [https://bugzilla.mozilla.org/show_bug.cgi?id=1380060 Sync new local bookmarks into Mobile Bookmarks] | ||
| | |Had a good session with Garvan from iOS team. Know what we need to do. | ||
|- | |- | ||
|align="left"| 1.2 Users can delete their mobile bookmarks from their iOS device. | |align="left"| 1.2 Users can delete their mobile bookmarks from their iOS device. | ||
| Line 68: | Line 66: | ||
|align="center"|% | |align="center"|% | ||
|align="left"| | |align="left"| | ||
Validated validation data | |||
*[https://bugzilla.mozilla.org/show_bug.cgi?id=1381642 | *[https://bugzilla.mozilla.org/show_bug.cgi?id=1381642 Validate bookmark validator dashboards with ad-hoc analysis] | ||
|No work done on this. Plan to work on it this week. | |No work done on this. Plan to work on it this week. | ||
|- | |- | ||
| Line 133: | Line 131: | ||
|Dependent upon 2.1 - 2.4. | |Dependent upon 2.1 - 2.4. | ||
|- | |- | ||
|rowspan=4| 3. | |rowspan=4| 3. To prevent bookmark corruption. | ||
| align="left"|3.1 | | align="left"|3.1 Land the bookmark Sync tracker on Android. | ||
|align="center"|% | |||
|align="center"|% | In support of Full Bookmark Management that’s being introduced in 56. | ||
Needs to land in nightly by: July 31st (56 goes to Beta on August 7). | |||
|align="center"|50% | |||
|align="center"|70% | |||
|align="center"|% | |align="center"|% | ||
|align="center"|% | |align="center"|% | ||
| Line 142: | Line 145: | ||
|align="center"|% | |align="center"|% | ||
|align="center"|% | |align="center"|% | ||
| | |Version-based syncing of bookmarks | ||
| | *[https://bugzilla.mozilla.org/show_bug.cgi?id=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. | |||
|- | |- | ||
| align="left"|3.2 | | align="left"|3.2 Prevent re-ordering of bookmarks on Android. | ||
|align="center"|% | |||
|align="center"|% | There is a recurring problem where users bookmarks are re-ordered on their mobile device. Fix this. | ||
|align="center"|50% | |||
|align="center"|50% | |||
|align="center"|% | |align="center"|% | ||
|align="center"|% | |align="center"|% | ||
| Line 154: | Line 160: | ||
|align="center"|% | |align="center"|% | ||
|align="left"| | |align="left"| | ||
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] | ||
|Haven’t touched yet. | |||
|- | |- | ||
| align="left"|3.3 | | align="left"|3.3 Analyze and prevent erroneous bookmark deduping on Desktop. | ||
|align="center"|% | |||
|align="center"|% | There is a problem where users’ bookmarks are incorrectly deduped or reordered on their desktop device. This is a form of data loss. | ||
|align="center"|60% | |||
|align="center"|60% | |||
|align="center"|% | |align="center"|% | ||
|align="center"|% | |align="center"|% | ||
| Line 167: | Line 176: | ||
|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=1366888 Only consider unsynced local bookmarks for deduping] | |||
*[https://bugzilla.mozilla.org/show_bug.cgi?id=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. | |||
|- | |- | ||
| align="left"|3.4 | | align="left"|3.4 Batch upload API enabled for all users, with more than 99% of uploads fitting into a single atomic batch. | ||
|align="center"|% | |align="center"|50% | ||
|align="center"|% | |align="center"|50% | ||
|align="center"|% | |align="center"|% | ||
|align="center"|% | |align="center"|% | ||
| Line 179: | Line 191: | ||
|align="left"| | |align="left"| | ||
Feature description | Feature description | ||
* | *[https://bugzilla.mozilla.org/show_bug.cgi?id=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] | |||
|Discussions have started about what needs to be done. | |||
| | |||
|- | |- | ||
|} | |} | ||