Special FxA Snippet on New Tab for Firefox 67
Categories
(Firefox :: Messaging System, enhancement, P1)
Tracking
()
People
(Reporter: rrosario, Assigned: rrosario)
References
Details
(Keywords: github-merged)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
This is a new special snippet template that renders below the search bar on newtab page. See Bug 1539700 for more details.
We would like to get this uplifted to beta as one commit for convenience and because we don't want individual pieces to be backed out (it's all or nothing).
Assignee | ||
Updated•6 years ago
|
Comment 1•6 years ago
|
||
Ricky, do you have an ETA on the uplift? Also, it is my understandig that you want it for Trailhead, not for 67.0, can you confirm please? Thanks
Assignee | ||
Comment 2•6 years ago
|
||
(In reply to Pascal Chevrel:pascalc from comment #1)
Ricky, do you have an ETA on the uplift? Also, it is my understandig that you want it for Trailhead, not for 67.0, can you confirm please? Thanks
We just need to export the patch from the github repo, so we can have that done today. And we want this part to be uplifted to 67.0 (not Trailhead).
Assignee | ||
Updated•6 years ago
|
Comment 3•6 years ago
|
||
If it's not targeting our Trailhead release, maybe it shouldn't block bug 1545802.
Assignee | ||
Comment 4•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 5•6 years ago
|
||
Comment on attachment 9063180 [details]
Bug 1548042 - Special FxA Snippet on New Tab for Firefox 67
Beta/Release Uplift Approval Request
- User impact if declined: Users won't see the new FXA snippet under the search box on about:newtab.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: The snippets team (:giorgos, :jcollings) will need to deploy a snippet that targets beta 67. Then the snippet will appear in the new spot on newtab page. See Bug 1539700 for more details.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): The patch is pretty big, mostly because it contains changes to the prerendered file for every single locale. But the actual changes aren't that complex and have been tested and verified on Nightly. Also, the snippet can be turned off from the snippets server if any issue were to come up.
- String changes made/needed: N/A
Assignee | ||
Updated•6 years ago
|
Updated•6 years ago
|
Updated•6 years ago
|
Comment 6•6 years ago
|
||
Joe and Reese - do you approve this uplift into 67.0 in support of Trailhead?
Comment 8•6 years ago
|
||
r+
Comment 9•6 years ago
|
||
Comment on attachment 9063180 [details]
Bug 1548042 - Special FxA Snippet on New Tab for Firefox 67
Evaluated as low risk, uplift approved for beta 19, thanks.
Comment 10•6 years ago
|
||
bugherder uplift |
Updated•6 years ago
|
Updated•6 years ago
|
Comment 11•6 years ago
|
||
Removing the qe-verify flag as we signed off (GREEN) on this feature via email on 5/15/2019. Link to test run.
Updated•5 years ago
|
Description
•