Page MenuHomePhabricator

Deal with PayPal messages with old_subscr_id
Closed, ResolvedPublic1 Estimated Story Points

Description

Pseudo-Tag: failmail

We're getting some subscr_cancel messages with two different subscr_ids. In the subscr_id field is a new-style I-XXX ID, and in the new field old_subscr_id, is an old-style S-XXX ID. We get failmail because we know the subscription under the old_subscr_id, but we're not passing that along to Civi.

To investigate: do we ever get a message with old_subscr_id where we have the new subscr_id stored in Civi?

If all messages with old_subscr_id correspond to old-style subscription IDs in Civi, just pass that one along.

Event Timeline

OK, I just looked up all the messages from this month with had old_subscr_id, and we're not storing a reference to the new subscr_id for any of them. Should be fine to use the old_subscr_id whenever that's present.

Change 468177 had a related patch set uploaded (by Ejegg; owner: Ejegg):
[wikimedia/fundraising/SmashPig@master] Deal with old_subscr_id, fix date for cancel

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

Ejegg triaged this task as Medium priority.
Ejegg set the point value for this task to 1.
Ejegg moved this task from Triage to Current Sprint on the Fundraising-Backlog board.

Change 468177 merged by jenkins-bot:
[wikimedia/fundraising/SmashPig@master] Deal with old_subscr_id, fix date for cancel

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