Page MenuHomePhabricator

Reconstruct opt_in data from buggy paypal form
Closed, ResolvedPublic1 Estimated Story Points

Description

Need to use the web logs. hopefully everyone with opt_in=1 will also have an email address in the querystring.

Event Timeline

You could look at drupal.contribution_tracking.utm_source instead, it should have either "optIn0" or "optIn1" in there.

Change 461508 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/crm@master] Fix opt_in data affected by form bug

https://gerrit.wikimedia.org/r/461508

Ejegg triaged this task as High priority.
Ejegg set the point value for this task to 1.

Change 461685 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/crm@master] Follow-up: fix opt-in data for 148 non-PayPal donors

https://gerrit.wikimedia.org/r/461685

Change 461508 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Fix PayPal opt_in data affected by form bug

https://gerrit.wikimedia.org/r/461508

OK, this is fixed for the 1,341 PayPal donors affected.

There are 148 credit card donors who were also potentially affected by this bug - they have optin0 in the utm_source, indicating that they clicked 'no' in the banner, but opt_in = 1 in Civi. The bug that @Pcoombe pointed out (payments form showing 'yes' checked for people with opt_in=0 in querystring) affected card donors as well, it's just that they had the chance to click 'no' a second time before submitting their details. It's possible some of them changed their mind and decided 'yes' was fine on the payments form, but I imagine we'd want to set all of those donors to opt_in=0 also. I've got a patch ready to do that.

Ah, I hadn't realised the form bug affected credit card too. I assume that was fixed at the same time as PayPal?

Correcting those donors to opt_in=0 sounds like the safest action to me.

Change 461685 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Follow-up: fix opt-in data for 148 non-PayPal donors

https://gerrit.wikimedia.org/r/461685

Change 470033 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/crm@master] One more opt_in fix, this time for folks with NULL

https://gerrit.wikimedia.org/r/470033

Change 470033 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] One more opt_in fix, this time for folks with NULL

https://gerrit.wikimedia.org/r/470033