[01:33:59] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Geocoding data does not appear to be updated correctly on merge - https://phabricator.wikimedia.org/T334151 (10Eileenmcnaughton) Just noting that stepping through the code for the other change this DOES appear to be happening - so we need to confirm that... [01:35:25] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Geocoding data does not appear to be updated correctly on merge - https://phabricator.wikimedia.org/T334151 (10Eileenmcnaughton) select log_action,log_date FROM civicrm.log_civicrm_address WHERE id IN (select id FROM civicrm.civicrm_address WHERE postal_c... [04:30:14] (03PS1) 10Eileen: Duck failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906800 (https://phabricator.wikimedia.org/T330231) [04:43:07] (03CR) 10CI reject: [V: 04-1] Duck failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906800 (https://phabricator.wikimedia.org/T330231) (owner: 10Eileen) [05:34:26] (03PS2) 10Eileen: Duck failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906800 (https://phabricator.wikimedia.org/T330231) [05:46:13] (03CR) 10CI reject: [V: 04-1] Duck failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906800 (https://phabricator.wikimedia.org/T330231) (owner: 10Eileen) [06:30:46] (03PS3) 10Eileen: Duck failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906800 (https://phabricator.wikimedia.org/T330231) [06:42:39] (03CR) 10CI reject: [V: 04-1] Duck failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906800 (https://phabricator.wikimedia.org/T330231) (owner: 10Eileen) [07:53:38] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907037 (owner: 10L10n-bot) [07:59:25] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907271 (owner: 10L10n-bot) [08:00:03] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/FundraisingEmailUnsubscribe] - 10https://gerrit.wikimedia.org/r/907295 (owner: 10L10n-bot) [08:05:18] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/907227 (owner: 10L10n-bot) [12:43:56] (03CR) 10Raimond Spekking: [V: 03+2] Localisation updates from https://translatewiki.net. [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/907227 (owner: 10L10n-bot) [12:57:27] (03CR) 10Raimond Spekking: [C: 03+1] "false positive" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907037 (owner: 10L10n-bot) [12:58:24] (03CR) 10Raimond Spekking: [C: 03+2] "false positive" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907037 (owner: 10L10n-bot) [13:11:53] (03Abandoned) 10Raimond Spekking: Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907271 (owner: 10L10n-bot) [14:38:04] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-eqiad: Q3:rack/setup/install frav1003 - https://phabricator.wikimedia.org/T334400 (10RobH) [14:38:10] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-eqiad: Q3:rack/setup/install frav1003 - https://phabricator.wikimedia.org/T334400 (10RobH) [15:44:20] those changes in the convert patch look good wfan, I'll just smoke test [15:50:58] 10Fundraising-Backlog, 10Diffusion-Repository-Administrators, 10Projects-Cleanup, 10Wikimedia-GitHub: Archive the FundraisingEmailUnsubscribe extension - https://phabricator.wikimedia.org/T224764 (10Ejegg) Yes @Amire80, we are unfortunately still using this old code in production, and sending the link to t... [15:54:02] 10fundraising-tech-ops: Deactivate fundraising accounts for tskaff - https://phabricator.wikimedia.org/T334404 (10Dwisehaupt) [15:56:24] 10fundraising-tech-ops: Deactivate fundraising accounts for tskaff - https://phabricator.wikimedia.org/T334404 (10Dwisehaupt) [15:57:55] 10fundraising-tech-ops: Deactivate fundraising accounts for sbenchagra - https://phabricator.wikimedia.org/T334406 (10Dwisehaupt) [15:58:35] 10Fundraising-Backlog, 10fr-donorservices: Donors seeing checkoutshopper-live-us.adyen.com error - https://phabricator.wikimedia.org/T334407 (10AMJohnson) [15:59:58] 10fundraising-tech-ops: Deactivate fundraising accounts for sbenchagra - https://phabricator.wikimedia.org/T334406 (10Dwisehaupt) [16:04:38] (03Abandoned) 10Raimond Spekking: Localisation updates from https://translatewiki.net. [extensions/FundraisingEmailUnsubscribe] - 10https://gerrit.wikimedia.org/r/907295 (owner: 10L10n-bot) [16:17:06] huh? I'm so confused. The ZA card form is showing me an ID field now [16:17:13] I swear it didn't have one last week [16:18:04] but it's right there in the country_fields [16:18:42] yeah ejegg [16:18:49] we put it back on friday sorry [16:18:53] oh i see [16:18:54] wfan: ^ [16:19:12] ok, let's put it back in the test links too then I guess [16:19:20] i think we did that too? [16:19:34] or maybe it was other links we were editing [16:19:36] haha yeah, we put it back too [16:19:50] oh ok, i just need to run the update script then [16:24:12] I think we'll want a country-specific validation message for that [16:24:19] 'fiscal number' is pretty opaque [16:24:27] anyway, MC works! [16:24:38] (03PS4) 10Ejegg: Enable monthly convert from DI for dlocal [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/905564 (https://phabricator.wikimedia.org/T324624) (owner: 10Wfan) [16:24:48] (03CR) 10Ejegg: [C: 03+2] "It works!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/905564 (https://phabricator.wikimedia.org/T324624) (owner: 10Wfan) [16:26:46] (03Merged) 10jenkins-bot: Enable monthly convert from DI for dlocal [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/905564 (https://phabricator.wikimedia.org/T324624) (owner: 10Wfan) [16:28:25] (03PS1) 10Ejegg: Add ZA message for invalid national ID [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907497 (https://phabricator.wikimedia.org/T324624) [16:28:56] thanks ejegg, I found some wired logic when test the MC for other currencies, that our country is getting form $donor['address_primary.country_id:abbr'], which is not good for dlocal recurring charge, since we needs correspond country, so I update it from the recurring mapper, while do we want to update the logic as save the country also besides currency to our recur table? [16:29:57] ah interesting [16:30:26] so we could add that to the custom fields table that is introduced here: https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/904385 [16:30:36] but that patch still needs someone to review it [16:36:49] (03PS1) 10Ejegg: Revert "Stop AstroPay failmail related to dLocal test IPNs" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907498 [16:37:07] cstone: we can revert that temp code now, right? ^^^ [16:37:38] i didnt trust it wouldnt happen again until we were fully live hah [16:38:02] ah ok [16:38:31] are there plans for the account switchover already? greg-g was asking in the other channel I wasn't up to date [16:38:37] but wouldn't we need a new patch? [16:38:42] oh I don't know [16:38:59] I guess if we're satisfied with the test result we should set a date for the switch [16:39:32] evelyn posted in the dlocal slack channel asking for real links which started the convo [16:39:38] also for UY, since our fiscal number is getting from $donor['legal_identifier'], mismatch also, so probably we would want to have default document for each currency if we do not want to store this part of info [16:49:41] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal: Dlocal prod account switcheroo - https://phabricator.wikimedia.org/T334414 (10Ejegg) [17:09:33] (03PS1) 10Wfan: Add mocked monthly convert amount suggestions to dlocal [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907500 (https://phabricator.wikimedia.org/T324624) [17:17:06] wfan hmm, actually I think it's probably unlikely that people will have multiple recurring donations from different countries with different tax ID numbers [17:17:31] well, real donors anyway [17:20:10] yeah, real donor is unlikely have multiple country and fiscal number, then I will just add the api map for dlocal to unblock the test only [17:21:02] wfan is there a ticket this is associated with? sorry, just catching up after Friday off and I'm not sure what is blocking which test [17:21:25] No ticket created, but I can create one~ [17:23:39] 10Fundraising-Backlog, 10Diffusion-Repository-Administrators, 10Projects-Cleanup, 10Wikimedia-GitHub: Archive the FundraisingEmailUnsubscribe extension - https://phabricator.wikimedia.org/T224764 (10Amire80) >>! In T224764#8768604, @Ejegg wrote: > Yes @Amire80, we are unfortunately still using this old cod... [17:25:00] 10Fundraising-Backlog, 10FR-dlocal: dlocal MC with default country and fiscal number not working for multiple currencies - https://phabricator.wikimedia.org/T334416 (10AnnWF) [17:26:16] (03PS1) 10Wfan: Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) [17:26:23] (03CR) 10CI reject: [V: 04-1] Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) (owner: 10Wfan) [17:26:36] 10Fundraising-Backlog, 10FR-dlocal, 10Patch-For-Review: dlocal MC with default country and fiscal number not working for multiple currencies - https://phabricator.wikimedia.org/T334416 (10AnnWF) a:03AnnWF [17:27:45] 10Fundraising-Backlog, 10FR-dlocal, 10Patch-For-Review: dlocal MC with default country and fiscal number not working for multiple currencies - https://phabricator.wikimedia.org/T334416 (10AnnWF) [17:27:47] thanks wfan! [17:27:50] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal, 10MediaWiki-extensions-DonationInterface, and 3 others: Enable Monthly Convert for DLocal in all markets that support recurring cards - https://phabricator.wikimedia.org/T324624 (10AnnWF) [17:28:20] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal: Dlocal prod account switcheroo - https://phabricator.wikimedia.org/T334414 (10Ejegg) [17:28:22] 10Fundraising-Backlog, 10FR-dlocal, 10Epic: Upgrade Dlocal to leverage Smartfields/no redirect - https://phabricator.wikimedia.org/T274300 (10Ejegg) [17:28:39] np :P [17:33:13] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal: Dlocal prod account switcheroo - https://phabricator.wikimedia.org/T334414 (10Ejegg) [17:34:31] (03PS2) 10Wfan: Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) [17:34:38] (03CR) 10CI reject: [V: 04-1] Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) (owner: 10Wfan) [17:36:32] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices: Adyen iframe config error messages Q2 2022: the sequel - https://phabricator.wikimedia.org/T324362 (10Cstone) This one has been open for a while and had multiple child tickets that were solved, I'm going to close it as we have deploy... [17:37:04] (03PS3) 10Wfan: Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) [17:37:22] PROBLEM - check_mysql on frdb2003 is CRITICAL: Slave IO: Yes Slave SQL: No Seconds Behind Master: (null) https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb2003&service=check_mysql [17:37:31] (03CR) 10jenkins-bot: Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) (owner: 10Wfan) [17:39:19] (03PS4) 10Wfan: Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) [17:39:26] (03CR) 10CI reject: [V: 04-1] Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) (owner: 10Wfan) [17:40:41] (03PS2) 10Wfan: Add mocked monthly convert amount suggestions to dlocal [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907500 (https://phabricator.wikimedia.org/T324624) [17:42:22] PROBLEM - check_mysql on frdb2003 is CRITICAL: Slave IO: Yes Slave SQL: No Seconds Behind Master: (null) https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb2003&service=check_mysql [17:43:34] (03PS5) 10Wfan: Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) [17:43:41] (03CR) 10CI reject: [V: 04-1] Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) (owner: 10Wfan) [17:46:03] (03CR) 10Wfan: unblock breadcrumb with style update from tivy.css (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906639 (owner: 10Wfan) [17:47:20] (03PS6) 10Wfan: Add default currency and fiscal number for dlocal MC [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907502 (https://phabricator.wikimedia.org/T334416) [17:47:22] PROBLEM - check_mysql on frdb2003 is CRITICAL: Slave IO: Yes Slave SQL: No Seconds Behind Master: (null) https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb2003&service=check_mysql [17:49:26] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal, 10Patch-For-Review: dlocal MC with default country and fiscal number not working for multiple currencies - https://phabricator.wikimedia.org/T334416 (10AnnWF) [17:52:06] ACKNOWLEDGEMENT - check_mysql on frdb2003 is CRITICAL: Slave IO: Yes Slave SQL: No Seconds Behind Master: (null) Dwisehaupt known issue - working on clearing up the intent of the replication issue https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb2003&service=check_mysql [17:53:49] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal: Ensure fiscal number for India (PAN) is not stored - https://phabricator.wikimedia.org/T333322 (10AnnWF) [18:20:08] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: UZS Donation Form Error - https://phabricator.wikimedia.org/T332945 (10Cstone) I think based on what you talked about @Pcoombe in {T326666} then we should switch the UZ form to show the not fundraising in your country message and remove from Ways to Give. [18:22:22] RECOVERY - check_mysql on frdb2003 is OK: Uptime: 4123426 Threads: 5 Questions: 1392225830 Slow queries: 1681 Opens: 19240 Open tables: 1694 Queries per second avg: 337.638 Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 0 https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdb2003&service=check_mysql [18:34:30] (03PS1) 10Ejegg: WIP pare down dlocal response factory code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907512 (https://phabricator.wikimedia.org/T333725) [18:34:37] (03CR) 10CI reject: [V: 04-1] WIP pare down dlocal response factory code [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907512 (https://phabricator.wikimedia.org/T333725) (owner: 10Ejegg) [18:36:02] (03CR) 10Ejegg: "damilare, do you want to take a look at the WIP patch I sent up on top of this one to see if you think it would be a good way to cut down " [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/905258 (https://phabricator.wikimedia.org/T333725) (owner: 10Damilare Adedoyin) [18:44:09] 10Fundraising-Backlog: Reenable AVS fraud score by 17 April 2023 - https://phabricator.wikimedia.org/T334422 (10EMartin) [19:07:53] 10Fundraising Sprint Fish HEAD^, 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-Smashpig, and 2 others: Update DLocal audit parsing to tag donations as 'dlocal' rather than 'astropay' when appropriate - https://phabricator.wikimedia.org/T324292 (10Ejegg) Since it seems like there is no d... [19:08:07] 10Fundraising Sprint Fish HEAD^, 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-Smashpig, and 2 others: Update DLocal audit parsing to tag donations as 'dlocal' rather than 'astropay' when appropriate - https://phabricator.wikimedia.org/T324292 (10Ejegg) a:03Ejegg [19:12:58] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal: Ensure fiscal number for India (PAN) is not stored - https://phabricator.wikimedia.org/T333322 (10AnnWF) Is that ok for us to use default fiscal number for local recurring enabled countries? as this patch did? https://gerrit.wikimedia.... [19:16:23] (03PS1) 10Ejegg: Fix UtcDate FIXME, add return type [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907516 [19:16:25] (03PS1) 10Ejegg: Set dlocal/astropay gateway per cutoff date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907517 (https://phabricator.wikimedia.org/T324292) [19:16:35] (03CR) 10CI reject: [V: 04-1] Set dlocal/astropay gateway per cutoff date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907517 (https://phabricator.wikimedia.org/T324292) (owner: 10Ejegg) [19:16:51] (03CR) 10CI reject: [V: 04-1] Fix UtcDate FIXME, add return type [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907516 (owner: 10Ejegg) [19:17:00] (03PS1) 10Ejegg: Fix bad constant in PayPal error mapper [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907518 [19:17:07] (03CR) 10CI reject: [V: 04-1] Fix bad constant in PayPal error mapper [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907518 (owner: 10Ejegg) [19:17:26] (03PS2) 10Ejegg: Set dlocal/astropay gateway per cutoff date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907517 (https://phabricator.wikimedia.org/T324292) [19:17:34] (03CR) 10CI reject: [V: 04-1] Set dlocal/astropay gateway per cutoff date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907517 (https://phabricator.wikimedia.org/T324292) (owner: 10Ejegg) [19:17:45] (03PS2) 10Ejegg: Fix UtcDate FIXME, add return type [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907516 [19:17:52] (03CR) 10CI reject: [V: 04-1] Fix UtcDate FIXME, add return type [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907516 (owner: 10Ejegg) [19:18:31] weird [19:18:36] I just rebased those over master [19:18:51] and they're all getting 'This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.' still [19:20:20] looks like releng is on it: https://phabricator.wikimedia.org/T334426 [19:25:54] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: GatewayChooser: No supported gateway for parameters - https://phabricator.wikimedia.org/T326666 (10Cstone) @Pcoombe XK (Kosovo) is still creating this error with Paypal. I looked into it and none of our payment processors support it. We don't have Kosovo... [20:09:26] (03PS15) 10Eileen: CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/899187 (https://phabricator.wikimedia.org/T303986) [20:33:14] 10Fundraising-Backlog: Complete technical work to enable Venmo through Braintree - https://phabricator.wikimedia.org/T334430 (10EMartin) [20:35:10] (03PS16) 10Eileen: CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/899187 (https://phabricator.wikimedia.org/T303986) [20:36:14] ahh, FundraiserMaintenance is not overridden per-gateway [20:36:49] OK, I guess I just set $wgAstroPayGatewayEnabled to false [20:40:17] (03PS8) 10Eileen: Adapt to change in form property on upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904049 [20:40:19] (03PS12) 10Eileen: Enable Civi-import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901739 [20:40:21] (03PS17) 10Eileen: Add hooks to import to do organization lookup from nick-name [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901734 (https://phabricator.wikimedia.org/T303986) [20:40:23] (03PS1) 10Eileen: Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) [20:47:45] (03CR) 10CI reject: [V: 04-1] CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/899187 (https://phabricator.wikimedia.org/T303986) (owner: 10Eileen) [20:47:52] !logo turned of AstroPay APIv1 payments forms [20:47:56] *off [20:49:19] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-dlocal: Dlocal prod account switcheroo - https://phabricator.wikimedia.org/T334414 (10Ejegg) @EMartin I have just sent the email and switched off the old APIv1 forms. As soon as we hear back, I will update the new forms to use the production a... [20:51:21] (03CR) 10CI reject: [V: 04-1] Enable Civi-import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901739 (owner: 10Eileen) [20:51:29] (03CR) 10CI reject: [V: 04-1] Adapt to change in form property on upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904049 (owner: 10Eileen) [20:51:57] (03CR) 10CI reject: [V: 04-1] Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) (owner: 10Eileen) [20:52:03] (03CR) 10CI reject: [V: 04-1] Add hooks to import to do organization lookup from nick-name [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901734 (https://phabricator.wikimedia.org/T303986) (owner: 10Eileen) [20:52:42] hmm - latest civi-tar-ball has an error cos CiviMember is not enabled - won't be hard to fix [20:55:07] (03PS2) 10Eileen: Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) [20:55:09] (03PS9) 10Eileen: Adapt to change in form property on upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904049 [20:55:11] (03PS13) 10Eileen: Enable Civi-import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901739 [20:55:13] (03PS18) 10Eileen: Add hooks to import to do organization lookup from nick-name [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901734 (https://phabricator.wikimedia.org/T303986) [20:55:15] (03PS1) 10Eileen: CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907528 (https://phabricator.wikimedia.org/T303986) [20:58:33] (03PS1) 10Ejegg: Switch AstroPay to Dlocal in GatewayChooser [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907529 (https://phabricator.wikimedia.org/T334414) [21:00:10] (03CR) 10CI reject: [V: 04-1] Switch AstroPay to Dlocal in GatewayChooser [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907529 (https://phabricator.wikimedia.org/T334414) (owner: 10Ejegg) [21:02:01] 10fundraising-tech-ops: Deactivate fundraising accounts for sbenchagra - https://phabricator.wikimedia.org/T334406 (10Dwisehaupt) [21:03:15] 10fundraising-tech-ops: Deactivate fundraising accounts for sbenchagra - https://phabricator.wikimedia.org/T334406 (10Dwisehaupt) 05Open→03Resolved a:03Dwisehaupt ssl certificate revoked and changes pushed out. Superset account marked inactive and removed access from archive folder. No other data or accounts. [21:06:30] (03CR) 10CI reject: [V: 04-1] Adapt to change in form property on upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904049 (owner: 10Eileen) [21:06:36] (03CR) 10CI reject: [V: 04-1] Enable Civi-import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901739 (owner: 10Eileen) [21:06:40] (03CR) 10CI reject: [V: 04-1] Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) (owner: 10Eileen) [21:06:57] (03CR) 10CI reject: [V: 04-1] Add hooks to import to do organization lookup from nick-name [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901734 (https://phabricator.wikimedia.org/T303986) (owner: 10Eileen) [21:07:23] (03CR) 10CI reject: [V: 04-1] CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907528 (https://phabricator.wikimedia.org/T303986) (owner: 10Eileen) [21:12:02] (03PS2) 10Ejegg: Switch AstroPay to Dlocal in GatewayChooser [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/907529 (https://phabricator.wikimedia.org/T334414) [21:12:13] (03PS17) 10Eileen: CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/899187 (https://phabricator.wikimedia.org/T303986) [21:14:11] (03Abandoned) 10Eileen: CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907528 (https://phabricator.wikimedia.org/T303986) (owner: 10Eileen) [21:23:52] (03CR) 10CI reject: [V: 04-1] CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/899187 (https://phabricator.wikimedia.org/T303986) (owner: 10Eileen) [21:25:23] (03PS18) 10Eileen: CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/899187 (https://phabricator.wikimedia.org/T303986) [21:38:18] 10fundraising-tech-ops: Deactivate fundraising accounts for tskaff - https://phabricator.wikimedia.org/T334404 (10Dwisehaupt) [21:42:49] 10fundraising-tech-ops: Deactivate fundraising accounts for tskaff - https://phabricator.wikimedia.org/T334404 (10Dwisehaupt) [21:47:33] wfan: just looking at your breadcrump patch - what does it mean that it is blocked? Where can I see that? [21:50:09] civi user page Eileen [21:50:27] I mean actually all pages [21:51:18] 10fundraising-tech-ops: Deactivate fundraising accounts for tskaff - https://phabricator.wikimedia.org/T334404 (10Dwisehaupt) [21:51:25] hmm - https://usercontent.irccloud-cdn.com/file/LTp7Fo64/image.png [21:51:31] so that is what I see [21:51:42] Oh it’s for civicrm [21:52:28] ah https://usercontent.irccloud-cdn.com/file/CZFX3wC9/image.png [21:52:51] like this https://civicrm.wikimedia.org/civicrm/contact/view?reset=1&cid=54243389 [21:53:00] right~ [21:53:07] oh - gotcha [21:53:12] (03CR) 10Eileen: [C: 03+2] unblock breadcrumb with style update from tivy.css [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/906639 (owner: 10Wfan) [21:53:21] 10fundraising-tech-ops: Deactivate fundraising accounts for tskaff - https://phabricator.wikimedia.org/T334404 (10Dwisehaupt) a:03Dwisehaupt SSL certificate and ssh key revoked. mysql account marked for removal, grants updated, and changes pushed out. Civi and superset accounts moved to blocked/inactive. Remov... [21:53:24] thanks :) [21:54:18] (03PS19) 10Eileen: CiviCRM 5.61 rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/899187 (https://phabricator.wikimedia.org/T303986) [21:54:20] (03PS3) 10Eileen: Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) [21:54:22] (03PS10) 10Eileen: Adapt to change in form property on upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904049 [21:54:24] (03PS14) 10Eileen: Enable Civi-import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901739 [21:54:26] (03PS19) 10Eileen: Add hooks to import to do organization lookup from nick-name [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901734 (https://phabricator.wikimedia.org/T303986) [21:56:39] (03CR) 10Ejegg: "recheck" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907517 (https://phabricator.wikimedia.org/T324292) (owner: 10Ejegg) [21:57:16] (03CR) 10CI reject: [V: 04-1] Set dlocal/astropay gateway per cutoff date [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/907517 (https://phabricator.wikimedia.org/T324292) (owner: 10Ejegg) [21:58:40] (03PS1) 10Eileen: E-notice fix [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907540 [21:59:24] oh hey that 5.61 rc path is passing tests [21:59:35] I've just upgraded locally and am clicking around [22:00:13] queue consumers are running fine [22:02:48] (03Abandoned) 10Eileen: Merging upstrea [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904949 (owner: 10Eileen) [22:04:18] (03PS7) 10Eileen: Add hook support to look up individual and handle soft credits during import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904380 (https://phabricator.wikimedia.org/T303986) [22:04:20] (03PS3) 10Eileen: Remove custom code that is obsolete [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904948 (https://phabricator.wikimedia.org/T332357) [22:05:18] (03PS3) 10Eileen: Fix to match individual on import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904954 [22:05:42] (03CR) 10CI reject: [V: 04-1] Adapt to change in form property on upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904049 (owner: 10Eileen) [22:05:48] (03CR) 10CI reject: [V: 04-1] Add hooks to import to do organization lookup from nick-name [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901734 (https://phabricator.wikimedia.org/T303986) (owner: 10Eileen) [22:05:50] (03CR) 10CI reject: [V: 04-1] Enable Civi-import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/901739 (owner: 10Eileen) [22:06:02] (03CR) 10CI reject: [V: 04-1] Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) (owner: 10Eileen) [22:08:26] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-Smashpig, 10FR-dlocal: Handle Refund/Chargeback IPNS for Dlocal - https://phabricator.wikimedia.org/T330723 (10Ejegg) [22:08:53] 10Fundraising Sprint Go Your Own Pay, 10Fundraising-Backlog, 10FR-Smashpig, 10FR-dlocal: Handle Refund/Chargeback IPNS for Dlocal - https://phabricator.wikimedia.org/T330723 (10Ejegg) [22:09:22] (03CR) 10CI reject: [V: 04-1] E-notice fix [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907540 (owner: 10Eileen) [22:15:38] (03CR) 10CI reject: [V: 04-1] Add hook support to look up individual and handle soft credits during import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904380 (https://phabricator.wikimedia.org/T303986) (owner: 10Eileen) [22:15:56] (03CR) 10CI reject: [V: 04-1] Fix to match individual on import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904954 (owner: 10Eileen) [22:16:54] (03CR) 10CI reject: [V: 04-1] Remove custom code that is obsolete [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904948 (https://phabricator.wikimedia.org/T332357) (owner: 10Eileen) [22:24:54] (03PS4) 10Eileen: Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) [22:32:46] ejegg: cool - just working on getting the ones over it to run clean :-) [22:33:35] our tests run in this kinda no-user mode which surfaced a couple of permissions things [22:36:25] (03CR) 10CI reject: [V: 04-1] Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) (owner: 10Eileen) [22:45:33] (03PS5) 10Eileen: Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) [22:48:16] (03PS6) 10Eileen: Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) [22:59:34] (03CR) 10CI reject: [V: 04-1] Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) (owner: 10Eileen) [23:12:59] (03CR) 10Eileen: Add new custom field for initial scheme txn ID (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904385 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [23:13:44] ejegg: I can give your extra field a spin - but I haven't been part of the discussion about adding it - I assume the tires are kicked on it as an approach / where to store the data? [23:23:41] (03PS4) 10Eileen: Add new custom field for initial scheme txn ID [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904385 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [23:35:17] (03CR) 10CI reject: [V: 04-1] Add new custom field for initial scheme txn ID [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904385 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [23:39:11] (03PS5) 10Eileen: Add new custom field for initial scheme txn ID [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904385 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [23:40:07] ejegg: we had probably better get that upgrade patch merged - at a pinch the one I'm fighting can wait a bit but there is one other 'must do at some time' one [23:46:35] (03CR) 10Eileen: [C: 03+2] Use API4 for inserting ContributionRecur records [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/905752 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [23:49:19] (03CR) 10Eileen: "I just have some reservations aboout using mixed case in the names - I feel like it would be less error prone if we used all lower on the " [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/905768 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [23:49:35] (03CR) 10Eileen: "in the name field I mean - label is good with mixed case" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/905768 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [23:51:25] (03PS11) 10Eileen: Adapt to change in form property on upgrade [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904049 [23:51:57] ok - so in addition to the main upgrade patch - the other one we really need ready to go in ... erm ... 9 minutes is https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/904049 [23:53:15] ejegg: cstone dwisehaupt wfan - is someone OK to merge the main upgrade patch so we can roll it out when the outage starts... https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/899187 [23:54:12] (03PS7) 10Eileen: Additional merge patch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/907526 (https://phabricator.wikimedia.org/T332357) [23:54:46] (03CR) 10Eileen: "argh - I pushed my change into this when testing it - revert revert" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/904385 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [23:55:06] looking. [23:55:31] i can too [23:55:36] thanks [23:56:19] not confidence to do so haha, seems lots files~ how to test it? [23:56:25] ejegg: did say he gave it a click-around - this is needed to prevent us trying to access a changed property https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/904049 [23:56:29] I think ejegg already tested it? [23:56:49] wfan: yeah - there is only so much we can do with these upstream updates - in terms of testing [23:57:03] apart from check the site loads.... [23:58:15] ejegg: said earlier on irc he gave it a click-around [23:58:32] i looked at 904049 and that looks sane to me. [23:58:41] cool [23:58:51] ejegg: from earlier [23:58:51] "oh hey that 5.61 rc path is passing tests [23:58:51] 9:59 AM [23:58:51] I've just upgraded locally and am clicking around [23:58:52] 10:00 AM [23:58:52] queue consumers are running fine" [23:59:30] i don't have a way to test it but looks good. [23:59:37] yeah im okay with +2ing it [23:59:40] dwisehaupt: well that's a start :-) [23:59:54] cool if you +2 can you try to click submit before it takes away the v+2