Closed Bug 1286559 Opened 8 years ago Closed 8 years ago

Basket tasks won't subscribe user to newsletter when token already associated

Categories

(Participation Infrastructure :: Phonebook, defect)

2016-7.4
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: nemo-yiannis, Assigned: nemo-yiannis)

References

Details

Steps to reproduce:

* Make sure user has token and is already subscribed to one newsletter
* Use admin actions to subscribe to another newsletter
* Celery task hits 3 retries and returns without any change
Blocks: 1282444
Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/c42364825167de94d740802ec3b636643ad6469a
[Fix bug 1286559] Fix update basket subscriptions task.

* Check first for email changes
* Subscribe to required newsletters
* Update token if no token stored for mozillian yet

https://github.com/mozilla/mozillians/commit/f52a777dc17219067fe8dc2ed2e93742f54c9d9c
Merge pull request #1482 from johngian/fix-basket-subscription

[Fix bug 1286559] Fix update basket subscriptions task.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Version: other → next
Assignee: nobody → jgiannelos
Verified on stage:

Subscriptions

* Unsubscribe user from both newsletters (nda, vouched)
* Hit admin action for vouched user
* Lookup user using API
* User is subscribed to the correct newsletter
* Hit admin action for vouched user
* Lookup user using API
* User is subscribed to the correct newsletter

Email change

* Subscribe a user to some newsletters
* Change the user email
* Trigger save
* Lookup user in basket using the old email
 * newsletters are empty
* Lookup user in basket using the new email
 * newsletters are the same with those in step 1
Status: RESOLVED → VERIFIED
Version: next → 2016-7.4
You need to log in before you can comment on or make changes to this bug.