[00:03:06] (PS2) Eileen: Improve import to allow downloading of ALL non imported rows and add tests [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/320560 (https://phabricator.wikimedia.org/T88460) [00:03:08] (PS1) Eileen: Comment to explain salt-wierdness [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321571 (https://phabricator.wikimedia.org/T15038) [00:03:32] cwd|afk: morebots was the subservient bot. stashbot took over and is much less obsequious [00:27:38] (PS1) Eileen: Restrict downloading of csvs to the creator of the file. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321575 (https://phabricator.wikimedia.org/T88460) [00:31:09] (CR) jenkins-bot: [V: -1] Restrict downloading of csvs to the creator of the file. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321575 (https://phabricator.wikimedia.org/T88460) (owner: Eileen) [00:57:53] (PS1) Eileen: Allow campaign option value labels to differ from the unique value. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321578 (https://phabricator.wikimedia.org/T150697) [01:01:30] (CR) jenkins-bot: [V: -1] Allow campaign option value labels to differ from the unique value. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321578 (https://phabricator.wikimedia.org/T150697) (owner: Eileen) [02:45:31] (CR) Isarra: "Wrong bug." [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321571 (https://phabricator.wikimedia.org/T15038) (owner: Eileen) [05:31:51] (PS3) Ejegg: Downgrade missing details to warning for RecordCaptureJob [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/319373 [05:31:53] (PS3) Ejegg: Capture Adyen payments missing pending messages [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/319489 (https://phabricator.wikimedia.org/T149861) [05:31:55] (PS1) Ejegg: Antifraud messages don't have gateway_txn_id [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/321603 [08:38:47] (PS1) Eileen: Fix Engage import handling of Postal. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321612 (https://phabricator.wikimedia.org/T148056) [08:42:19] (PS2) Eileen: Fix Engage import handling of Postal. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321612 (https://phabricator.wikimedia.org/T148056) [08:42:21] (PS2) Eileen: Allow campaign option value labels to differ from the unique value. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321578 (https://phabricator.wikimedia.org/T150697) [08:42:23] (PS2) Eileen: Restrict downloading of csvs to the creator of the file. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321575 (https://phabricator.wikimedia.org/T88460) [08:45:47] (CR) jenkins-bot: [V: -1] Fix Engage import handling of Postal. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321612 (https://phabricator.wikimedia.org/T148056) (owner: Eileen) [08:46:16] (CR) jenkins-bot: [V: -1] Allow campaign option value labels to differ from the unique value. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321578 (https://phabricator.wikimedia.org/T150697) (owner: Eileen) [08:51:35] (CR) jenkins-bot: [V: -1] Restrict downloading of csvs to the creator of the file. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321575 (https://phabricator.wikimedia.org/T88460) (owner: Eileen) [09:14:05] (PS2) Eileen: Comment to explain salt-wierdness [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321571 (https://phabricator.wikimedia.org/T15038) [09:14:45] (PS3) Eileen: Allow campaign option value labels to differ from the unique value. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321578 (https://phabricator.wikimedia.org/T150697) [09:16:00] (PS3) Eileen: Fix Engage import handling of Postal. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321612 (https://phabricator.wikimedia.org/T148056) [09:16:02] (PS1) Eileen: Improve import to allow downloading of ALL non imported rows and add tests [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321616 (https://phabricator.wikimedia.org/T88460) [09:17:05] (Abandoned) Eileen: Improve import to allow downloading of ALL non imported rows and add tests [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/320560 (https://phabricator.wikimedia.org/T88460) (owner: Eileen) [09:17:20] (Abandoned) Eileen: Restrict downloading of csvs to the creator of the file. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321575 (https://phabricator.wikimedia.org/T88460) (owner: Eileen) [16:40:27] hi cwd [16:40:43] cwd: ejegg: hi hi :) [16:41:12] ejegg: quick question: do you have credentials for the github repo where Ellery put his code? [16:41:36] hi! [16:42:46] ejegg: you still back in MA? [16:45:40] AndyRussG: looking [16:45:49] cwd got back to NOLA last night [16:46:06] sorry I didn't get back online before my flight - the security line was longer than usual [16:46:19] looks like your fix worked though! [16:46:48] no prob! and yep :) [16:47:07] can still see plenty of those failures in the listener log [16:47:23] hopefully the 403 resend thing works [16:47:42] otherwise i guess we're counting on the audit files for those? [16:50:12] cwd yeah [16:53:42] ejegg: do you think we should take their recommendation and set up our own retry mechanism for when it returns an html page? [16:54:58] cwd I guess we could - maybe add a 'verified' flag on job messages [16:55:06] that is true in the normal case [16:55:32] but let the job try to do the verification if it's busted when the listener gets it. [16:55:58] and use the retryableException to requeue it for later verification if it's busted again when the job tries it [16:56:44] I guess the old IPN code actually was retrying the verification up to 7 times consecutively during the initial IPN reception [16:57:52] yeah i did notice that, although i haven't seen the behavior the comment in that code suggested [16:58:25] which is to say trying these 7 times in a row wouldn't help since the outage has been like...5 days by now [16:59:51] although, i guess we haven't retried any. maybe they would work the 2nd time. [17:12:20] i guess it's like a round robin on the back end, and most of the servers usually work [17:13:35] yeah must be [17:13:55] what an awesome way to handle application errors [17:14:20] i wonder if we can say to donors, if you get an error please go to the bank and mail a money order [17:18:57] https://imgur.com/cNY0Fkz [17:24:49] hehe [17:56:37] fr-tech hi! does anyone understand these changes? https://phabricator.wikimedia.org/rECNO6cb7847999db95000ac3e06d2d29120cd3e6224d https://phabricator.wikimedia.org/rECNOa20e673143ff9c0a7abf0815e44a6f0d2f19e9c2 [17:56:50] (the second just corrects a typo in the first) [17:57:10] It's causing a merge conflict when I try to merge master into wmf_deploy to put that stuff on train [18:00:11] Also, strangely, I can't find those in Gerrit [18:00:38] fr-tech: My advice to you, my violent friend, is to seek out gold and sit on it. [18:00:38] -- The Dragon to Grendel, in John Gardner's "Grendel" [18:00:38] -- discuss. [18:01:03] Maybe it was a thing that had to be done non-Gerrit wise because branches [18:01:39] afaik releng is the one dept that actually uses differential for everything [18:02:13] hmmm that would seem to follow... [18:02:36] as far as the change, it does seem like it should have a branch name to track right? [18:04:48] I guess, or maybe it activates some automagic feature [18:05:56] AndyRussG: it's just a change in how git-review works [18:06:24] so it'll submit against whichever upstream branch your current checkout is tracking [18:06:48] rather than specifically the branch named in the .gitreview file [18:06:54] oh, that's neat. so you could do feature branches? [18:07:06] cwd yeah, and you wouldn't have to keep tweaking that file [18:07:14] sweet [18:07:41] ejegg: so should I merge the change into wmf_deploy? currently that line in wmf_deploy is defaultbranch=wmf_deploy [18:07:42] so it's meant to prevent merge conflicts in the future, though of course it'll cause one when it's initially applied to existing branches [18:07:57] K so I guess all the branches should say the same thing [18:08:04] AndyRussG: yeah, exactly [18:08:22] just keep an eye on the next git-review to make sure it's ending up in the right place [18:08:41] Ah K makes sense ;) [18:08:43] XenoRyet had trouble with the DI repo after that change was merged there [18:08:51] huh [18:08:53] K [18:09:03] I know there's funky stuff in CN config that we actually need to deal with [18:09:06] XenoRyet: did you figure out why git-review wasn't targeting the upstream tracking branch? [18:09:50] No, never did figure out any more on that. I'm trying to remember what we did that finally made git review do the right thing with it. [18:10:11] I think I had to just manually specify the remote branch in the command. [18:10:24] dang [18:10:26] (PS1) AndyRussG: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - https://gerrit.wikimedia.org/r/321699 [18:10:49] MBeat: think this is cool to resolve? https://phabricator.wikimedia.org/T149902 [18:11:11] yep, that looks good cwd [18:11:19] thanks! [18:12:51] Hmmm this bit looks OK [18:12:53] https://gerrit.wikimedia.org/r/#/c/321699/ [18:13:10] But this looks a bit weird: https://gerrit.wikimedia.org/r/#/c/321699/1/.gitreview [18:13:52] AndyRussG: looks like a normal conflict resolution [18:13:57] as they appear in gerrit [18:14:04] fr-tech: I'm in the -talk hangout [18:14:41] be there in a minute! [18:14:43] Let me find my headphones and I'll jump in as well. [18:15:22] There's also some weird difference in whitespace between the result in wmf_deploy and master [18:15:42] ejegg: I didn't expect to see the conflicty file as the "previous" version in the Gerrit diff [18:16:02] AndyRussG: yeah, gerrit's weird like that on conflicts [18:16:49] It must have something to do with how it doesn't show all of the things that will change during a merge [18:17:01] just shows the additional changes on top of the merged commits [18:17:20] huh [18:17:27] OK cool, I guess I just never noticed it [18:18:01] The whitespace changes bother me, but I guess not too much, so I'll +2 so it gets train-ed today [18:20:08] thx! [18:20:57] Which hangout are we tech talking in? Hangouts is telling me the regular one is empty. [18:23:28] XenoRyet: https://hangouts.google.com/hangouts/_/wikimedia.org/fr-tech-talk [18:23:37] I'm just switching browsers due to audio issues [18:24:54] 320710 [18:25:03] and 319489 [18:26:32] (CR) AndyRussG: [C: 2] Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - https://gerrit.wikimedia.org/r/321699 (owner: AndyRussG) [18:28:17] (Merged) jenkins-bot: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - https://gerrit.wikimedia.org/r/321699 (owner: AndyRussG) [19:08:51] Fundraising Sprint Value Subtracting, Fundraising-Backlog, FR-Paypal, Patch-For-Review: PayPal is creating random one-time charges against PayPal subscriptions - https://phabricator.wikimedia.org/T143903#2796730 (DStrine) Open>Resolved [19:09:58] Fundraising Sprint Value Subtracting, Fundraising-Backlog, FR-Paypal, Patch-For-Review: PayPal is creating random one-time charges against PayPal subscriptions - https://phabricator.wikimedia.org/T143903#2582696 (DStrine) No SP value on this. I'm resolving in the sprint to note when we think it w... [19:13:52] Fundraising-Backlog: Spike: investigate odd safari behaviors - https://phabricator.wikimedia.org/T150792#2796756 (DStrine) [19:14:36] (CR) Ejegg: [C: 2] Allow campaign option value labels to differ from the unique value. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321578 (https://phabricator.wikimedia.org/T150697) (owner: Eileen) [19:18:10] ejegg: thanks! [19:19:31] (Merged) jenkins-bot: Allow campaign option value labels to differ from the unique value. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321578 (https://phabricator.wikimedia.org/T150697) (owner: Eileen) [19:19:57] (PS4) Cdentinger: Downgrade missing details to warning for RecordCaptureJob [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/319373 (owner: Ejegg) [19:22:27] (CR) Cdentinger: [C: 2] Downgrade missing details to warning for RecordCaptureJob [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/319373 (owner: Ejegg) [19:22:44] (PS2) Cdentinger: Antifraud messages don't have gateway_txn_id [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/321603 (owner: Ejegg) [19:24:31] (Merged) jenkins-bot: Downgrade missing details to warning for RecordCaptureJob [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/319373 (owner: Ejegg) [19:35:01] (CR) Cdentinger: [C: 2] Antifraud messages don't have gateway_txn_id [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/321603 (owner: Ejegg) [19:36:17] (Merged) jenkins-bot: Antifraud messages don't have gateway_txn_id [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/321603 (owner: Ejegg) [19:41:10] i guess no one is showing up to the cross org meetup [19:44:35] (PS4) Cdentinger: Capture Adyen payments missing pending messages [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/319489 (https://phabricator.wikimedia.org/T149861) (owner: Ejegg) [20:25:35] Hayyyyyy!!!! Can someone tell me if wikimediafoundation.org/tracker/bannerImpression.php is still called/used from anything? My grep-fu says no, but I wanna be sure :D [20:25:50] (AIUI, there's like a Special page we use for banner impressions now, or something else :)) [20:32:17] ostriches: I think we're using EventLogging... lemme check [20:34:53] ostriches: Ah, we just do a sendBeacon call (falling back to creating an img element) [20:35:01] to a url set in wgCentralBannerRecorder [20:35:34] which is $wgCentralBannerRecorder = "{$wgServer}/beacon/impression"; [20:35:49] so yeah, that tracker/bannerImpression.php is obsolete [20:40:58] ejegg: Awesome thanks! [20:41:08] (I'm on a mission cleaning up crappy legacy stuff in our docroot) [20:47:19] (CR) Ejegg: "I did a semi- smoke test of this, and got good results making the findMerchantReference call on AmazonApi. Will try going through the full" [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/320710 (https://phabricator.wikimedia.org/T147973) (owner: Ejegg) [20:59:01] (CR) Cdentinger: "Still curious about my previous comment, wondering if we can stop this situation before it starts. Seems like there should be a way to av" [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/319489 (https://phabricator.wikimedia.org/T149861) (owner: Ejegg) [21:04:05] (CR) Ejegg: "We do increment the order ID every time we send it. It's not clear how the donors are able to make repeated attempts against the same orde" [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/319489 (https://phabricator.wikimedia.org/T149861) (owner: Ejegg) [21:04:47] ejegg: Similar one.... wmf.org/funddrive/(fundgraph|index|report|reportConfig).php ? They all just redirect to fundraising.wikimedia.org, can't find any incoming references. [21:05:05] They've been just redirecting like that since 2012 or earlier. [21:06:23] errr, those are things I've never heard of [21:06:35] maybe ask K4-713 for some historical context? [21:06:43] I think they predate her too :) [21:06:55] Searching looks like it's from '05 [21:07:01] oh wow [21:07:01] Wow, that's freaking ancient. [21:07:04] 11 years. [21:07:05] cruftastic [21:08:17] inurl:wikimediafoundation.org/funddrive comes up blank [21:10:15] https://wikimediafoundation.org/logos/quote.png man I'm finding some old goodies [21:11:49] * ejegg shudders at text in graphics [21:15:59] ejegg: ah ha, confounding situation with those dupe IDs. have you asked adyen about it? [21:16:03] Fundraising Sprint Value Subtracting, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, FR-2016-17-Q2-Bugs: Engage import failing to import certain significant fields - https://phabricator.wikimedia.org/T146295#2656564 (Eileenmcnaughton) @LeanneS can you provide any additional information here... [21:18:00] cwd no, but I guess I should [21:20:36] might be good to at least get the straight dope to be sure we're fixing for the right thing [21:21:18] I could also do the experiment with two frames [21:21:42] ejegg: is order ID in this case generated by us or adyen? [21:21:50] by us [21:21:59] do they have a separate unique ID? [21:22:31] yeah, but our payments site doesn't know what it is before we open the iframe [21:22:47] gotcha [21:22:51] so we have to associate the donor details with our ID [21:23:04] doesn't it seem weird that they would override the previous session with a new ID, considering it's provided by us? [21:23:30] testing it out now [21:24:27] Fundraising Sprint Value Subtracting, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, FR-2016-17-Q2-Bugs: Engage import failing to import certain significant fields - https://phabricator.wikimedia.org/T146295#2797210 (Eileenmcnaughton) There was a commit on Aug 2 that caused Campaigns to be c... [21:26:33] Fundraising-Backlog: Edit Civi Reminder email messaging - https://phabricator.wikimedia.org/T150804#2797212 (CaitVirtue) [21:26:48] Fundraising-Backlog: Edit Civi Reminder email messaging - https://phabricator.wikimedia.org/T150804#2797224 (CaitVirtue) p:Triage>Normal [21:28:39] hah, oh wow - I tried to submit in the first iframe after opening a second and got this: "It appears you are performing multiple payments simultaneously. Please complete the other payment first. If you are sure that there is no other payment session currently open, you may need to restart the browser before attempting another payment." [21:30:30] haha, thorough! [21:30:46] they do seem like the most tech savvy of the psps [21:31:11] So.... I'm still pretty puzzled. We never see duplicate order IDs in the logs of what we tell browsers to post to their iframe [21:31:57] you are pretty sure dupe order IDs are the problem? [21:32:19] but we still sometimes get an IPN message for a failed attempt follwed by a message for a successful auth, with the same order ID [21:32:34] yeah, we delete donor details when we get the 'failed' message [21:33:02] yeah... maybe it's a bug at adyen. we should ask? [21:33:19] Fundraising Sprint Value Subtracting, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, FR-2016-17-Q2-Bugs: Engage import failing to import certain significant fields - https://phabricator.wikimedia.org/T146295#2797259 (Eileenmcnaughton) Hmm - There are still being created - e.g https://civicr... [21:34:03] ejegg: I was looking at this one https://phabricator.wikimedia.org/T146295 & I'm not quite clear if there is a problem or not. [21:34:06] Fundraising Sprint Value Subtracting, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, FR-2016-17-Q2-Bugs: Engage import failing to import certain significant fields - https://phabricator.wikimedia.org/T146295#2797263 (Eileenmcnaughton) SElect YEAR(receive_date), MONTH(receive_date), count(*),... [21:36:48] cwd i guess so. [21:37:17] i can write the email if you want [21:37:29] thing is, if donors have some way of reusing the iframe post, this is going to happen. I'm not sure how it happens, but that's all client-side. [21:38:10] isn't that what you just tested? [21:38:38] Nah, I tried two different iframe posts in different windows, with different order ids [21:40:02] then I tried pressing back after a failed charge, but in firefox the iframe was gone after the back button, and when I clicked 'continue' I got a new order ID [21:41:04] what link do you use for an adyen form? [21:41:45] eileen1: I'll take a look shortly! [21:42:03] cwd / eileen1 I just gotta get some food and recharge the laptop a bit. I'll be back before long [21:50:28] cool [22:07:46] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Edit Civi Reminder email messaging - https://phabricator.wikimedia.org/T150804#2797212 (DStrine) [22:08:49] Fundraising-Backlog: Spike: investigate odd safari behaviors - https://phabricator.wikimedia.org/T150792#2797332 (DStrine) p:Triage>Normal [22:13:04] Fundraising Sprint Value Subtracting, Fundraising-Backlog, FR-Paypal, Unplanned-Sprint-Work: Make PayPal IPN verification retry a few times - https://phabricator.wikimedia.org/T150780#2797365 (DStrine) [22:14:05] Fundraising Sprint Value Subtracting, Fundraising-Backlog, FR-Paypal, Unplanned-Sprint-Work: Make PayPal IPN verification retry a few times - https://phabricator.wikimedia.org/T150780#2796153 (cwdent) a:cwdent [22:14:44] Fundraising Sprint Value Subtracting, Fundraising-Backlog, FR-Amazon, FR-Smashpig, Unplanned-Sprint-Work: Amazon: SSL errors on closeOrderReference API call - https://phabricator.wikimedia.org/T150770#2797370 (DStrine) p:Triage>High [22:20:52] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Square Cash import not working - https://phabricator.wikimedia.org/T150810#2797389 (LeanneS) [22:33:56] Fundraising Dash, Fundraising-Backlog, Spike: Spike: create a donation funnel widget for the fr-tech dashboard - https://phabricator.wikimedia.org/T150789#2797442 (DStrine) [22:36:19] Fundraising Sprint Value Subtracting, Patch-For-Review, Unplanned-Sprint-Work: Direct Mail Appeal not being reflected correctly in Civi - https://phabricator.wikimedia.org/T150697#2797449 (DStrine) a:DStrine>None [22:38:04] Fundraising Sprint Value Subtracting, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Unplanned-Sprint-Work: Square Cash import not working - https://phabricator.wikimedia.org/T150810#2797451 (DStrine) p:Triage>High [22:41:14] Fundraising Sprint Value Subtracting, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Unplanned-Sprint-Work: Square Cash import not working - https://phabricator.wikimedia.org/T150810#2797389 (Eileenmcnaughton) @LeanneS can I get a copy of the file? [23:02:48] My unexciting update: fixing an issue with the server I use for IRC presence, some code review, merged CN master into wmf_deploy--someohow it didn't get on the train, not sure why--and looking at some of Ellery's old code for banner history, currently being worked on by CPS data folks... and engagement survey [23:02:52] fr-tech ^ [23:03:00] dstrine ^ [23:03:09] sorry AndyRussG, totally missed your standup! [23:03:20] AndyRussG: thanks and sorry that I got booted from the room [23:05:15] dstrine: ejegg: np! [23:05:34] * AndyRussG wipes away a tear of standup abandoment [23:05:42] (jk) ;p [23:05:44] sorry AndyRussG [23:05:51] eileen1: np.... :) [23:06:29] also - I have one minor patch merged - if anyone feels like they can look at another today I'll try to deploy both / all - but if everyone busy will just deploy that one [23:07:10] eileen1: which one needs looking at? [23:07:18] eileen1: I'd been looking at the errors/download patch [23:07:22] I can get back to that [23:07:45] XenoRyet: I think this is a fairly easy win https://gerrit.wikimedia.org/r/#/c/321612/ [23:08:00] Cool, I'll take a look. [23:09:23] esp since the non test change is only to accept 'United States' instead of 'US" [23:09:52] Yea, that part's really easy. Just looking over the test now. [23:09:52] Fundraising Sprint Value Subtracting, Fundraising-Backlog, FR-Paypal, Unplanned-Sprint-Work: Make PayPal IPN verification retry a few times - https://phabricator.wikimedia.org/T150780#2796153 (K4-713) I have done some archaeology in my own old inbox, and here are some things that I found which sh... [23:11:20] Yea, looks good to me. [23:11:26] (CR) XenoRyet: [C: 2] Fix Engage import handling of Postal. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321612 (https://phabricator.wikimedia.org/T148056) (owner: Eileen) [23:13:40] gonna bounce for a bit, back later! [23:14:12] thanks XenoRyet [23:14:23] No worries. [23:22:13] (CR) Ejegg: [C: 2] "Looks fine, smoke tested locally with no plume!" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321616 (https://phabricator.wikimedia.org/T88460) (owner: Eileen) [23:25:44] relocating [23:27:04] (Merged) jenkins-bot: Improve import to allow downloading of ALL non imported rows and add tests [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321616 (https://phabricator.wikimedia.org/T88460) (owner: Eileen) [23:27:06] (Merged) jenkins-bot: Fix Engage import handling of Postal. [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/321612 (https://phabricator.wikimedia.org/T148056) (owner: Eileen)