1155515 - Disable Reading List and RL Sync on all channels

Status: VERIFIED FIXED

(Firefox Graveyard :: Reading List, defect)

Description

[Justin Dolske [:Dolske]]

For the near term (and especially 38.1), we will be focusing on Pocket integration instead of the Reading List/Sync work we've been doing. Until we understand how "Reading List" and Pocket may coexist, we will disable Reading List and the new Reading List Sync service.

[Note that Android will, for now, continue to support the Reading List that they've shipped with for a long time. Although the newly-written Sync code for it will be disabled, since Desktop and iOS won't be using it.]

See also bug 1155191, where we disabled this on 38.0.

Reader View is largely unaffected by this change.

Users on pre-release channels (Nightly/DevEdition/Beta) who have already added things to their Reading List will no longer have access to them. There will be no migration to Pocket --  sorry. If affected users need to get back to their Reading List items, I'd suggest they download an older build, go through their list, and convert items to bookmarks (which will sync across devices with Firefox Sync). Or wait until our Pocket integration is functional, and s/bookmarks/Pocket/.

Comment 1

[Sylvestre Ledru [:Sylvestre]]

Tracked for 38 for obvious reasons.

Comment 2

[Liz Henry (:lizzard) (relman/hg->git project)]

Tracking for 39+ as well.

Comment 3

[Justin Dolske [:Dolske]]

Attachment: Patch v.1

Unlike bug 1155191, this just softly flips the default prefs. This will land on Nightly and Aurora (40+39).

I disabled it harder in the other bug for 38.* because it seems like there's significant short-term risk of confusion around this feature, and I wanted to avoid having Release users think this is a new-and-almost-ready feature based on something they read in a random blog post. That should be less of a problem when 39 makes it to release, and by then we'll hopefully have a better understanding of the future plans, if any, for Reading List.

Comment 4

[Justin Dolske [:Dolske]]

(In reply to Sylvestre Ledru [:sylvestre] from comment #1)
> Tracked for 38 for obvious reasons.

Actually bug 1155191 is disabling this for 38.0 and 38.1 (same thing, since we haven't branched yet). So I don't think we need to track _this_ bug for 38?

Comment 5

[Justin Dolske [:Dolske]]

(In reply to Justin Dolske [:Dolske] from comment #3)
> Created attachment 8594427 [details] [diff] [review]
> Patch v.1

https://treeherder.mozilla.org/#/jobs?repo=try&revision=7ecaa8cd2d96

Comment 6

[Sylvestre Ledru [:Sylvestre]]

ok, no worries! Untracking it then

Comment 7

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/b11d72334163

Comment 8

[Justin Dolske [:Dolske]]

(Added a pref flip to browser/components/readinglist/test/xpcshell/test_scheduler.js, glad I did that Try run!)

Comment 9

[Justin Dolske [:Dolske]]

Comment on attachment 8594427 [details] [diff] [review]
Patch v.1

[Triage Comment]

(As noted above, bug 1155191 landed a stricter version of this on beta, so no beta approval needed here.)

Comment 10

[Justin Dolske [:Dolske]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/674987e8e0e1

Comment 11

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/b11d72334163

Comment 12

[Liz Henry (:lizzard) (relman/hg->git project)]

Untracking; looks like this is fixed and has had no issues for the last week.

Comment 13

[Andrei Vaida [:avaida]]

The patch pushed for this bug was also verified through the testing conducted on Bug 1155191. Closing.