Page MenuHomePhabricator

Populate country column when creating c_t rows during offline import
Closed, ResolvedPublic1 Estimated Story Points

Description

When we create contribution_tracking rows for donations imported via spreadsheet, we should populate the country column when we know it.

Related Objects

StatusSubtypeAssignedTask
ResolvedEjegg

Event Timeline

Change 367806 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/crm@master] Add country to c_t rows created during imports

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

Pcoombe added a parent task: Restricted Task.Jul 26 2017, 9:31 AM
Ejegg triaged this task as Low priority.
Ejegg set the point value for this task to 1.

Change 367806 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Add country to c_t rows created during imports

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

Huh, recent Benevity imports still have null country in c_t. Taking another look.

Change 374597 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/crm@master] WIP fix c_t.country population, add test

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

Change 374597 merged by jenkins-bot:
[wikimedia/fundraising/crm@master] Fix c_t.country population, add test

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

Still waiting for non-check imports to test this one. For historical reasons, we're still not creating contribution_tracking records for checks.

Finally some rows to check this against! Seems to be working.