[Basket] Error returned when trying to unsubscribe a user.

VERIFIED FIXED

Status

VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: tasos, Assigned: tasos)

Tracking

2016-10.2

Details

(Assignee)

Description

2 years ago
The current implementation of the unsubscribe task returns an error from time to time (User not found). We should refactor this task to first check the subscription status of the user before trying to unsubscribe this user.

Comment 1

2 years ago
Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/f5d5c25d864f9d8a1ca521139e628f1dcb5936c2
[fix bug 1291239] Refactor basket unsubscribe task.

https://github.com/mozilla/mozillians/commit/78a4ad429a3a05b8a553b21f7dfec5e841dd930f
Merge pull request #1537 from akatsoulas/refactor-unsubscribe-task

[fix bug 1291239] Refactor basket unsubscribe task.

Updated

2 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED

Comment 2

2 years ago
Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/48f47caf58aa5d7a479ae91389657716f19c7738
[bug 1291239] Unsubscribe users only if there are newsletters.

* Pass correct parameter to optout flag.

https://github.com/mozilla/mozillians/commit/8dd506bfa139646b66dfb27188c90c55023f23d4
Merge pull request #1539 from akatsoulas/unsubscribe-task-traceback

[bug 1291239] Unsubscribe users only if there are newsletters.
Version: other → next
Verified on stage:

* Unsubscribe task from admin works fine
* Unvouch profile + profile save trigger unsubscribes user from vouched newsletter
Status: RESOLVED → VERIFIED
Version: next → 2016-10.2
You need to log in before you can comment on or make changes to this bug.