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

Fix bug 1185999: Look up users in Confirmed DE properly. #139

Merged
merged 1 commit into from
Jul 22, 2015

Conversation

pmclanahan
Copy link

From the bug:

The utils.get_user_data function was attempting to look for users that were not in Master_Subscribers, but were in Double_Opt_In in the Confirmation data extension using whatever info was passed to the function. When the lookup was via email, the email address was used, but the Confirmation extension does not have an EMAIL_ADDRESS_ column. This caused errors for all of these users and basket was returning a 400 error. The expected behavior is for basket to return a 200 response indicating that the user was found, but unconfirmed.

@pmclanahan
Copy link
Author

@jgmize r?

@jgmize
Copy link

jgmize commented Jul 22, 2015

r+, good catch.

jgmize added a commit that referenced this pull request Jul 22, 2015
Fix bug 1185999: Look up users in Confirmed DE properly.
@jgmize jgmize merged commit 034a156 into mozilla:master Jul 22, 2015
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.

None yet

3 participants