Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

Bug 967795: Make updating the mobile subscribers DE a separate task. #119

Merged

Conversation

pmclanahan
Copy link

If the data-extension update fails the task will retry, and send another
SMS to the user. Moving this part to its own task means that both calls
will retry independently.

I believe the current tests cover this well enough in that they ensure
that the ET stuff is called if optin is true.

If the data-extension update fails the task will retry, and send another
SMS to the user. Moving this part to its own task means that both calls
will retry independently.
@pmclanahan
Copy link
Author

@jgmize one more quick one.

jgmize added a commit that referenced this pull request Oct 22, 2014
…e-task-967795

Bug 967795: Make updating the mobile subscribers DE a separate task.
@jgmize jgmize merged commit 8c61090 into mozilla:master Oct 22, 2014
@pmclanahan pmclanahan deleted the make-sms-optin-update-separate-task-967795 branch October 23, 2014 12:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants