[07:32:15] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073081 (owner: 10L10n-bot) [07:37:47] (03CR) 10Raimond Spekking: [C:03+2] "false positive" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073081 (owner: 10L10n-bot) [08:19:45] 10fundraising-tech-ops, 06Infrastructure-Foundations, 10netops: Test prototype fundraising pybal replacement based on haproxy + anycast-healthchecker. - https://phabricator.wikimedia.org/T373942#10147625 (10ayounsi) Catching up on backlog, I think {T374803} is related. [13:20:34] 10fundraising-tech-ops, 06Infrastructure-Foundations, 10netops, 13Patch-For-Review: Test prototype fundraising pybal replacement based on haproxy + anycast-healthchecker. - https://phabricator.wikimedia.org/T373942#10148844 (10Jgreen) [13:26:37] (03CR) 10Damilare Adedoyin: [C:03+1] "Thanks cstone for putting up the patch, we only need to add a message for the "donate_interface-error-msg-street_number-in" key in extensi" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1072802 (https://phabricator.wikimedia.org/T374726) (owner: 10Cstone) [13:30:12] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 13Patch-For-Review: Update dLocal India form to require house number field - https://phabricator.wikimedia.org/T374726#10148854 (10Damilare) a:03Cstone [13:46:17] (03PS4) 10Jgleeson: WIP: Add Gravy Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1068813 [13:46:57] (03PS5) 10Jgleeson: WIP: Add Gravy Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1068813 [13:56:56] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Docker: Update Fundraising-dev to use syslog date formatting in log timestamps. - https://phabricator.wikimedia.org/T374843 (10jgleeson) 03NEW [14:23:02] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: codfw:frack:rack/install/configuration new switches - https://phabricator.wikimedia.org/T374587#10149067 (10ayounsi) Those won't be in a VC, especially as we didn't pay for the extra VC license :) This means a bit more... [16:13:43] it'd be a cool feature if searchkit could cache results and not make new queries on page load [16:14:31] maybe with a "last data fetch time:" info at the bottom too [16:14:44] sounds good [17:46:41] (03PS1) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T370583) [17:48:43] (03CR) 10CI reject: [V:04-1] Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T370583) (owner: 10Damilare Adedoyin) [17:50:06] (03PS6) 10Jgleeson: Add Gravy Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1068813 [17:51:55] (03PS2) 10Cstone: Make street_number required [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1072802 (https://phabricator.wikimedia.org/T374726) [17:52:07] damilare: ^ that adds in the missing translate msgs [17:53:38] (03PS2) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T370583) [17:54:19] cstone: thanks I just dropped another comment as I noticed we weren't sending the field in the api request [17:55:32] Ahh I think all this chaos is my fault as we weren't getting a response from dlocal with what was required and their API docs just showed a couple fields when I originally added it [17:55:41] (03CR) 10CI reject: [V:04-1] Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T370583) (owner: 10Damilare Adedoyin) [17:56:46] yh it keeps changing [17:59:04] (03PS1) 10Ebrahim: Replace uses of mw.Uri with browser native URL [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073271 (https://phabricator.wikimedia.org/T374314) [18:01:24] 10fundraising-tech-ops, 10decommission-hardware, 13Patch-For-Review: decommission frban2001.frack.codfw.wmnet - https://phabricator.wikimedia.org/T374741#10150409 (10Dwisehaupt) [18:02:51] 10fundraising-tech-ops, 06DC-Ops, 10decommission-hardware, 10ops-codfw, 13Patch-For-Review: decommission frban2001.frack.codfw.wmnet - https://phabricator.wikimedia.org/T374741#10150411 (10Dwisehaupt) a:05Dwisehaupt→03None [18:04:31] (03PS3) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T37058) [18:09:36] (03PS4) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T37058) [18:10:40] (03PS5) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T37058) [18:14:35] 10fundraising-tech-ops: install/configure frpm2002 - https://phabricator.wikimedia.org/T373629#10150455 (10Dwisehaupt) [18:22:43] (03PS1) 10Ebrahim: Replace uses of mw.Uri with browser native URL [extensions/FundraiserLandingPage] - 10https://gerrit.wikimedia.org/r/1073275 (https://phabricator.wikimedia.org/T374314) [18:31:06] 10fundraising-tech-ops: install/configure frpm2002 - https://phabricator.wikimedia.org/T373629#10150493 (10Dwisehaupt) [18:38:49] (03PS3) 10Cstone: Make street_number required [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1072802 (https://phabricator.wikimedia.org/T374726) [18:41:04] (03CR) 10CI reject: [V:04-1] Make street_number required [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1072802 (https://phabricator.wikimedia.org/T374726) (owner: 10Cstone) [18:41:56] (03PS4) 10Cstone: Make street_number required [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1072802 (https://phabricator.wikimedia.org/T374726) [18:42:32] hmm damilare with this validation is it coming from razorpay itself ? or is it dlocals api and just the fields arent required but if they are there all this validation is happening? [18:44:17] I think its dlocal cstone, at least that's what Rakhi said on the phab [18:44:37] does razorpay process upi too? [18:44:44] razorpay is the backend for upi [18:44:48] its what they switched too [18:46:03] ohh ok [18:46:42] looks like the fields are required from the errors [18:47:40] i was able to do test transactions with just 2 fields i guess what was confusing me but they might have changed their validation in the meantime too [18:47:50] (03CR) 10Damilare Adedoyin: [C:03+2] Make street_number required [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1072802 (https://phabricator.wikimedia.org/T374726) (owner: 10Cstone) [19:10:00] (03PS6) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T37058) [19:10:57] (03PS7) 10Jgleeson: Add Gravy Audit Processor [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1068813 (https://phabricator.wikimedia.org/T367788) [19:11:26] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Docker: Update Fundraising-dev to use syslog date formatting in log timestamps. - https://phabricator.wikimedia.org/T374843#10150596 (10jgleeson) [19:11:30] (03Merged) 10jenkins-bot: Make street_number required [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1072802 (https://phabricator.wikimedia.org/T374726) (owner: 10Cstone) [19:11:46] (03PS7) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T37058) [19:29:21] (03CR) 10Jgleeson: [C:04-1] "I like the general purpose config-driven approach. I left some comments about the default mins and a note on one of the transformer names." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T37058) (owner: 10Damilare Adedoyin) [19:47:04] 03Fundraising Sprint: reasonablyAccurate(), 06Fundraising-Backlog: Gravy Adyen test rollout plan - https://phabricator.wikimedia.org/T372469#10150722 (10XenoRyet) [19:48:11] 06Fundraising-Backlog: Migration of India recurring to an alternate dlocal processor - https://phabricator.wikimedia.org/T371723#10150731 (10XenoRyet) [20:02:10] 06Fundraising-Analysis, 06Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice: Archive banner impression stats on a rolling basis - https://phabricator.wikimedia.org/T373529#10150764 (10Damilare) [20:04:27] 06Fundraising-Backlog: Annual Recurring - Tagging and segmenting in Civi and Acoustic - https://phabricator.wikimedia.org/T372378#10150769 (10Damilare) [20:11:12] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 07Security: new wording for CN &banner= CSP error message - https://phabricator.wikimedia.org/T374625#10150773 (10AKanji-WMF) [20:11:46] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07payments-orchestration: Add backend processor to gravy refunds - https://phabricator.wikimedia.org/T374634#10150778 (10AKanji-WMF) [20:13:47] (03PS8) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T370583) [20:15:46] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: New Solicit Codes - https://phabricator.wikimedia.org/T374720#10150787 (10AKanji-WMF) Hi @NNichols can you please confirm when we'd want these options in Civi by? [20:20:21] 06Fundraising-Backlog: Recurring donors status id - 11 donors still using 80 - 95 - https://phabricator.wikimedia.org/T374755#10150815 (10AKanji-WMF) p:05Triage→03Low [20:20:51] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Recurring donors status id - 11 donors still using 80 - 95 - https://phabricator.wikimedia.org/T374755#10150816 (10AKanji-WMF) [20:24:11] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10fundraising-tech-ops, 10FR-Docker: Update Fundraising-dev to use syslog date formatting in log timestamps. - https://phabricator.wikimedia.org/T374843#10150820 (10AKanji-WMF) p:05Triage→03Low [20:25:50] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Add default record limit to refund scripts - https://phabricator.wikimedia.org/T374867#10150827 (10AKanji-WMF) p:05Triage→03Low [20:27:15] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gr4vy one-time donation in IE failed. - https://phabricator.wikimedia.org/T374527#10150831 (10AKanji-WMF) [20:28:53] 10fundraising-tech-ops, 06DC-Ops, 06Infrastructure-Foundations, 10netops, and 2 others: codfw:frack:rack/install/configuration new switches - https://phabricator.wikimedia.org/T374587#10150834 (10Papaul) [20:30:16] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy error: Invalid checkout session ID. Session does not exist, is incomplete or has expired. - https://phabricator.wikimedia.org/T374527#10150836 (10Cstone) [20:31:03] 06Fundraising-Backlog, 07payments-orchestration: Investigate application of Gravy guest checkout - https://phabricator.wikimedia.org/T374461#10150840 (10AKanji-WMF) p:05Triage→03Medium [20:37:30] 06Fundraising-Backlog, 07payments-orchestration: Display Gravy reconciliation ID in CiviCRM - https://phabricator.wikimedia.org/T374144#10150858 (10AKanji-WMF) Ideally this can be done during email blackout period we're proposing October 9-14 - in advance of our 1hr test [20:38:38] jgleeson: the validation figures on that phab were from Rakhi's comment here: https://phabricator.wikimedia.org/T370583#10129653 [20:38:48] patch * [20:39:44] I could also add it to the commit on that patch for proper description [20:40:21] (03PS9) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T370583) [20:43:19] (03PS10) 10Damilare Adedoyin: Add validation for dlocal fields: postal_code, street_number, and street_address [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T370583) [20:48:30] (03CR) 10Damilare Adedoyin: "Thanks for the CR jgleeson, I added the source of the validation values to the commit message and changed the class name to StreetNumberVa" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1073270 (https://phabricator.wikimedia.org/T370583) (owner: 10Damilare Adedoyin) [20:50:57] damilare: might be worth confirming that all India addresses have house numbers >= 6 digits in the database. 6 digits feels excessive e.g. 123 streetname wouldn't cut it [20:51:28] 66666 fleet street, would fail too [20:52:23] I'd be amazed if any street had house numbers in the 6 digits range. How big are the streets? [20:57:54] true, I'd confirm tomorrow [21:01:24] np [21:29:45] i'mma just take half a sick day [21:32:46] jgleeson|away: https://www.google.com/maps/place/986039+Perth-Oxford+Rd,+Tavistock,+ON+N0B+2R0/@43.3528248,-80.7410419,17z/data=!3m1!4b1!4m6!3m5!1s0x882c0143fbe8b0e9:0xdb28fbf826833a48!8m2!3d43.3528248!4d-80.738467!16s%2Fg%2F11c1334v5g?entry=ttu&g_ep=EgoyMDI0MDkxMS4wIKXMDSoASAFQAw%3D%3D [21:33:07] about a two hour drive away from me [21:44:46] hah anilk jgleeson|away places here love to start huge like its a 5 house street and all the numbers are like 15340 [21:58:59] adyen why wont you send me a recurring contract ipnnn [22:04:48] (03PS1) 10Cstone: Ignore expire [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1073293 [22:05:19] huh guess those expire messages were getting there first