[04:23:06] (03PS3) 10Eileen: civicrm5.66-rc - stock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/952953 (https://phabricator.wikimedia.org/T344663) [04:23:08] (03PS2) 10Eileen: Do not update tax_amount to 0 - too slow.... [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/954989 (https://phabricator.wikimedia.org/T344663) [05:07:58] (03PS4) 10Eileen: civicrm5.66-rc - stock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/952953 (https://phabricator.wikimedia.org/T344663) [05:10:47] (03PS3) 10Eileen: Do not update tax_amount to 0 - too slow.... [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/954989 (https://phabricator.wikimedia.org/T344663) [07:51:35] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_40) - 10https://gerrit.wikimedia.org/r/955504 (owner: 10L10n-bot) [08:53:17] 10Fundraising-Backlog: India Recurring - Figure out when to actually cancel donations - https://phabricator.wikimedia.org/T339181 (10krobinson) Documenting this for posterity, as recommended by @Cstone We've seen a couple of cases of donors setting up a new recurring thinking the old one didn't work, and endin... [09:01:16] 10Fundraising-Backlog, 10fundraising-tech-ops: Set up Community CiviCRM instance. - https://phabricator.wikimedia.org/T343486 (10Qgil) (((I just want to say thank you for your notes here. I'm reading them with high interest and if Phabricator would feature emoji reactions I wouldn't have posted this noisy usel... [11:53:32] (03PS6) 10Damilare Adedoyin: Disable Fiscal Number Field in Dlocal ZA [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/953594 (https://phabricator.wikimedia.org/T307743) [12:22:26] (03CR) 10Damilare Adedoyin: "thanks jgleeson for the CR, I've moved the method to the parent class to allow access in the cardPaymentProvider class" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/954299 (https://phabricator.wikimedia.org/T307743) (owner: 10Damilare Adedoyin) [13:23:34] (03CR) 10Ejegg: "One simplification possible, and we should probably test the new countries that aren't in Adyen's documentation as supported, before addin" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/954983 (https://phabricator.wikimedia.org/T328964) (owner: 10Wfan) [13:39:20] 10Fundraising Sprint Quit playing games with my Phab 🎶, 10Fundraising-Backlog, 10Patch-For-Review: Change non campaign markets from Ingenico to Adyen - https://phabricator.wikimedia.org/T328964 (10Ejegg) @EMartin Many of these countries are not listed as supported in Adyen's documentation, or only support pa... [13:40:53] rockin Jeff_Green ! Well, I guess just the boxes where we use the SmashPig library or standalone [13:41:11] so that's frpig, civi, and payments [13:41:20] hmm, maybe email pref center as well [13:41:41] I'm just telling the releng team we can shut off php7.3 testing for the SmashPig repo [13:42:46] yup, makes sense! [13:45:12] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Adyen SEPA Direct debit unsettled transactions June - August 2023 - https://phabricator.wikimedia.org/T344203 (10Ejegg) Thinking about ways we could have detected this: The problem was that we wrote code assuming the Adyen settings would remain at auto-ca... [13:59:18] (03PS1) 10Ejegg: Update minimum php version to php7.4, update dependencies [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/955739 [13:59:47] (03CR) 10CI reject: [V: 04-1] Update minimum php version to php7.4, update dependencies [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/955739 (owner: 10Ejegg) [14:00:09] fr-tech I think those dependency updates should be fine ^^ mostly just point releases, except for a dev-dependency [14:00:21] right, it'll fail the php7.3 tests till those are removed [14:09:28] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Donate form UI work - https://phabricator.wikimedia.org/T345846 (10Tsevener) [14:10:38] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: As a donor I want to be able to make an easy and convenient donation via Apple Pay in the Wikipedia app - https://phabricator.wikimedia.org/T288285 (10Tsevener) a:05Tsevener→03scblr [14:15:46] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Fetch and cache payment networks and config - https://phabricator.wikimedia.org/T345847 (10Tsevener) [14:22:38] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Add path to donate form - https://phabricator.wikimedia.org/T345851 (10Tsevener) [14:27:14] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Add script to inject testing and production merchant IDs and FR Tech API key - https://phabricator.wikimedia.org/T345852 (10Tsevener) [14:27:30] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Add script to inject testing and production merchant IDs and FR Tech API key - https://phabricator.wikimedia.org/T345852 (10Tsevener) [14:30:11] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Add script to inject testing and production merchant IDs and FR Tech API key - https://phabricator.wikimedia.org/T345852 (10Tsevener) a:05scblr→03Tsevener [14:30:27] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Add path to donate form - https://phabricator.wikimedia.org/T345851 (10Tsevener) a:05scblr→03Tsevener [14:30:44] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Fetch and cache payment networks and config - https://phabricator.wikimedia.org/T345847 (10Tsevener) a:05scblr→03Tsevener [14:33:05] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Donate form UI work - https://phabricator.wikimedia.org/T345846 (10Tsevener) p:05Triage→03High [14:33:09] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Fetch and cache payment networks and config - https://phabricator.wikimedia.org/T345847 (10Tsevener) p:05Triage→03High [14:33:15] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Add script to inject testing and production merchant IDs and FR Tech API key - https://phabricator.wikimedia.org/T345852 (10Tsevener) p:05Triage→03Medium [14:33:21] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Add path to donate form - https://phabricator.wikimedia.org/T345851 (10Tsevener) p:05Triage→03Medium [14:44:49] (03PS1) 10Ejegg: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/955756 [14:45:25] fr-tech does that all look fine to go out to production? ^^^ [14:45:41] I'm pretty solid on the table pager change [14:48:13] 10Fundraising-Backlog, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Apple Pay: Add path to donate form - https://phabricator.wikimedia.org/T345851 (10Tsevener) [15:07:48] (03CR) 10Jgleeson: [C: 03+2] "Thanks for the work on this Dami!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/954299 (https://phabricator.wikimedia.org/T307743) (owner: 10Damilare Adedoyin) [15:08:33] (03Merged) 10jenkins-bot: Add Placeholder value in fiscal_number field for Dlocal ZA transactions [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/954299 (https://phabricator.wikimedia.org/T307743) (owner: 10Damilare Adedoyin) [15:13:11] ejegg: that patch LGTM [15:13:33] do you want another body to +2 [15:15:51] 10Fundraising-Backlog: Check the Adyen audit files August 25 to September 15 - https://phabricator.wikimedia.org/T343171 (10Ejegg) Change log for the settlement detail report: https://assets.ctfassets.net/ihg38ochctiz/6x5yvTSoSSUH09VzGAxsZA/4ba6b961ff84ff238c5f47c80aea0025/Change_log_-_Settlement_Detail_Report... [15:17:57] sure, thanks jgleeson ! [15:18:35] (03CR) 10Jgleeson: [C: 03+2] "LGTM!" [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/955756 (owner: 10Ejegg) [15:19:06] rockin, that should go out with the train next week I believe [15:19:09] (03CR) 10Jforrester: "<3" [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/955756 (owner: 10Ejegg) [15:31:07] (03Merged) 10jenkins-bot: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/955756 (owner: 10Ejegg) [15:39:56] (03PS1) 10Damilare Adedoyin: WIP: Implement Auto-Rescue Webhook handling [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/955775 (https://phabricator.wikimedia.org/T340164) [15:44:01] thanks Jack for the CR [15:45:23] np! [15:59:22] (03PS2) 10Wfan: fix smashpig exception for May not add a required argument after optional arguments already in the stack [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/955381 [16:50:39] 10Fundraising-Backlog: Edit formchooser to redirect Ingenico requests - https://phabricator.wikimedia.org/T345461 (10Dwisehaupt) [17:37:24] (03PS7) 10Ejegg: Disable Fiscal Number Field in Dlocal ZA [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/953594 (https://phabricator.wikimedia.org/T307743) (owner: 10Damilare Adedoyin) [17:37:32] (03CR) 10Ejegg: [C: 03+2] Disable Fiscal Number Field in Dlocal ZA [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/953594 (https://phabricator.wikimedia.org/T307743) (owner: 10Damilare Adedoyin) [17:45:11] (03Merged) 10jenkins-bot: Disable Fiscal Number Field in Dlocal ZA [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/953594 (https://phabricator.wikimedia.org/T307743) (owner: 10Damilare Adedoyin) [17:46:35] 10Fundraising Tech - Chaos Crew: RefundQueueConsumer error with MISSING_PREDECESSOR - https://phabricator.wikimedia.org/T345097 (10Ejegg) Are we successfully marking ANY Adyen refunds at this point? Are we potentially saving the wrong txn ID (auth vs capture)? Should we save both? [17:50:29] 10Fundraising Tech - Chaos Crew: RefundQueueConsumer error with MISSING_PREDECESSOR - https://phabricator.wikimedia.org/T345097 (10Cstone) Three were successful August 29 cid=47920404 [18:00:06] fr-tech I'll deploy donationinterface shortly if nobody has a reason i should wait [18:10:37] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/955824 [18:10:41] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/955824 (owner: 10Ejegg) [18:13:06] (03CR) 10Wfan: [C: 03+2] "LGTM" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/954141 (owner: 10Jgleeson) [18:18:35] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/955824 (owner: 10Ejegg) [18:27:13] 10Fundraising Tech - Chaos Crew: RefundQueueConsumer error with MISSING_PREDECESSOR - https://phabricator.wikimedia.org/T345097 (10Ejegg) The current top one in the damaged queue (ZHBRDHJSVFHSD272) is an iDEAL donation - we recorded the capture notification in payments-listener-adyen-20230818.gz . The donation i... [18:37:18] 10Fundraising Sprint Quit playing games with my Phab 🎶, 10Fundraising-Backlog, 10FR-tech-data-integrity: Disabling Gift Sources - https://phabricator.wikimedia.org/T343735 (10NNichols) @Damilare @Eileenmcnaughton I'm seeing the new Online Gift source coming through starting on the 5th. Can I go ahead and di... [19:15:16] 10Fundraising Sprint Quit playing games with my Phab 🎶, 10Fundraising-Backlog, 10Patch-For-Review: Change non campaign markets from Ingenico to Adyen - https://phabricator.wikimedia.org/T328964 (10EMartin) @Ejegg Adyen confirmed they can support ANY Visa and MC supported presentment currencies. [19:42:17] 10Fundraising-Backlog, 10FR-Adyen, 10FR-Smashpig: Clean up Adyen SmashPig code - https://phabricator.wikimedia.org/T345878 (10Ejegg) [20:20:56] 10Fundraising Sprint Quit playing games with my Phab 🎶, 10Fundraising-Backlog, 10FR-tech-data-integrity: Disabling Gift Sources - https://phabricator.wikimedia.org/T343735 (10Eileenmcnaughton) @NNichols - I guess it's a choice of 1) monitor for a few days / weeks & see if any Benevity / Community gifts come... [20:34:52] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Successful Adyen transactions not making it into CiviCRM - https://phabricator.wikimedia.org/T345736 (10jgleeson) Okay, after more digging, it appears that the mixup surrounding 191327632 is due to Adyen currently auto-capturing credit card transactions.... [21:35:37] (03PS5) 10Eileen: civicrm5.66-rc - stock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/952953 (https://phabricator.wikimedia.org/T344663) [21:35:43] (03PS4) 10Eileen: Do not update tax_amount to 0 - too slow.... [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/954989 (https://phabricator.wikimedia.org/T344663) [21:36:30] 10Fundraising-Backlog, 10fundraising-tech-ops: FR-Tech FY2324Q2 maintenance window - https://phabricator.wikimedia.org/T337579 (10Eileenmcnaughton) [21:56:56] (03CR) 10Ejegg: [C: 03+2] civicrm5.66-rc - stock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/952953 (https://phabricator.wikimedia.org/T344663) (owner: 10Eileen) [21:57:04] (03CR) 10Ejegg: [C: 03+2] Do not update tax_amount to 0 - too slow.... [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/954989 (https://phabricator.wikimedia.org/T344663) (owner: 10Eileen) [22:11:49] (03Merged) 10jenkins-bot: civicrm5.66-rc - stock [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/952953 (https://phabricator.wikimedia.org/T344663) (owner: 10Eileen) [22:12:09] (03Merged) 10jenkins-bot: Do not update tax_amount to 0 - too slow.... [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/954989 (https://phabricator.wikimedia.org/T344663) (owner: 10Eileen) [22:27:24] (03PS1) 10Ejegg: Consume de-concatenated c_t fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/955844 (https://phabricator.wikimedia.org/T334904) [22:27:51] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10MediaWiki-extensions-DonationInterface, 10Patch-For-Review: CT-to-Civi-Phase 2: send & consume de-concatenated fields - https://phabricator.wikimedia.org/T334904 (10AKanji-WMF) Moving to triage given testing requirement (always-on monthly convert) h... [22:41:19] (03PS1) 10Ejegg: Update DonationInterface submodule [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/955845 [22:41:23] (03CR) 10Ejegg: [C: 03+2] Update DonationInterface submodule [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/955845 (owner: 10Ejegg) [22:49:23] (03Merged) 10jenkins-bot: Update DonationInterface submodule [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/955845 (owner: 10Ejegg) [23:08:37] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/955846 [23:08:54] (03CR) 10Eileen: [C: 03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/955846 (owner: 10Eileen) [23:09:38] (03Merged) 10jenkins-bot: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/955846 (owner: 10Eileen) [23:12:38] !log payments-wiki upgraded from 639a8d6a to c524f53f [23:12:39] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [23:22:26] 10Fundraising Sprint Incantation optimisation, 10Fundraising Sprint Quit playing games with my Phab 🎶, 10Fundraising-Backlog, 10MW-1.39-notes (1.39.0-wmf.13; 2022-05-23), 10MW-1.41-notes (1.41.0-wmf.26; 2023-09-12): Disable ZA national ID field - https://phabricator.wikimedia.org/T307743 (10Ejegg) The DI... [23:22:38] 10fundraising-tech-ops: Migrate fr-tech-ops documentation from Collab Wiki to Wikitech - https://phabricator.wikimedia.org/T342658 (10Dwisehaupt) Updated FR_Tech_Managed_Accounts, I think it's ready for transfer. I added the grafana bits. Nessus_Exceptions is probably best in fundraising_system_documentation as... [23:28:31] (03PS3) 10Wfan: prefer adyen over ingenico for non-campain country and currencies [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/954983 (https://phabricator.wikimedia.org/T328964) [23:29:03] (03CR) 10Wfan: prefer adyen over ingenico for non-campain country and currencies (032 comments) [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/954983 (https://phabricator.wikimedia.org/T328964) (owner: 10Wfan) [23:45:47] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Successful Adyen transactions not making it into CiviCRM - https://phabricator.wikimedia.org/T345736 (10Cstone) Looks like same issue with the audits, for ones that auto settle the Modification Merchant Reference is instead of an actual transaction...