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)

enhancement

Tracking

()

VERIFIED FIXED
109 Branch
Tracking Status
firefox108 + verified
firefox109 --- verified

People

(Reporter: thecount, Assigned: thecount)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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.

[Tracking Requested - why for this release]: Should be a 1 line fix, and would impact an upcoming experiment.

Assignee: nobody → sdowne
Blocks: 1794019
Pushed by sdowne@getpocket.com: https://hg.mozilla.org/integration/autoland/rev/898e7b642333 Pocket newtab topsite spoc use title instead of sponsor. r=nanj
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 109 Branch

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?
Flags: qe-verify+

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+
QA Whiteboard: [qa-triaged]

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.

Attachment

General

Created:
Updated:
Size: