Closed
Bug 1161881
Opened 9 years ago
Closed 9 years ago
Enable Pocket by default (in supported locales)
Categories
(Firefox :: Pocket, defect)
Firefox
Pocket
Tracking
()
VERIFIED
FIXED
Firefox 40
Tracking | Status | |
---|---|---|
firefox38 | --- | unaffected |
firefox38.0.5 | --- | verified |
firefox39 | --- | verified |
firefox40 | --- | fixed |
People
(Reporter: Dolske, Assigned: Dolske)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Gavin
:
review+
Sylvestre
:
approval-mozilla-aurora+
Sylvestre
:
approval-mozilla-release+
|
Details | Diff | Splinter Review |
Enable Pocket by default in the locales that support it (currently just en-US, as controlled by the browser.pocket.useLocaleList and .enabledLocales prefs)
Attachment #8601866 -
Flags: review?(gavin.sharp)
Assignee | ||
Comment 1•9 years ago
|
||
So far it's green. https://treeherder.mozilla.org/#/jobs?repo=try&revision=79915ac4ab81
Updated•9 years ago
|
Attachment #8601866 -
Flags: review?(gavin.sharp) → review+
Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox38:
--- → unaffected
status-firefox38.0.5:
--- → affected
status-firefox39:
--- → affected
Flags: qe-verify?
Resolution: --- → FIXED
Target Milestone: --- → Firefox 40
Assignee | ||
Comment 3•9 years ago
|
||
Comment on attachment 8601866 [details] [diff] [review]
Patch v.1
Approval Request Comment
[Feature/regressing bug #]: n/a
[User impact if declined]: My management will hire hitmen and kill us all.
[Describe test coverage new/current, TreeHerder]: none! But has been manually tested and did a Try run to ensure it doesn't break anything else.
[Risks and why]: Hard to say, but is unlikely to affect users who don't use it, and we have no major known bugs.
[String/UUID change made/needed]: None.
Attachment #8601866 -
Flags: approval-mozilla-release?
Attachment #8601866 -
Flags: approval-mozilla-aurora?
Updated•9 years ago
|
Flags: qe-verify? → qe-verify+
QA Contact: andrei.vaida
Updated•9 years ago
|
Attachment #8601866 -
Flags: approval-mozilla-release?
Attachment #8601866 -
Flags: approval-mozilla-release+
Attachment #8601866 -
Flags: approval-mozilla-aurora?
Attachment #8601866 -
Flags: approval-mozilla-aurora+
Comment 5•9 years ago
|
||
Egh, I accidentally still pushed this to fx-team as well, as this seems to not have been merged around yet. :-(
Assignee: nobody → dolske
And a followup because I somehow pushed a merge conflict despite clearly resolving it locally before pushing...
https://hg.mozilla.org/releases/mozilla-aurora/rev/a3c2c1c0ebab
Assignee | ||
Comment 9•9 years ago
|
||
Temporary backout on Aurora, so that we can get other stuff uplifted.
https://hg.mozilla.org/releases/mozilla-aurora/rev/43ebc7c3dfcb
Bug 1163231 describes the test failures that need to be fixed. Technically this is caused by bug 1155521 actually adding the Pocket button to the toolbar when enabled, but it's easier to keep up with the uplifts by just temporarily disabling Pocket with a pref flip. Hopefully we can get tests working and Pocket re-enabled in a few days.
Assignee | ||
Comment 10•9 years ago
|
||
Comment 11•9 years ago
|
||
Verified Firefox 38.0.5 beta 1 en-US and locales (fr, ml, es-ES, ja, de) builds under Win 7 64-bit, Ubuntu 14.04 32-bit and Mac OS X 10.9.5.
For all builds, the following prefs have the same value:
browser.pocket.useLocaleList - true
browser.pocket.enabledLocales - en-US
browser.pocket.enabled - true
The only difference is that the Pocket icon is not available on locales builds, so it can't be used there while the en-US build has the icon on the toolbar.
Is there anything else I should verify for this? Thanks
Flags: needinfo?(dolske)
Assignee | ||
Comment 12•9 years ago
|
||
Nope, enabling the Pocket locales is bug 1163645, which unfortunately missed the beta1 uplift.
Flags: needinfo?(dolske)
Assignee | ||
Comment 13•9 years ago
|
||
(In reply to Justin Dolske [:Dolske] from comment #9)
> Temporary backout on Aurora, so that we can get other stuff uplifted.
>
> https://hg.mozilla.org/releases/mozilla-aurora/rev/43ebc7c3dfcb
>
> Bug 1163231 describes the test failures that need to be fixed. Technically
> this is caused by bug 1155521 actually adding the Pocket button to the
> toolbar when enabled, but it's easier to keep up with the uplifts by just
> temporarily disabling Pocket with a pref flip. Hopefully we can get tests
> working and Pocket re-enabled in a few days.
Now that today's merge-day uplift has completed, I've re-disabled Pocket on the new Aurora (FF40), and enabled it on the new Beta (FF39). Pocket remains enabled on Nightly (FF40) and for the impending 38.0.5 release.
https://hg.mozilla.org/releases/mozilla-aurora/rev/d9093db18dc6
https://hg.mozilla.org/releases/mozilla-beta/rev/f75252949669
status-firefox40:
--- → affected
Updated•9 years ago
|
Comment 14•9 years ago
|
||
Verification on 38.0.5 should suffice here.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Comment 15•9 years ago
|
||
Verified that de, es-ES, ja and ru locales have Pocket enabled by default in Firefox 39 beta 1 under all platforms.
No Pocket icon for other locales.
Updated•9 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•