Closed
Bug 1802265
Opened 2 years ago
Closed 2 years ago
Pocket newtab sponsored topsites should use title instead as sponsor
Categories
(Firefox :: New Tab Page, enhancement)
Firefox
New Tab Page
Tracking
()
VERIFIED
FIXED
109 Branch
People
(Reporter: thecount, Assigned: thecount)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
diannaS
:
approval-mozilla-beta+
|
Details | Review |
Right now sponsored topsites from Pocket are using the sponsor label as the topsite title.
We should update this to be title if available, and fall back to sponsor label if there is no title.
Assignee | ||
Comment 1•2 years ago
|
||
[Tracking Requested - why for this release]: Should be a 1 line fix, and would impact an upcoming experiment.
status-firefox108:
--- → affected
tracking-firefox108:
--- → ?
Assignee | ||
Updated•2 years ago
|
Assignee: nobody → sdowne
Assignee | ||
Comment 2•2 years ago
|
||
Updated•2 years ago
|
Pushed by sdowne@getpocket.com:
https://hg.mozilla.org/integration/autoland/rev/898e7b642333
Pocket newtab topsite spoc use title instead of sponsor. r=nanj
Comment 4•2 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 2 years ago
status-firefox109:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch
Assignee | ||
Comment 5•2 years ago
|
||
Comment on attachment 9305057 [details]
Bug 1802265 - Pocket newtab topsite spoc use title instead of sponsor.
Beta/Release Uplift Approval Request
- User impact if declined: Impact to an experiment
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: Testing should be regression testing for sponsored topsite title is still visible.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): It is a simple 2 line change.
- String changes made/needed: None
- Is Android affected?: No
Attachment #9305057 -
Flags: approval-mozilla-beta?
Assignee | ||
Updated•2 years ago
|
Flags: qe-verify+
Comment 6•2 years ago
|
||
Comment on attachment 9305057 [details]
Bug 1802265 - Pocket newtab topsite spoc use title instead of sponsor.
Approved for 108.0b7
Attachment #9305057 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 7•2 years ago
|
||
bugherder uplift |
Updated•2 years ago
|
QA Whiteboard: [qa-triaged]
Comment 8•2 years ago
|
||
I have verified this issue on the latest Nightly 109.0a1 (Build ID: 20221129084032) and the latest Beta 108.0b7 (Build ID: 20221127190117) on Windows 10 x64, macOS 12.4 and Linux Mint 20.
- The Sponsored tiles via Pocket have the title correctly displayed.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•