Closed Bug 1398368 Opened 4 years ago Closed 4 years ago
Drop Pocket items without title, image
_url, or url
See last item on the attached screenshot.
Looks like it was probably a bad item from the Pocket API end - I saw it on Desktop too and the story has been removed from the feed (both the trending page and from the API when I pulled again to see what the problem might be). We can add a check to require title and image_src, and drop the item if it's malformed.
tracking-fennec: --- → ?
Summary: Pocket recommendation without a title or a preview image → Drop Pocket items without title, image_url, or url
This should be a quick fix to drop malformed items, and should include increasing the buffer fetched to +1 the number of Pocket items shown.
tracking-fennec: ? → +
Priority: -- → P2
4 years ago
Assignee: nobody → michael.l.comella
4 years ago
Iteration: --- → 1.30
Priority: P2 → P1
Comment on attachment 8907338 [details] Bug 1398368: Drop malformed Pocket Top Stories. https://reviewboard.mozilla.org/r/179012/#review184602
Attachment #8907338 - Flags: review?(liuche) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/3384e2855508 Drop malformed Pocket Top Stories. r=liuche
Devices: - Note 4 (Android 5.0.1) - English (US); - Galaxy S8 (Android 7.0) - Deutsch (Germany); No pocket stories without a title, image_url or url seem to have been displayed when monitoring pocket across several hours for 2 different locales. Marking this as verified.
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.