Closed Bug 1416971 Opened 3 years ago Closed 3 years ago

Use Correct Pocket Read More Endpoint


(Firefox for Android Graveyard :: Activity Stream, defect, P1)

Firefox 57


(firefox58 verified, firefox59 fixed)

Firefox 59
Tracking Status
firefox58 --- verified
firefox59 --- fixed


(Reporter: sdaswani, Assigned: mcomella)



(1 file)

While we are making this change, can we update it to:
Priority: -- → P1
Matt, currently the URL we're using is:

Unlike this URL, the new URL doesn't provide a source (i.e. coming from FF Android) and we don't intentionally send any other indication of where these links could be coming from (i.e. no referrer): is that okay?

- Server-side, we could extract FF for Android from the user agent, but we wouldn't know if it's specifically coming from the MORE button
- Client-side, we have telemetry for how many times the MORE button is clicked
Flags: needinfo?(matt)
Oh, nevermind - the src in comment 0 and comment 1 are different. comment 1 specifies ff_android as the source.
Flags: needinfo?(matt)
Mike - correct - is what we want to use and that includes the `src`
Pushed by
Update Pocket MORE link. r=liuche
Comment on attachment 8932186 [details]
Bug 1416971: Update Pocket MORE link.

Approval Request Comment
[Feature/Bug causing the regression]: Improper initial implementation (via miscommunication?)
[User impact if declined]: Users have the same experience - I'm not sure why Pocket wants the URL change
[Is this code covered by automated tests?]: None
[Has the fix been verified in Nightly?]: No
[Needs manual test from QE? If yes, steps to reproduce]: Likely unnecessary but STR: 1) Open top sites, 2) Click MORE, 3) Verify URL and that trending pocket stories appear. Ideally verify this works in at least en-US and de-DE with the appropriate language, but here's all locales we support [1]

[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: No
[Why is the change risky/not risky?]: Simple, one line change: we replace the URL value in a constant
[String changes made/needed]: None

Attachment #8932186 - Flags: approval-mozilla-beta?
Closed: 3 years ago
Resolution: --- → FIXED
Comment on attachment 8932186 [details]
Bug 1416971: Update Pocket MORE link.

Update the pocket more link. Beta58+.
Attachment #8932186 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Target Milestone: Firefox 57 → Firefox 59
Verified as fixed on Beta 58.0b9 on Samsung Galaxy Note 4 (Android 5.0.1).
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.