[01:08:01] does anyone else find they get pinged on slack but when they get to try to reply they can't actually find where the ping came from [01:19:43] Sometimes - but I set up slack to only show me unreads in my left column. So if a conversation thread (e.g. a single channel convo with just you) doesn't have a new convo from you in it (e.g. when you ping me to ask me a question) then it disappears. That way it's easier for me to find things I need to respond to [02:15:58] (03CR) 10Ejegg: [C:03+2] "Looks good!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014669 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [02:25:17] (03PS3) 10Ejegg: Move WMFTransaction to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015442 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [02:25:53] (03CR) 10Ejegg: [C:03+2] Move WMFTransaction to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015442 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [02:32:15] (03PS9) 10Ejegg: Add a getter for PaymentInstrument [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1012460 (https://phabricator.wikimedia.org/T357471) (owner: 10Eileen) [02:33:18] (03CR) 10Ejegg: [C:03+2] "New function looks logical. Could maybe lead to a different outcome in a weird edge case where both the id and the name were set in the me" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1012460 (https://phabricator.wikimedia.org/T357471) (owner: 10Eileen) [02:34:09] (03CR) 10Abijeet Patro: [V:03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_39) - 10https://gerrit.wikimedia.org/r/1015745 (owner: 10L10n-bot) [02:35:53] (03Merged) 10jenkins-bot: Introduce RecurringModifyAmountQueueConsumer & also add RecurringModifyAmountMessage & use it for basic validation [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014669 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [02:41:48] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Find all the places we aren't rounding properly resulting in weird floating point issues - https://phabricator.wikimedia.org/T353378#9678205 (10Cstone) [02:41:52] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Pending transaction resolver resolving multiple donations per donor per run - https://phabricator.wikimedia.org/T353089#9678208 (10Cstone) [02:42:09] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Cannot update the option value of a custom field - https://phabricator.wikimedia.org/T352343#9678214 (10Cstone) [02:42:21] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Uncaught TypeError: Cannot read properties of undefined (reading 'origin') - https://phabricator.wikimedia.org/T352458#9678217 (10Cstone) [02:44:38] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Email sent through civi giving Mailing Error Unable to send email. Please report this message to the site administrator - https://phabricator.wikimedia.org/T353975#9678227 (10Cstone) [02:44:59] 06Fundraising Tech Chaos Holding Pen, 06Fundraising-Backlog, 07Documentation: Document on wikitech the inputs behind our grafana graphs - https://phabricator.wikimedia.org/T332421#9678230 (10Cstone) [02:45:50] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07Documentation: Documentation: Update Payment Platform Providers - https://phabricator.wikimedia.org/T336003#9678232 (10Cstone) [02:48:05] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: 14Wired Client Error - 14https://phabricator.wikimedia.org/T310737#9678235 (10Cstone) 05Open→03Invalid 14I made this and its old and vague we can make a new one if we see it again [02:49:31] (03Merged) 10jenkins-bot: Move WMFTransaction to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015442 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [02:51:44] (03CR) 10Ejegg: "Just wondering if a bool arg to validate amounts would make sense" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1012606 (https://phabricator.wikimedia.org/T357471) (owner: 10Eileen) [02:52:45] (03Merged) 10jenkins-bot: Add a getter for PaymentInstrument [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1012460 (https://phabricator.wikimedia.org/T357471) (owner: 10Eileen) [02:56:15] (03PS2) 10Ejegg: Decommission wmf_civicrm function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1013722 (owner: 10Eileen) [02:56:17] (03CR) 10Ejegg: [C:03+2] Decommission wmf_civicrm function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1013722 (owner: 10Eileen) [02:59:37] (03CR) 10Ejegg: "I'm still ambivalent about hardcoding the currency in the function names rather than using settledCurrency or the like. Also the casing! I" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014667 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [02:59:57] eileen: I'll look at more patches tomorrow. Getting drowsy now. [03:03:40] (03CR) 10Cstone: "I think I was the one arguing for Usd, would it be called getSettlementAmount instead? We had gone deep into naming and were trying to avo" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014667 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [03:14:56] (03Merged) 10jenkins-bot: Decommission wmf_civicrm function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1013722 (owner: 10Eileen) [03:17:13] (03PS2) 10Cstone: Minor extration - getDonationMessage [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015443 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [03:21:04] (03CR) 10Cstone: "hmm this is causing a bunch of tests to fail locally with Error: Class 'WmfTransaction' not found, but not seeing anything obvious, will l" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015443 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [03:23:06] (03PS2) 10Cstone: Less array wrapping for RecurUpgrade API call [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014577 (owner: 10Ejegg) [03:28:59] (03CR) 10Cstone: [C:03+2] "farewell []" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014577 (owner: 10Ejegg) [03:30:30] 06Fundraising-Backlog, 07Epic: [EPIC] Fix and improve geolocation, and ensure it's working properly for FR campaigns - https://phabricator.wikimedia.org/T121937#9678284 (10Aklapper) [03:31:52] (03PS6) 10Cstone: If recurring, add (recurring) after description for dlocal console [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1008936 (https://phabricator.wikimedia.org/T356963) (owner: 10Wfan) [03:34:43] (03CR) 10Cstone: [C:03+2] If recurring, add (recurring) after description for dlocal console [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1008936 (https://phabricator.wikimedia.org/T356963) (owner: 10Wfan) [03:37:00] (03PS6) 10Eileen: Move validation of required fields, duplicates onto the message [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1012606 (https://phabricator.wikimedia.org/T357471) [03:37:14] (03CR) 10Eileen: Move validation of required fields, duplicates onto the message (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1012606 (https://phabricator.wikimedia.org/T357471) (owner: 10Eileen) [03:37:37] 06Fundraising-Backlog: Can we figure out a way to have all the descriptors come from one place - https://phabricator.wikimedia.org/T361539 (10Cstone) 03NEW [03:40:03] 06Fundraising-Backlog: Can we figure out a way to have all the descriptors come from one place - https://phabricator.wikimedia.org/T361539#9678321 (10Cstone) [03:48:05] (03Merged) 10jenkins-bot: Less array wrapping for RecurUpgrade API call [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014577 (owner: 10Ejegg) [03:48:06] (03Merged) 10jenkins-bot: If recurring, add (recurring) after description for dlocal console [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1008936 (https://phabricator.wikimedia.org/T356963) (owner: 10Wfan) [06:14:11] (03PS1) 10VolkerE: styles: Update color in from #d33 to #d73333 [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1016082 (https://phabricator.wikimedia.org/T343239) [06:25:32] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_40) - 10https://gerrit.wikimedia.org/r/1016131 (owner: 10L10n-bot) [07:56:41] (03CR) 10Eileen: "I think that might relate to another patch merged around the same time" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015443 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [09:06:02] (03CR) 10Abijeet Patro: [V:03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_40) - 10https://gerrit.wikimedia.org/r/1016131 (owner: 10L10n-bot) [13:16:55] (03CR) 10Ejegg: "I'm still in favor of moving to 'getSettledXXX' functions, but let's merge this to avoid a rebase mess, then decide on name changes in a l" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014667 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [13:24:39] (03CR) 10Ejegg: "Oh fun, the wmf_contribution_extra columns are settlement_currency and settlement_usd (!). But they seem to be completely unused. I still " [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014667 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [13:26:25] (03PS8) 10Ejegg: Rename get Amount functions to include usd [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014667 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [13:26:48] (03CR) 10Ejegg: [C:03+2] "Merging because following patches are C+2ed, but let's reconsider these names." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014667 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [13:27:29] (03PS5) 10Ejegg: Move some validation to the validate() function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014704 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [13:27:47] (03PS7) 10Ejegg: Consolidate amount handling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014921 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [13:28:07] (03CR) 10Ejegg: "needs manual rebase" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014705 (owner: 10Eileen) [13:45:48] (03Merged) 10jenkins-bot: Rename get Amount functions to include usd [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014667 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [13:47:14] (03CR) 10LWatson: [C:03+2] styles: Update color in from #d33 to #d73333 [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1016082 (https://phabricator.wikimedia.org/T343239) (owner: 10VolkerE) [13:52:05] (03Merged) 10jenkins-bot: styles: Update color in from #d33 to #d73333 [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1016082 (https://phabricator.wikimedia.org/T343239) (owner: 10VolkerE) [13:52:23] (03CR) 10LWatson: [V:03+2 C:03+2] styles: Update color in from #d33 to #d73333 [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1016082 (https://phabricator.wikimedia.org/T343239) (owner: 10VolkerE) [14:05:21] (03PS6) 10Ejegg: Use more get functions for amounts [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014705 (owner: 10Eileen) [14:33:43] (03CR) 10Ejegg: [C:03+2] "Re-applying C+2 after manual rebase" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014705 (owner: 10Eileen) [14:37:40] (03PS1) 10Umherirrender: Migrate to IDatabase::newInsertQueryBuilder for upsert [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1016353 [15:00:20] (03Merged) 10jenkins-bot: Move some validation to the validate() function [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014704 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [15:00:24] (03Merged) 10jenkins-bot: Consolidate amount handling [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014921 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [15:00:25] (03Merged) 10jenkins-bot: Use more get functions for amounts [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014705 (owner: 10Eileen) [15:04:46] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-donorservices, 13Patch-For-Review: Civi Snooze feature stopped working! - https://phabricator.wikimedia.org/T358259#9680673 (10SHust) @Damilare thanks for the awesome news, and all the work to make this happen! [15:13:22] (03PS9) 10Ejegg: Use SmashPig to get PayPal donor details [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1008584 (https://phabricator.wikimedia.org/T336449) [15:23:34] (03PS2) 10Damilare Adedoyin: WIP: Fundraiseup audit import for recur plan change [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015383 (https://phabricator.wikimedia.org/T349663) [15:36:39] (03PS3) 10Damilare Adedoyin: WIP: Fundraiseup audit import for recur plan change [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015383 (https://phabricator.wikimedia.org/T349663) [15:47:23] (03PS10) 10Ejegg: Use SmashPig to get PayPal donor details [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1008584 (https://phabricator.wikimedia.org/T336449) [15:47:23] (03PS1) 10Ejegg: WIP use SmashPig for PayPal approvePayment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1016388 (https://phabricator.wikimedia.org/T336449) [15:51:18] (03CR) 10CI reject: [V:04-1] WIP: Fundraiseup audit import for recur plan change [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1015383 (https://phabricator.wikimedia.org/T349663) (owner: 10Damilare Adedoyin) [15:51:44] 03Fundraising Sprint: fridayFridayGottaGetDownOnFriday($fun,$fun,$fun), 06Fundraising-Backlog, 13Patch-For-Review: Update contact on CiviCRM when FRUP donor updates their email through donor portal - https://phabricator.wikimedia.org/T349663#9681051 (10Damilare) a:03Damilare [15:51:48] 06Fundraising-Backlog: Adyen request for info / troubleshooting Maestro BIN - https://phabricator.wikimedia.org/T361610 (10MBeat33) 03NEW [15:52:40] (03PS11) 10Ejegg: Use SmashPig to get PayPal donor details [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1008584 (https://phabricator.wikimedia.org/T336449) [15:52:41] (03PS2) 10Ejegg: WIP use SmashPig for PayPal approvePayment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1016388 (https://phabricator.wikimedia.org/T336449) [15:54:59] (03CR) 10CI reject: [V:04-1] WIP use SmashPig for PayPal approvePayment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1016388 (https://phabricator.wikimedia.org/T336449) (owner: 10Ejegg) [15:55:22] (03CR) 10CI reject: [V:04-1] Use SmashPig to get PayPal donor details [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1008584 (https://phabricator.wikimedia.org/T336449) (owner: 10Ejegg) [17:18:30] 06Fundraising-Backlog: CiviMail Unsubscribe Footer Broken - https://phabricator.wikimedia.org/T361630 (10AKanji-WMF) 03NEW [17:19:17] 06Fundraising-Backlog: CiviMail Unsubscribe Footer Broken - https://phabricator.wikimedia.org/T361630#9681588 (10AKanji-WMF) Clicking on the link @Eileenmcnaughton in the email I receive (both test and final) try to create an email send as per the image. [17:25:30] oops, we need to add a new payment instrument in Civi for Bancomer [17:32:03] ejegg: im looking at that [17:32:07] its there its because we changed it [17:32:14] sorry was in the other channel [17:32:25] we need to update the old one? or just have two [17:34:41] oh huh, is it in that big switch statement? [17:35:04] it got switched to type BT in the payment method cleanup [17:35:17] in the databse its not Bank Transfer: Bancomer is that the issue? [17:35:26] yep cstone, that would be it [17:36:08] maybe it is easier to just add a new one the editing UI is vague [17:36:31] do we usually just add them in the UI ejegg or through code [17:36:45] through code, so it matches in prod + dev [17:37:26] PaymentInstruments.mgd.php [17:37:52] so it would just changed to Bank Transfer: Bancomer there? [17:39:04] (03PS1) 10Cstone: Update Bancomer to be Bank Transfer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016404 [17:42:20] fr-tech ^ that should let us get those donations into civi [17:43:16] (03PS2) 10Ejegg: Update Bancomer to be Bank Transfer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016404 (owner: 10Cstone) [17:43:17] (03CR) 10Ejegg: [C:03+2] Update Bancomer to be Bank Transfer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016404 (owner: 10Cstone) [17:44:03] let's see, i think that will not delete the old one as long as there are some records linked to it [17:44:12] yeah or should I have left the old one? [17:47:12] we just might want to update all those to point to the new value [17:47:25] but for now that one change will let us re-import the damaged messages [17:47:33] and get the thank you emails out [18:03:30] (03Merged) 10jenkins-bot: Update Bancomer to be Bank Transfer [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016404 (owner: 10Cstone) [18:07:29] 06Fundraising-Backlog: Search kit fields for pledges - https://phabricator.wikimedia.org/T361635 (10MDemosWMF) 03NEW [18:13:47] does wmf_civicrm.info differ between dev and production? [18:14:04] this actually might explain my test issue last night [18:14:08] 06Fundraising-Backlog, 10FR-donorservices: ask Civi to use PayPal's API when humans cancel recurrings - https://phabricator.wikimedia.org/T361636 (10MBeat33) 03NEW [18:21:59] (03CR) 10Eileen: "thanks ejegg - I'm up for reconsidering the names but yeah prob easier to do on top" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1014667 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [18:38:57] (03PS1) 10Eileen: Switch usd to Settled [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016419 [18:39:44] (03PS1) 10Cstone: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1016420 [18:46:05] (03CR) 10Cstone: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1016420 (owner: 10Cstone) [18:46:56] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1016420 (owner: 10Cstone) [18:48:36] !log civicrm upgraded from ed776060 to 5b7f9e06 [18:48:37] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:55:33] (03PS1) 10Eileen: Remove old hook call [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016434 (https://phabricator.wikimedia.org/T357469) [20:39:53] ejegg: I am not sure https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1012606 will be the final word but I think it will be easier to get it merged & 'circle back' (that one was for dwisehaupt & anil) [20:48:05] eileen: hmm, ok. How about just rename that property from isPayment to needsAmountValidation? [20:48:08] ehh [20:49:35] ejegg: I think where we will wind up is that we will have a getter isPayment() that looks at the details of the message to see if it is a payment - which would mostly be by virture of looking up the txn_type [20:55:13] ok, i'll look again after a preschool run! [21:41:42] (03PS1) 10Eileen: Move some recur payment tests to the extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016445 (https://phabricator.wikimedia.org/T357469) [22:13:03] (03PS1) 10Eileen: Sync old RecurTest class with the new one a bit [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016451 (https://phabricator.wikimedia.org/T357469) [22:34:18] (03PS1) 10Eileen: Further sync from old test class to new [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016455 (https://phabricator.wikimedia.org/T357469) [22:52:57] (03CR) 10CI reject: [V:04-1] Further sync from old test class to new [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016455 (https://phabricator.wikimedia.org/T357469) (owner: 10Eileen) [22:55:57] (03PS2) 10Eileen: Further sync from old test class to new [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016455 (https://phabricator.wikimedia.org/T357469) [23:21:24] (03PS1) 10Eileen: Move 2 more tests over [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016461 (https://phabricator.wikimedia.org/T357469) [23:29:55] (03PS12) 10Ejegg: Use SmashPig to get PayPal donor details [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1008584 (https://phabricator.wikimedia.org/T336449) [23:29:55] (03PS3) 10Ejegg: WIP use SmashPig for PayPal approvePayment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1016388 (https://phabricator.wikimedia.org/T336449) [23:31:24] (03CR) 10CI reject: [V:04-1] WIP use SmashPig for PayPal approvePayment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1016388 (https://phabricator.wikimedia.org/T336449) (owner: 10Ejegg) [23:46:29] (03PS1) 10Eileen: RecurTest - extract getDamagedRows() [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1016464 (https://phabricator.wikimedia.org/T357469) [23:52:29] (03PS13) 10Ejegg: Use SmashPig to make initial PayPal EC setup calls [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1004252 (https://phabricator.wikimedia.org/T336449) [23:52:29] (03PS13) 10Ejegg: Use SmashPig to get PayPal donor details [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1008584 (https://phabricator.wikimedia.org/T336449) [23:52:29] (03PS4) 10Ejegg: WIP use SmashPig for PayPal approvePayment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1016388 (https://phabricator.wikimedia.org/T336449) [23:54:28] (03CR) 10CI reject: [V:04-1] WIP use SmashPig for PayPal approvePayment [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1016388 (https://phabricator.wikimedia.org/T336449) (owner: 10Ejegg)