[00:13:29] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Refund shows as contribution line without original contribution line - https://phabricator.wikimedia.org/T393036#10846258 (10AKanji-WMF) Gravy mentioned this is a known issue, a mismatch in settlement reporting, and they can create separate line items - a... [01:08:32] 03Fundraising Sprint: Jollof Rice, 06Fundraising-Backlog: Test and remove user information fields in Trustly ACH flow - https://phabricator.wikimedia.org/T393647#10846280 (10AnnWF) [01:12:34] (03PS1) 10Wfan: Add email from ach form as buyer external identifier [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148992 (https://phabricator.wikimedia.org/T393647) [01:16:44] (03PS1) 10Wfan: still use email from ach form since trustly not always contain email [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1148993 (https://phabricator.wikimedia.org/T393647) [01:17:47] 03Fundraising Sprint: Jollof Rice, 06Fundraising-Backlog, 13Patch-For-Review: Test and remove user information fields in Trustly ACH flow - https://phabricator.wikimedia.org/T393647#10846290 (10AnnWF) a:03AnnWF [01:28:31] 03Fundraising Sprint: Jollof Rice, 06Fundraising-Backlog, 13Patch-For-Review, 07payments-orchestration: Add placeholder for Gravy Phone number for IN - https://phabricator.wikimedia.org/T391970#10846293 (10AnnWF) [06:02:53] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Mass Card Declines Following Japan Email 1 - https://phabricator.wikimedia.org/T394007#10846640 (10RKumar_WMF) {F60367377} We tried to test in Japan using JCB card but still seeing the same error in Gr4vy console. Gr4vy transact... [06:03:22] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Mass Card Declines Following Japan Email 1 - https://phabricator.wikimedia.org/T394007#10846641 (10RKumar_WMF) 05Resolved→03Open [06:06:36] 06Fundraising-Backlog: PIX and Boleto after completing the donation do not show TY page - https://phabricator.wikimedia.org/T394979 (10RKumar_WMF) 03NEW [06:07:11] PROBLEM - check_load on fransw2002 is CRITICAL: LOAD CRITICAL - total load average: 10.43, 20.70, 12.30 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2002&service=check_load [06:12:11] RECOVERY - check_load on fransw2002 is OK: LOAD OK - total load average: 0.79, 8.28, 9.25 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=fransw2002&service=check_load [06:39:47] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_42) - 10https://gerrit.wikimedia.org/r/1149052 (owner: 10L10n-bot) [08:22:12] (03CR) 10Abijeet Patro: [V:03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_42) - 10https://gerrit.wikimedia.org/r/1149052 (owner: 10L10n-bot) [08:42:13] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: No support of required debit mandate for SEPA - https://phabricator.wikimedia.org/T394932#10847063 (10krobinson) Thanks @EMartin - do we need to include Adyen here? If so, we may need to explain who Adyen is on the form otherwise it may raise security con... [08:44:08] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: No support of required debit mandate for SEPA - https://phabricator.wikimedia.org/T394932#10847072 (10krobinson) @jgleeson let me know if we need translations - we can do some of them in house. cc @CDenes_WMF too for those as @TomaszGorski is out. [08:54:38] 06Fundraising-Backlog: Recurring token migration from all PSPs to Gr4vy - https://phabricator.wikimedia.org/T394996 (10RKumar_WMF) 03NEW [08:56:20] 06Fundraising-Backlog: Recurring token migration from all PSPs to Gr4vy - https://phabricator.wikimedia.org/T394996#10847135 (10RKumar_WMF) Given the adoption of Gr4vy, we will need consolidate various recurring donation tokens onto the platform. Migrations of the various specific recurring tokens onto Gr4vy to... [09:06:34] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Sweden cards high failure rate - https://phabricator.wikimedia.org/T394911#10847190 (10krobinson) This was bugging me and so I did some digging in our campaign reporting, and we used to have super high 3DS failure rates in Sweden with Ingenico, which drop... [09:34:09] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Mass Card Declines Following Japan Email 1 - https://phabricator.wikimedia.org/T394007#10847320 (10jgleeson) I've confirmed we are using the latest details sent from Adyen for JCB 3D-Secure. However, it's still failing. I've emaile... [10:42:00] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fail Mail (civi1002) run-job: Silverpop emails - https://phabricator.wikimedia.org/T395011 (10ppenloglou) 03NEW [10:58:03] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fail Mail (civi1002) run-job: Silverpop emails - https://phabricator.wikimedia.org/T395011#10847589 (10ppenloglou) [11:42:54] (03CR) 10Damilare Adedoyin: [C:03+2] Select active translation in API4 wrapper, not draft (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1148910 (https://phabricator.wikimedia.org/T392300) (owner: 10Ejegg) [11:51:01] (03CR) 10Jgleeson: [C:04-1] "Thanks Dami. I have some concerns with the current approach, and I've written them up with a suggested alternative inline. Happy to discus" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148895 (https://phabricator.wikimedia.org/T393958) (owner: 10Damilare Adedoyin) [12:02:19] (03Merged) 10jenkins-bot: Select active translation in API4 wrapper, not draft [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1148910 (https://phabricator.wikimedia.org/T392300) (owner: 10Ejegg) [12:03:07] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: No support of required debit mandate for SEPA - https://phabricator.wikimedia.org/T394932#10847822 (10jgleeson) a:03jgleeson [12:50:01] 14Fundraising Sprint Impossible shepherds pie: End of Sprint Notes - https://phabricator.wikimedia.org/T394773#10847979 (10Damilare) [13:07:33] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: PayPal 'contribution_info' donation missing from Civi - https://phabricator.wikimedia.org/T387171#10848033 (10Damilare) Hi @SHust, I expect the dedupe job to pick some of this as they fit into the rule group. However, some of th... [13:58:05] (03PS1) 10Jgleeson: WIP: Add SEPA mandate notice to SEPA forms [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1149411 (https://phabricator.wikimedia.org/T394932) [13:59:16] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: No support of required debit mandate for SEPA - https://phabricator.wikimedia.org/T394932#10848324 (10jgleeson) @EMartin @krobinson is this placement and font ok? {F60378945, size=full} [13:59:48] (03CR) 10CI reject: [V:04-1] WIP: Add SEPA mandate notice to SEPA forms [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1149411 (https://phabricator.wikimedia.org/T394932) (owner: 10Jgleeson) [14:39:33] (03CR) 10Damilare Adedoyin: "Thanks for the CR jgleeson, I considered your suggestions but couldn't find a path forward, I added my thoughts in the response." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148895 (https://phabricator.wikimedia.org/T393958) (owner: 10Damilare Adedoyin) [14:48:15] PROBLEM - check_mysql on frdb1004 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 11635 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [14:53:15] RECOVERY - check_mysql on frdb1004 is OK: Uptime: 1957745 Threads: 6 Questions: 100180695 Slow queries: 963 Opens: 6677 Open tables: 1241 Queries per second avg: 51.171 Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 0 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb1004&service=check_mysql [15:12:14] (03CR) 10Jgleeson: [C:04-1] Make getResponseMapper public in RedirectPaymentProvider class (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148895 (https://phabricator.wikimedia.org/T393958) (owner: 10Damilare Adedoyin) [15:30:03] (03PS2) 10Damilare Adedoyin: WIP [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148920 [15:30:29] (03CR) 10CI reject: [V:04-1] WIP [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148920 (owner: 10Damilare Adedoyin) [15:47:53] (03PS2) 10Damilare Adedoyin: Make getResponseMapper public in RedirectPaymentProvider class [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148895 (https://phabricator.wikimedia.org/T393958) [15:47:58] (03PS3) 10Damilare Adedoyin: WIP [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148920 [15:48:26] (03CR) 10CI reject: [V:04-1] Make getResponseMapper public in RedirectPaymentProvider class [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148895 (https://phabricator.wikimedia.org/T393958) (owner: 10Damilare Adedoyin) [15:48:30] (03CR) 10CI reject: [V:04-1] WIP [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148920 (owner: 10Damilare Adedoyin) [16:19:33] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: No support of required debit mandate for SEPA - https://phabricator.wikimedia.org/T394932#10849271 (10EMartin) Looks great. Thanks. [16:21:00] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fail Mail (civi1002) run-job: Silverpop emails - https://phabricator.wikimedia.org/T395011#10849274 (10AKanji-WMF) p:05Triage→03High [16:41:47] 03Fundraising Sprint: Jollof Rice, 06Fundraising-Backlog, 10fundraising-tech-ops, 06Fundraising-Tech-Roadmap: FR-Tech FY2425Q4 maintenance window - https://phabricator.wikimedia.org/T337585#10849374 (10Dwisehaupt) [17:06:53] 06Fundraising-Backlog, 06FR-donorrelations: Gravy & the PayPal recurring cancellation process - https://phabricator.wikimedia.org/T393226#10849488 (10AKanji-WMF) p:05Triage→03High [17:25:57] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fail Mail (civi1002) run-job: Silverpop emails - https://phabricator.wikimedia.org/T395011#10849577 (10jgleeson) a:03jgleeson [17:26:38] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface: Banner history log IDs missing since May 2024 - https://phabricator.wikimedia.org/T395060 (10Ejegg) 03NEW [17:29:16] (03PS1) 10Ejegg: Fix sending banner history log id to ct queue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1149450 (https://phabricator.wikimedia.org/T395060) [17:35:47] (03CR) 10Cstone: [C:03+2] "works for me" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1149450 (https://phabricator.wikimedia.org/T395060) (owner: 10Ejegg) [17:42:12] 06Fundraising-Backlog: braintree webhook for PaymentMethodCustomerDataUpdated - https://phabricator.wikimedia.org/T344935#10849724 (10AnnWF) a:05AnnWF→03None [17:42:33] 06Fundraising-Backlog: Bizum online banking for Spain - https://phabricator.wikimedia.org/T358517#10849725 (10AnnWF) a:05AnnWF→03None [17:42:53] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1149451 [17:42:57] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1149451 (owner: 10Ejegg) [17:43:48] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1149451 (owner: 10Ejegg) [17:46:30] !log fundraising civicrm upgraded from 5b155eaa to 4dc5f911 [17:46:32] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [17:47:02] 06Fundraising-Backlog: Get better stats on the silver pop upload - https://phabricator.wikimedia.org/T395067 (10Cstone) 03NEW [17:47:40] 06Fundraising-Backlog: Get better stats on the silverpop upload - https://phabricator.wikimedia.org/T395067#10849762 (10Cstone) [17:49:44] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fail Mail (civi1002) run-job: Silverpop emails - https://phabricator.wikimedia.org/T395011#10849775 (10Eileenmcnaughton) Hi @PPenloglou-WMF this doesn't seem quite clear- the nightly job that has 30 million-ish rows is just the checksum job - the main dat... [17:49:46] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: CiviCRM Message Templates: "Draft" version being sent instead of "Active" - https://phabricator.wikimedia.org/T392300#10849777 (10Ejegg) >>! In T392300#10820836, @ppenloglou wrote: > Hi team! Wondering about the status on this one? Thanks! Hi @ppenloglou... [17:52:10] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Transactions settled at Adyen but stuck at Capturing at Gr4vy? - https://phabricator.wikimedia.org/T393112#10849801 (10MBeat33) @jgleeson I checked on the three transactions from the beginning of this Task: 1. cid=66464263 37a1f3d... [17:55:17] (03Merged) 10jenkins-bot: Fix sending banner history log id to ct queue [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1149450 (https://phabricator.wikimedia.org/T395060) (owner: 10Ejegg) [17:57:07] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Transactions settled at Adyen but stuck at Capturing at Gr4vy? - https://phabricator.wikimedia.org/T393112#10849818 (10jgleeson) Thanks @MBeat33. The one on the 16th was one of the non-recurring stragglers that got picked up and ad... [17:58:33] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Backfill greeting fields - https://phabricator.wikimedia.org/T321609#10849837 (10Eileenmcnaughton) a:05Eileenmcnaughton→03None [18:00:52] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: eqiad: determine second frack - https://phabricator.wikimedia.org/T392007#10849847 (10RobH) 05Open→03Resolved I should have closed this, we went with ordering a second rack and putting it into the new expansion... [18:01:29] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1149453 [18:01:33] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1149453 (owner: 10Ejegg) [18:02:05] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: eqiad: second frack parent tracking task - https://phabricator.wikimedia.org/T392006#10849856 (10RobH) The solution selected was ordering and installing a second new rack in the ML expansion row for frack usage. So no... [18:02:35] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1149453 (owner: 10Ejegg) [18:05:24] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fail Mail (civi1002) run-job: Silverpop emails - https://phabricator.wikimedia.org/T395011#10849896 (10jgleeson) @Eileenmcnaughton, that looks right to me. The numbers for the main job look consistent based on what I see in the logs below, and the checksu... [18:21:35] 06Fundraising-Backlog, 10fundraising-tech-ops, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM, 10Fr-drupal-upgrade-2021: Next maintenance window - Add banner_history_id to civicrm contribution tracking table - https://phabricator.wikimedia.org/T357375#10850028 (10Dwisehaupt) 05Open→03... [18:33:10] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Fail Mail (civi1002) run-job: Silverpop emails - https://phabricator.wikimedia.org/T395011#10850118 (10ppenloglou) Thanks @Eileenmcnaughton , I don't think I had a clear understanding of each job... Thanks for sharing the wikitech page as well! [18:47:10] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06FR-donorrelations: Transactions settled at Adyen but stuck at Capturing at Gr4vy? - https://phabricator.wikimedia.org/T393112#10850189 (10Damilare) Thanks @jgleeson and @MBeat33 Gravy sent the [[ https://wikimedia.slack.com/archives/C070F1DVBRN/p17479... [18:52:55] !log payments-wiki upgraded from 7537e0df to 1a4ef678 [18:52:57] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [19:01:46] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface, 10MW-1.45-notes (1.45.0-wmf.3; 2025-05-27): Banner history log IDs missing since May 2024 - https://phabricator.wikimedia.org/T395060#10850251 (10Ejegg) OK, we have new data coming into this column. We may be... [21:03:23] 14Fundraising Sprint Turtles that are robotic that destroy the whole world with their foot theory, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 07fr-acoustic: Adding and storing rml_source data for RML donations reporting - https://phabricator.wikimedia.org/T321163#10850698 (10Eileenmcnaughton) a:05... [21:03:40] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM, 10Fr-drupal-upgrade-2021: Either remove option to send a receipt from the Contribution edit page, or fix to send a thank you - https://phabricator.wikimedia.org/T325956#10850703 (10Eileenmcnaughton) a:05Eileenmcnaughton→... [21:10:41] 06Fundraising-Backlog, 06FR-donorrelations: JO donation form: other amount issue - https://phabricator.wikimedia.org/T395093 (10MBeat33) 03NEW [21:34:52] 10fundraising-tech-ops, 06Data-Engineering, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Low volume in new webrequest feed - https://phabricator.wikimedia.org/T395089#10850795 (10Jgreen) [21:47:39] 10fundraising-tech-ops, 06Data-Engineering, 10FR-Tech-Analytics, 10FR-tech-data-integrity: Low volume in new webrequest feed - https://phabricator.wikimedia.org/T395089#10850820 (10Jgreen) [21:59:16] (03PS1) 10Jgleeson: Gravy: inject mappers & validators via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149494 (https://phabricator.wikimedia.org/T393958) [21:59:17] (03PS1) 10Jgleeson: Gravy: update BT/RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149495 (https://phabricator.wikimedia.org/T393958) [21:59:18] (03PS1) 10Jgleeson: Gravy: update ApplePayPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149496 (https://phabricator.wikimedia.org/T393958) [21:59:19] (03PS1) 10Jgleeson: Gravy: update GooglePayPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149497 (https://phabricator.wikimedia.org/T393958) [21:59:21] (03PS1) 10Jgleeson: Gravy: update BankPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149498 (https://phabricator.wikimedia.org/T393958) [21:59:22] (03PS1) 10Jgleeson: Gravy: update PayPalPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149499 (https://phabricator.wikimedia.org/T393958) [21:59:24] (03PS1) 10Jgleeson: Gravy: update Venmo RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149500 (https://phabricator.wikimedia.org/T393958) [21:59:27] (03PS1) 10Jgleeson: Gravy: cash RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149501 (https://phabricator.wikimedia.org/T393958) [22:00:19] (03PS2) 10Jgleeson: Gravy: update cash RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149501 (https://phabricator.wikimedia.org/T393958) [22:00:21] (03CR) 10CI reject: [V:04-1] Gravy: update BankPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149498 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:00:37] (03CR) 10CI reject: [V:04-1] Gravy: update PayPalPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149499 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:00:44] (03CR) 10CI reject: [V:04-1] Gravy: update Venmo RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149500 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:00:55] (03CR) 10CI reject: [V:04-1] Gravy: update cash RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149501 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:01:04] (03CR) 10CI reject: [V:04-1] Gravy: update cash RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149501 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:13:47] (03PS2) 10Jgleeson: Gravy: inject mappers & validators via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149494 (https://phabricator.wikimedia.org/T393958) [22:13:47] (03PS2) 10Jgleeson: Gravy: update BT/RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149495 (https://phabricator.wikimedia.org/T393958) [22:13:47] (03PS2) 10Jgleeson: Gravy: update ApplePayPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149496 (https://phabricator.wikimedia.org/T393958) [22:13:48] (03PS2) 10Jgleeson: Gravy: update GooglePayPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149497 (https://phabricator.wikimedia.org/T393958) [22:13:48] (03PS2) 10Jgleeson: Gravy: update DirectDepositPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149498 (https://phabricator.wikimedia.org/T393958) [22:13:51] (03PS2) 10Jgleeson: Gravy: update PayPalPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149499 (https://phabricator.wikimedia.org/T393958) [22:13:55] (03PS2) 10Jgleeson: Gravy: update Venmo RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149500 (https://phabricator.wikimedia.org/T393958) [22:13:59] (03PS3) 10Jgleeson: Gravy: update cash RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149501 (https://phabricator.wikimedia.org/T393958) [22:14:26] I wish I would have pulled before pushing all of those out :) [22:15:08] (03CR) 10CI reject: [V:04-1] Gravy: update DirectDepositPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149498 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:15:17] (03CR) 10CI reject: [V:04-1] Gravy: update PayPalPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149499 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:15:24] (03CR) 10CI reject: [V:04-1] Gravy: update Venmo RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149500 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:15:33] (03CR) 10CI reject: [V:04-1] Gravy: update cash RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149501 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:16:20] (03PS3) 10Jgleeson: Gravy: update DirectDepositPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149498 (https://phabricator.wikimedia.org/T393958) [22:16:55] (03CR) 10CI reject: [V:04-1] Gravy: update DirectDepositPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149498 (https://phabricator.wikimedia.org/T393958) (owner: 10Jgleeson) [22:19:52] (03PS4) 10Jgleeson: Gravy: update DirectDepositPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149498 (https://phabricator.wikimedia.org/T393958) [22:19:52] (03PS3) 10Jgleeson: Gravy: update PayPalPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149499 (https://phabricator.wikimedia.org/T393958) [22:19:52] (03PS3) 10Jgleeson: Gravy: update Venmo RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149500 (https://phabricator.wikimedia.org/T393958) [22:19:53] (03PS4) 10Jgleeson: Gravy: update cash RedirectPaymentProvider to use DI via config [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1149501 (https://phabricator.wikimedia.org/T393958) [22:23:05] CI gods are happy [22:32:55] (03CR) 10Jgleeson: Make getResponseMapper public in RedirectPaymentProvider class (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1148895 (https://phabricator.wikimedia.org/T393958) (owner: 10Damilare Adedoyin)