[02:33:04] (03PS1) 10Ejegg: Handle new link messages with email or id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135147 (https://phabricator.wikimedia.org/T388739) [02:34:05] (03PS5) 10Ejegg: Add NoticeHideUrls CentralNotice setting [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1128510 [02:34:05] (03PS5) 10Ejegg: Update donatewiki content dump [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1130777 [02:34:05] (03PS5) 10Ejegg: Fix hiding login link on email prefs page [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1131122 [02:34:06] (03PS9) 10Ejegg: Add CiviCRM standalone composer build [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1133552 (owner: 10Jgleeson) [02:34:07] (03PS1) 10Ejegg: Consume new-checksum-link queue [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1135148 (https://phabricator.wikimedia.org/T388739) [02:35:39] (03PS2) 10Ejegg: Switch new checksum link to require email [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) [02:38:15] (03CR) 10CI reject: [V:04-1] Switch new checksum link to require email [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [04:32:06] (03PS3) 10Cstone: WIP: Add example messages to a queue [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1114411 [04:32:32] (03CR) 10CI reject: [V:04-1] WIP: Add example messages to a queue [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1114411 (owner: 10Cstone) [04:47:35] (03PS1) 10Cstone: Update references to the new script name [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1135152 [05:25:20] (03PS1) 10Cstone: WIP: Also send recurring failure email on autorescue failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1135155 (https://phabricator.wikimedia.org/T386733) [05:28:25] (03PS4) 10Cstone: WIP: Add example messages to a queue [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1114411 [05:28:51] (03CR) 10CI reject: [V:04-1] WIP: Add example messages to a queue [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1114411 (owner: 10Cstone) [05:57:01] (03CR) 10Abijeet Patro: [V:03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_43) - 10https://gerrit.wikimedia.org/r/1134832 (owner: 10L10n-bot) [06:09:27] 06Fundraising-Backlog, 10fundraising-tech-ops: Update dlocal sftp keypair to ed25519 - https://phabricator.wikimedia.org/T386985#10724600 (10RKumar_WMF) Sent follow up email to dLocal. [06:38:10] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_41) - 10https://gerrit.wikimedia.org/r/1135197 (owner: 10L10n-bot) [10:20:47] 10fundraising-tech-ops: Q1:rack/setup/install frban1002 - https://phabricator.wikimedia.org/T369947#10725324 (10MatthewVernon) [10:23:57] 10fundraising-tech-ops: Q1:rack/setup/install fran1002 - https://phabricator.wikimedia.org/T369940#10725328 (10MatthewVernon) [10:24:14] 10fundraising-tech-ops: Q1:rack/setup/install frdb1007 - https://phabricator.wikimedia.org/T369922#10725333 (10MatthewVernon) [10:24:24] 10fundraising-tech-ops: Q1:rack/setup/install franio100[1-3] - https://phabricator.wikimedia.org/T367820#10725334 (10MatthewVernon) [10:24:36] 10fundraising-tech-ops: Q1:rack/setup/install fransc1001 - https://phabricator.wikimedia.org/T367814#10725335 (10MatthewVernon) [10:24:49] 10fundraising-tech-ops: Q1:rack/setup/install fransw1001.frack.eqiad.wmnet - https://phabricator.wikimedia.org/T367801#10725336 (10MatthewVernon) [13:31:39] 06Fundraising Tech Chaos Holding Pen, 06Fundraising-Backlog, 10FR-donorservices, 07payments-orchestration: Gr4vy audit file edge cases - https://phabricator.wikimedia.org/T381012#10726085 (10AMJohnson) Thank you @jgleeson! I've let that donor know that the donation failed if they'd like to retry. [13:36:22] 06Fundraising Tech Chaos Holding Pen, 06Fundraising-Backlog, 10FR-donorservices, 07payments-orchestration: Gr4vy audit file edge cases - https://phabricator.wikimedia.org/T381012#10726095 (10AMJohnson) Thanks @jgleeson! Just to confirm before I reply to the donor and invite them to retry -- The transaction... [13:43:33] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog, 13Patch-For-Review: Come up with a better name for PaymentDetailResponse in SmashPig - https://phabricator.wikimedia.org/T383400#10726113 (10jgleeson) [15:10:00] 06Fundraising Tech Chaos Holding Pen, 06Fundraising-Backlog, 10FR-donorservices, 07payments-orchestration: Gr4vy audit file edge cases - https://phabricator.wikimedia.org/T381012#10726505 (10jgleeson) So I tested out the Gravy payment sync API call on this latest case, but unfortunately, it did not work, a... [15:12:36] (03PS3) 10Ejegg: Switch new checksum link to require email [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1131125 (https://phabricator.wikimedia.org/T388739) [16:00:32] ok, that new checksum link stuff is ready for review [16:03:40] awesome! thanks ejegg and we are in the standup meeting [16:23:46] (03CR) 10Jgleeson: [V:03+2 C:03+2] Update references to the new script name [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1135152 (owner: 10Cstone) [16:24:26] (03CR) 10Wfan: [C:03+2] Update references to the new script name [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1135152 (owner: 10Cstone) [16:27:21] 06Fundraising-Backlog, 10LPL Onboarding and Development, 10MediaWiki-extensions-DonationInterface, 07affects-translatewiki.net, and 2 others: Remove " 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog, 07payments-orchestration: Connection to Gr4vy's consolidated output file into Civi - https://phabricator.wikimedia.org/T379837#10726809 (10EMartin) 05Open→03Resolved a:03EMartin [16:57:44] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog, 07payments-orchestration: Gravy dLocal Internal Test Plan - https://phabricator.wikimedia.org/T389547#10726969 (10AnnWF) [17:00:04] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog, 07payments-orchestration: Gravy dLocal Internal Test Plan - https://phabricator.wikimedia.org/T389547#10726983 (10AnnWF) Added all the connection in gr4vy prod config, and links for cc and non-card payment, https://docs.google.com/spreadsheets/d/1... [17:15:22] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10MediaWiki-extensions-CentralNotice, 10Temporary accounts (Major pilot wiki deployment): [Temporary Accounts] Update CentralNotice extension to support Temporary Accounts - https://phabricator.wikimedia.org/T374437#10727058 (10AKanji-WMF) Hi @Oreoluwa -... [17:20:33] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog, 07payments-orchestration: How will we expose payment processor errors/issues with gravy - https://phabricator.wikimedia.org/T387411#10727075 (10Cstone) Here's a list of gravy error codes: https://docs.gr4vy.com/guides/api/resources/transactions/er... [18:53:44] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog, 07payments-orchestration: Connection to Gr4vy's consolidated output file into Civi - https://phabricator.wikimedia.org/T379837#10727376 (10greg) I believe this is superseded by {T391496} (created today). [18:58:02] 03Fundraising Sprint: Grits & Catfish, 06Fundraising-Backlog, 07payments-orchestration: Connection to Gr4vy's consolidated output file into Civi - https://phabricator.wikimedia.org/T379837#10727390 (10EMartin) Yes, I resolved the prior case as this comes more into focus. [19:10:11] (03PS4) 10Wfan: Fix submethod country lists [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135134 (https://phabricator.wikimedia.org/T391412) (owner: 10Ejegg) [19:10:55] (03PS5) 10Wfan: Fix sub-method country lists [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135134 (https://phabricator.wikimedia.org/T391412) (owner: 10Ejegg) [19:11:54] 06Fundraising-Backlog: Setting up consolidated settlement reporting for all PSPs via Gr4vy for new recon process - https://phabricator.wikimedia.org/T391496#10727430 (10Cstone) We configured these columns on our end for our audits, the payment accounting report and the settlement detail report are the ones the a... [20:30:36] 06Fundraising-Backlog: Switch off CVV fraud filter for non-cc payment methods - https://phabricator.wikimedia.org/T391390#10727673 (10jgleeson) [20:38:03] (03PS1) 10Jgleeson: Move payment method constants to PaymentMethod class [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135489 (https://phabricator.wikimedia.org/T391390) [20:38:04] (03PS1) 10Jgleeson: Set a default cvv score of '0' for non-cc payment methods [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135490 (https://phabricator.wikimedia.org/T391390) [20:39:18] wfan: those are the patches to skip CVV for gravy non-cc payments [20:40:34] hmm ok so setting it to 0 treats it as unavailable ha [20:40:40] (03CR) 10CI reject: [V:04-1] Set a default cvv score of '0' for non-cc payment methods [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135490 (https://phabricator.wikimedia.org/T391390) (owner: 10Jgleeson) [20:40:45] which then leads to '2025-04-09T20:39:50+00:00 547c6764e585 SmashPig: gravy::376:376.3 | CVV result 'unavailable' adds risk score 55. | | [20:40:45] ' [20:40:46] (03CR) 10CI reject: [V:04-1] Move payment method constants to PaymentMethod class [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135489 (https://phabricator.wikimedia.org/T391390) (owner: 10Jgleeson) [20:46:07] (03PS2) 10Jgleeson: Move payment method constants to PaymentMethod class [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135489 (https://phabricator.wikimedia.org/T391390) [20:46:07] (03PS2) 10Jgleeson: WIP: Skip cvv score for non-cc payment methods. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135490 (https://phabricator.wikimedia.org/T391390) [20:49:00] argh [20:49:09] I don't think this is going to work [20:49:20] we need to switch it off entirely [20:49:37] otherwise with the current workaround, we'd need to mark is as a 'match' to get 0 which would suck [20:49:58] so I think it'll be better to just remove the getCVV filter at the root [20:50:15] I'm out of time for tonight but will check in tomorrow! [20:51:31] (03CR) 10Jgleeson: [V:03+2 C:03+2] Add NoticeHideUrls CentralNotice setting [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1128510 (owner: 10Ejegg) [20:51:40] (03CR) 10Jgleeson: [C:03+2] Update donatewiki content dump [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1130777 (owner: 10Ejegg) [20:51:45] (03CR) 10Jgleeson: [V:03+2 C:03+2] Update donatewiki content dump [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1130777 (owner: 10Ejegg) [20:52:01] (03CR) 10Jgleeson: [C:03+2] Fix hiding login link on email prefs page [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1131122 (owner: 10Ejegg) [20:52:05] (03CR) 10Jgleeson: [V:03+2 C:03+2] Fix hiding login link on email prefs page [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1131122 (owner: 10Ejegg) [20:53:41] dwisehaupt / Jeff_Green I'm going to deploy the change that updates the exchange rates lookup URL from a path on www.oanda.com to exchange-rates-api.oanda.com [20:54:12] locally those both resolve to the same IPs [20:54:19] (03CR) 10Jgleeson: [V:03+2 C:03+2] "I think we can +2 this one as it works out of the box without customisation. We're not going to use it for our WMFF standalone build as we" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1133552 (owner: 10Jgleeson) [20:54:22] so I'm hoping we won't get any FW errors [20:54:45] (03CR) 10Jgleeson: [V:03+2 C:03+2] Consume new-checksum-link queue [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1135148 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [20:55:59] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1135496 [20:56:07] thanks for all the merges jgleeson ! [20:57:36] (03CR) 10Jgleeson: [V:03+2 C:03+2] "civicrm.wmf.INFO: Successfully processed 2 from queue new-checksum-link" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1135148 (https://phabricator.wikimedia.org/T388739) (owner: 10Ejegg) [20:57:45] np! [21:00:05] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1135496 (owner: 10Ejegg) [21:00:58] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1135496 (owner: 10Ejegg) [21:01:45] jgleeson: I think that 0 in the runFraudFilters fn is a real true 0 [21:02:08] all the mapping from gravy codes to scores should take place down in the smashpig layer before we get here [21:02:43] (03CR) 10Jgleeson: [C:04-1] "Argh. Let's abort the current approach. It looks like returning 0 treats this as unavailable which results in:" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135490 (https://phabricator.wikimedia.org/T391390) (owner: 10Jgleeson) [21:02:59] oh ejegg [21:03:06] when testing with 0, I got 025-04-09T20:39:50+00:00 547c6764e585 SmashPig: gravy::376:376.3 | CVV result 'unavailable' adds risk score 55. | | [21:03:15] maybe I messed something up [21:04:05] or do yo mean int vs string? [21:04:09] you* [21:04:20] I've gotta drop but I'll check in tomorrow. thanks! [21:08:01] oh weird [21:08:59] ahh right, in the logs we'll always see the smashpig-level cvv mapping [21:09:42] but then we'd override it with the score set in addResponseData, which is actully used to calculate the go/no-go [21:11:49] !log fundraising civicrm upgraded from b20436a2 to 38a7a649 [21:11:50] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [21:56:27] (03CR) 10Wfan: [C:03+2] Indicate non-recurring dlocal methods [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135129 (https://phabricator.wikimedia.org/T391412) (owner: 10Ejegg) [22:00:02] (03Merged) 10jenkins-bot: Indicate non-recurring dlocal methods [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135129 (https://phabricator.wikimedia.org/T391412) (owner: 10Ejegg) [23:25:03] (03CR) 10Wfan: [C:03+2] Check for submethod-level recurring config in chooser [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135131 (https://phabricator.wikimedia.org/T391412) (owner: 10Ejegg) [23:26:30] 06Fundraising-Backlog, 10MW-1.44-notes (1.44.0-wmf.25; 2025-04-15), 13Patch-For-Review: Bad UI from payments form when recurring=1 for non recurring methods - https://phabricator.wikimedia.org/T391412#10728225 (10AnnWF) in additional for update the gateway choose, should figure out a way to render error mess... [23:27:40] (03Merged) 10jenkins-bot: Check for submethod-level recurring config in chooser [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1135131 (https://phabricator.wikimedia.org/T391412) (owner: 10Ejegg)