1946345 - Delete Pocket profile on enrolment of the MARS API

Status: VERIFIED FIXED

(Firefox for Android :: Stories, task, P3)

Description

[Gabriel Luong [:gl] (ΦωΦ)]

On enrolment to the Unified Ads API rollout, we should delete the profile associated with the pocket Id.

Comment 1

[Gabriel Luong [:gl] (ΦωΦ)]

[Tracking Requested - why for this release]:

We are looking to perform a Nimbus rollout of the new Unified Ads API in Fx136. So, we are looking to uplift any necessary changes that is needed into Fx136.

Comment 2

[Gabriel Luong [:gl] (ΦωΦ)]

Attachment: Bug 1946345 - Delete Pocket profile on enrolment of the MARS API

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

The bug is marked as tracked for firefox136 (beta) and tracked for firefox137 (nightly). However, the bug still has low priority.

:devota, could you please increase the priority for this tracked bug? If you disagree with the tracking decision, please talk with the release managers.

For more information, please visit BugBot documentation.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a4802844dbde
Delete Pocket profile on enrolment of the MARS API r=android-reviewers,devota

Comment 5

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/a4802844dbde

Comment 6

[Donal Meehan [:dmeehan]]

:gl please add a beta uplift request on this when you're ready

Comment 7

[Gabriel Luong [:gl] (ΦωΦ)]

Comment on attachment 9464516 [details]
Bug 1946345 - Delete Pocket profile on enrolment of the MARS API

Beta/Release Uplift Approval Request

• User impact if declined/Reason for urgency: We should delete the old Pocket profile associated with an user when they are migrated to use the new Unified Ads API.

We need this uplifted because we are looking to roll out the new Unified Ads API in Fx136

• 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: QA will be covered in https://mozilla-hub.atlassian.net/browse/QA-3191
• List of other uplifts needed: None
• Risk to taking this patch: Low
• Why is the change risky/not risky? (and alternatives if risky): Low risk because this has no user facing impact and would simply clear the old Pocket sponsored stories as they are migrated over to the new Unified Ads API.
• String changes made/needed:
• Is Android affected?: Yes

Comment 8

[Donal Meehan [:dmeehan]]

Comment on attachment 9464516 [details]
Bug 1946345 - Delete Pocket profile on enrolment of the MARS API

Approved for 136.0b6

Comment 9

[Pulsebot]

https://hg.mozilla.org/releases/mozilla-beta/rev/a33fc98e7a87

Comment 10

[Delia Pop (Android QA)]

Attachment: Screenshot 2025-02-24 at 15.35.07.png

Verified as fixed on debug build 1.0.2508 (Build #1) GV: 137.0a1-20250211213536 AS: 137.20250208050325 received from gl, with Samsung Galaxy S22 Ultra (Android 14).
After enabling the "Enable MARS API Integration" feature, a DELETE request was sent to https://spocs.getpocket.com/user . The Request object contains a “pocket_id”.

Comment 11

[Delia Pop (Android QA)]

Discussed with gl on slack, bug verification on Beta 136 can be skipped.