Bug 1411657: Disable Pocket by default. r=liuche draft
authorMichael Comella <michael.l.comella@gmail.com>
Wed, 25 Oct 2017 12:05:17 -0700
changeset 686339 9bb693e83ec98a8c1bae6476b2c800cf89968bd3
parent 686338 8a451817fdf318f8ff2f8536c386e320d88fe293
child 686352 8f78370f7b141024d5e7d5a6c668572e33f525d1
child 686382 f61005db19cf942828ba3bc4aecac85a0def5f8f
push id86168
push usermichael.l.comella@gmail.com
push dateWed, 25 Oct 2017 19:05:50 +0000
reviewersliuche
bugs1411657
milestone58.0a1
Bug 1411657: Disable Pocket by default. r=liuche From product (bug 1411657 comment 0): Activity Stream may be pushed to a later release on desktop. With that, we want the Pocket stories feature to be put behind a feature flag on Firefox Android,so we can launch the feature across all platforms simultaneously. MozReview-Commit-ID: 3KmZir3tGPc
mobile/android/app/src/main/res/values/bool.xml
--- a/mobile/android/app/src/main/res/values/bool.xml
+++ b/mobile/android/app/src/main/res/values/bool.xml
@@ -10,12 +10,15 @@
          with the configuration retrieved by HardwareUtils (e.g. some custom ROMs allow the user to
          choose a phone or tablet version of the UI even though the hardware stays the same). This
          can cause crashes when we branch on the value returned by HardwareUtils.
 
          In order to work around this, we define the resource size in resources with the expectation that
          we branch on that value, rather than HardwareUtils, so our code is consistent with the used resources.
          See bug 1277379 for a initiative to move all of the HardwareUtils code over. -->
     <bool name="is_large_resource">false</bool>
-    <bool name="pref_activitystream_pocket_enabled_default">true</bool>
+
+    <!-- Pocket recommendations feature flag: change this to true when we want to ship (bug 1411657). -->
+    <bool name="pref_activitystream_pocket_enabled_default">false</bool>
+
     <bool name="pref_activitystream_visited_enabled_default">true</bool>
     <bool name="pref_activitystream_recentbookmarks_enabled_default">true</bool>
 </resources>