[00:28:13] 10Fundraising Sprint Can't touch $this, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Fr-drupal-upgrade-2021: Swap all our process logging to Civi::log('wmf') from watchdog() - https://phabricator.wikimedia.org/T288585 (10Cstone) Yep they are showing up in the logs! [00:28:32] (03PS2) 10AndyRussG: [DRAFT FOR TESTING, PLS DO NOT MERGE] civi-jobs service [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/841441 (https://phabricator.wikimedia.org/T269708) [00:47:33] (03PS2) 10Wfan: Add process-control and fresh to docker [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/883524 (https://phabricator.wikimedia.org/T326974) [00:52:19] 10Fundraising Tech - Chaos Crew: Docker dev setup: Fundraising process-control cron - https://phabricator.wikimedia.org/T328506 (10AnnWF) [00:53:46] (03PS3) 10Wfan: Add process-control and fresh to docker [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/883524 (https://phabricator.wikimedia.org/T326974) [00:53:47] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Patch-For-Review: Bug: duplicate Adyen transactions in short sequence - https://phabricator.wikimedia.org/T324860 (10AnnWF) a:05AnnWF→03None [01:31:43] (03PS1) 10AndyRussG: Donut: Enable plain HTTP [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/885446 (https://phabricator.wikimedia.org/T326974) [01:32:09] wfan here's the fr-dev patch ^ [01:41:12] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-Docker, 10Patch-For-Review: Docker: make all MediaWiki services work with Fresh - https://phabricator.wikimedia.org/T326974 (10AndyRussG) This is working for Donut, with this patch: https://gerrit.wikimedia.org/r/885446. To run: - Download Fresh... [01:46:03] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 10JavaScript, and 2 others: flaky qunit test in CentralNotice: Uncaught TypeError: Cannot read property 'timeRemaining' of undefined - https://phabricator.wikimedia.org/T318103 (10AndyRussG) Fresh is now working o... [01:48:09] Nice, testing locally now [01:48:45] yayyyy hope it works! [01:49:03] (just gonna walk the dog now but I'll be back soon) [02:00:44] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-Docker, 10Patch-For-Review: Docker: make all MediaWiki services work with Fresh - https://phabricator.wikimedia.org/T326974 (10AnnWF) when we go inside donut with fresh-node, we can run curl http://localhost:9011/w/index.php?title=Special:JavaSc... [02:07:41] (03CR) 10Wfan: [V: 03+2 C: 03+2] "Works fine locally, thanks so much!" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/885446 (https://phabricator.wikimedia.org/T326974) (owner: 10AndyRussG) [02:10:53] (03CR) 10Ejegg: [C: 03+2] "recheck" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/885436 (owner: 10Ejegg) [02:18:45] (03Merged) 10jenkins-bot: Update SmashPig dependency and own version [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/885436 (owner: 10Ejegg) [02:24:44] (03PS1) 10Ejegg: Stop using @expectedException [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/885452 [02:26:54] (03CR) 10Ejegg: "Looks like ->expectException() handles the namespaced stuff better (and isn't deprecated like @expectExcpetion) - feel free to squash the " [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/801835 (owner: 10Eileen) [03:05:52] (03PS4) 10Ejegg: Make pending txn resolver work for PayPal [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/884149 (https://phabricator.wikimedia.org/T324642) [03:05:54] (03PS1) 10Ejegg: Suppress civicrm.wmf.WARNING: CVV score mismatch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/885454 [03:06:24] (03PS4) 10Ejegg: Check for success on getStatus call [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/883670 [03:21:34] (03CR) 10CI reject: [V: 04-1] Suppress civicrm.wmf.WARNING: CVV score mismatch [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/885454 (owner: 10Ejegg) [03:48:29] (03PS3) 10Ejegg: Allow info from status call to overwrite blank keys [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/884148 (https://phabricator.wikimedia.org/T324642) [03:48:31] (03PS5) 10Ejegg: Make pending txn resolver work for PayPal [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/884149 (https://phabricator.wikimedia.org/T324642) [03:50:42] AndyRussG: are you still planning to do the _phonetic name import patches for Civi, or can I pick that up? [03:51:41] oh hmm, it's later than I thought [03:51:49] well, I'm not going to pick it up tonight anyway! [03:58:15] ejegg yes :) [03:58:34] to both questions [06:25:45] (03PS1) 10Amire80: Fix comma to full stop in a message [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/885462 (https://phabricator.wikimedia.org/T322793) [11:43:48] (03CR) 10Jgleeson: [C: 04-1] Implement createHostedPayment for DLocal REDIRECT methods in SmashPig (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [12:22:30] (03PS15) 10Damilare Adedoyin: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) [12:22:58] (03CR) 10CI reject: [V: 04-1] Implement createHostedPayment for DLocal REDIRECT methods in SmashPig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [12:23:25] (03PS16) 10Damilare Adedoyin: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) [12:24:22] (03CR) 10Damilare Adedoyin: "good spot jgleeson, the test has been fixed. thanks" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [12:24:33] (03PS17) 10Damilare Adedoyin: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) [12:25:09] thanks damilare|away [12:49:51] (03PS5) 10Damilare Adedoyin: Pass through description for Dlocal authorize call [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/885052 (owner: 10Ejegg) [12:49:58] hey damilare so I can see you made some updates to that test [12:50:03] (03CR) 10Damilare Adedoyin: [C: 03+2] Pass through description for Dlocal authorize call [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/885052 (owner: 10Ejegg) [12:50:31] oh that's odd damilare I thought I'd +2ed that patch [12:50:41] and it was just waiting on your to be +2ed before it was merged [12:52:24] jgleeson: do you mean the one I just +2'd [12:52:28] yeah [12:53:02] i guess ejegg|away left it in his eod [12:54:53] ohh didn't see the +2 sorry and it had this merge conflict ish at the bottom. [12:55:09] ohh looks like ejegg|away added some changes after your CR jgleeson [12:56:18] oh I see. the fillNestedArray thing [12:56:43] yh [12:56:51] hmm I guess it was snagged so he didn't think it was worth adding a follow on patch to the already approved version [12:57:05] I think that would have been better a new patch explaining the change [12:57:09] as * [12:58:10] no that was something I worked on yesterday actually. [12:58:27] on the patch you're currently reviewinh [12:58:43] must have been a rebase [12:58:55] https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/885052/3..4/PaymentProviders/dlocal/Api.php [12:59:09] yep that's what I'm looking at rn [12:59:19] so ejegg|away didn't write that bit? [12:59:26] no he didn't [12:59:42] ok we probably should have not merged that as is [13:00:07] as you've effectively reviewed your own code I think [13:00:15] https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/883985/12..13 [13:00:16] if I understand it correctly [13:00:33] it isn't merged yet, no? [13:00:45] awaiting review on the parent [13:01:25] sorry I mean +2ed [13:01:40] if ejegg|away didn't add those changes in the patch, they shouldn't be in that patch right? [13:01:43] even with a rebase [13:02:28] well they affected the part he was modifying. So he probably had to pull down and resolve the merge conflict to do the rebase [13:03:07] ah ok so he did change some of it [13:03:21] if he resolved a conflict [13:03:32] yh [13:03:55] let's take the +2 off that patch [13:04:11] and it can be re-reviewed when https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/883985/17 is +2ed [13:04:26] so we don't have that again [13:05:21] ok, done [13:12:00] thanks [13:19:33] (03CR) 10Jgleeson: "Thanks Dami. Still a couple of issues with ApiTest" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [13:22:53] damilare: I'm gonna continue with the review of your patch so we can get it merged today. Could we skip our call and chat on IRC? I find it easier to review stuff off call [13:23:15] or if you had other stuff unrelated to that patch we could jump on the call [13:23:16] jgleeson, I had the name with Authorize earlier but it doesn't seem technically correct. This is because at the first payment call that yields a redirect url, the payment isn't authorized. Hence, the use of session felt appropriate [13:24:19] sure we could skip today's call. Was just going to ask about how you went about your last expense claim. Which we can sync on offline or later [13:25:13] ah so for that I completed as part of the 'Coupa Onboarding Training: Expense Claims' [13:25:31] here https://wikimedia.coupahost.com/expenses [13:26:34] ok tnx [13:27:48] jgleeson: did you mean the course in learnupon [13:27:59] yes that was the one [13:29:01] Ok I was wondering how you combined the accomodation and sprint fees. Did you fill seperate claims or all in one? [13:29:49] damilare: ok I see what you mean about the authorize name now. you're right this isn't the same [13:30:20] so this is specifically for non-cc pending payments right? [13:31:09] damilare: re: hotel and fees. I broke them out on to different lines [13:31:32] 09/12/22Training - Individual Staff DevelopmentTraining and Sprint event com [13:31:36] 09/12/22Training - Individual Staff DevelopmentHotel stay for CiviCRM training [13:31:45] 09/12/22Training - Individual Staff DevelopmentHotel stay for CiviCRM sprint [13:32:21] 10/05/22Outside USA Per Diemper diem dinner for 3/5 nights [13:32:43] that doesn't paste so well here [13:32:50] but those are the 4 line items I ended up with [13:33:32] thanks [13:34:20] re- your question about the REDIRECT payments, I'm not sure if it'd be specifically only for non-cc payments as the link actually allows cards too [13:34:30] here's a link I just generated: php PaymentProviders/dlocal/Maintenance/HealthCheck.php [13:34:41] https://sandbox.dlocal.com/collect/select_payment_method?id=M-8c93084e-805b-46b4-9628-6643465a001b&xtid=CATH-ST-1675258395-1616621231 [13:34:45] yeah I saw those last night [13:35:06] ok so i guess for now we just need to think of the "right" concept for the API [13:35:26] to me payment session feels too close to what we do for Adyen [13:36:42] what do you suggest we call it? [13:36:55] we're technically still creating payment aren't we're just using a multi-step flow [13:37:00] what do dlocal call it [13:37:05] let's check the docs [13:38:10] so i think yesterday ejegg|away mentioned he saw this flow we're looking at now sitting under the cash payment flow [13:38:31] albeit it's very confusing because we're not filtering payment methods on the redirect form to cash only [13:38:41] it looks like dlocal group them like this https://docs.dlocal.com/docs/overview-payins [13:38:49] cc, cash, bank transfer and wallet [13:39:24] so if we follow that grouping, this would be in the cash/bank/wallet space [13:40:07] hmm [13:40:22] so this isn't helped by the fact that we have direct and redirect cash payment flows [13:40:26] in the first example here: https://docs.dlocal.com/docs/card-payments [13:40:32] card has a redirect flow [13:41:21] I wonder if that is for 3dsecure [13:41:30] and they are not making that clear [13:41:50] or if it's just an example of how to get folks to enter card details on their form [13:42:19] so I think we already know that we're not going to be asking people to enter their card details on a dlocal form [13:42:21] yh it's most likely this ^ [13:42:38] has 3d secure's flow is DIRECT: https://docs.dlocal.com/docs/3d-secure [13:42:39] otherwise we'd be asking them to enter it twice, once on our form and the second on the dlocal [13:42:51] however it returns a return_url [13:43:20] the 3d secure link looks different anyway [13:43:35] I guess it's left to us to define our flow [13:44:11] since they're giving us multiple options for CARD, but a single flow for BANK/CASH/WALLET [13:44:58] hmm ok this is useful [13:45:15] I feel like we can assume that we won't be using REDIRECT for our intended card flow [13:45:26] especially if 3dsecure is DIRECT [13:45:27] yeah I agree [13:46:30] also wallet has two flow options, but I don't think we need to worry about this for now [13:48:07] ah [13:48:23] could we call the API method, redirectPayment() ? [13:48:41] how does that sound [13:49:19] authorize, capture (for cards) and redirectPayment (for non-cards that redirect) [13:49:39] sounds valid... [13:49:52] we'll then need a diretPayment call in the future for DIRECT cash payments [13:50:06] but at least the interface will tell us right away what the differences are [13:50:13] right [13:50:21] directPayment* [13:51:49] would we ever have a direct payment, given we wouldn't be loading the submethods? [13:52:02] for cash I mean [13:52:47] redirectPayment sounds good actually, embodies the expected flow [13:52:59] according to this we might https://phabricator.wikimedia.org/T324295 [13:53:12] although I can't think how we wouldn't redirected [13:53:17] ahh that's the problem [13:53:24] we have two concepts of redirect on our side [13:53:36] the 3dsecure redirect url from a DIRECT call [13:53:45] presumably the same for the DIRECT UPI cash payment too [13:54:12] and then the separate REDIRECT flow, which takes the donor away to a list of payment options [13:54:26] I think this is why it's confusing me [13:55:07] so we can be careful to make that distinction clear in the code and comments, to save others the pain [13:55:32] testing a direct cash flow locally now [13:55:36] in the doc here: https://docs.dlocal.com/docs/3d-secure [13:55:50] sorry: https://docs.dlocal.com/docs/cash-payments#example-request-1 [13:56:00] I don't see a redirect_url [13:57:17] hmm I wonder if that magic barcode handles the payment [13:58:36] hmmm [13:58:52] ok having read that page, I don't think we will display those barcodes on our side [13:59:11] I think that might be a PCI issue [13:59:31] actually PCI is payment cards [13:59:40] so maybe it's not an issue [14:00:01] hmm so how do we get redirected to that page [14:00:06] I'm curious how we'd present the final result on our side. my guess is that it would be easier to use the redirect [14:00:19] also my test payments are being rejected with details" rejected_other_reason [14:00:21] I clicked the link above https://docs.dlocal.com/docs/cash-payments#redirect-cash-payments [14:00:26] from your original link [14:01:16] I'm gonna continue with review on your patch. i wanna try and finish before I head off to pick up Oscr [14:01:23] Oscar* [14:02:00] yh I think that's what we'd have to do [14:02:37] that'd just skip the first stage of having multiple methods to the final payment form [14:02:56] sure np, I'd rename the method to redirectPayment in that case [14:04:20] ty! [14:09:32] (03CR) 10Jgleeson: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig (032 comments) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [14:09:43] just resolving those two rename points ^ [14:10:22] (03PS18) 10Damilare Adedoyin: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) [14:12:56] (03PS19) 10Damilare Adedoyin: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) [14:48:33] (03CR) 10Jgleeson: "Thanks Dami! These are my final comments after the latest round of review. Thanks for all the updates and work on this!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [14:48:53] only a few small things there damilare ^ [14:49:14] other than that it looks ready to rock! [14:49:19] back soon [14:52:57] (03CR) 10Jgleeson: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig (031 comment) [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [15:06:27] (03PS20) 10Damilare Adedoyin: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) [15:08:06] (03CR) 10Damilare Adedoyin: "thanks jgleeson, fixed!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [15:12:08] (03PS6) 10Damilare Adedoyin: Pass through description for Dlocal authorize call [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/885052 (owner: 10Ejegg) [15:23:12] 10Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for jbrungs - https://phabricator.wikimedia.org/T328382 (10Dwisehaupt) SSL certificate renewed and sent via email. Password sent via SMS. [15:27:36] 10Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for tgorski - https://phabricator.wikimedia.org/T328062 (10Dwisehaupt) SSL certificate renewed and sent via email. Password sent via SMS. [15:37:23] (03CR) 10Jgleeson: [C: 03+2] "Thanks Dami for all the work on this. Great job!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [15:37:55] (03Merged) 10jenkins-bot: Implement createHostedPayment for DLocal REDIRECT methods in SmashPig [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/883985 (https://phabricator.wikimedia.org/T324289) (owner: 10Damilare Adedoyin) [15:38:54] (03PS7) 10Jgleeson: Pass through description for Dlocal authorize call [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/885052 (owner: 10Ejegg) [15:39:39] damilare: ^ I just rebased that one over master as your one is now merged so feel free to reheck and re +2 [15:39:54] thanks jgleeson [15:40:28] (03CR) 10Damilare Adedoyin: [C: 03+2] Pass through description for Dlocal authorize call [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/885052 (owner: 10Ejegg) [15:40:59] (03Merged) 10jenkins-bot: Pass through description for Dlocal authorize call [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/885052 (owner: 10Ejegg) [15:41:14] I'm gonna break earlier than usual today as we've got a meeting-heavy evening coming up. back later [15:41:54] oh actually fr-tech there's a really interesting call happening today at the same time as standup [15:42:01] Ethical ML Salon - chatGPT + Wikimedia [15:43:19] so I was gonna ask if others are interested and if so could we move standup out a bit? or if no one else is interested, I could send a text standup, or join and just skip the other meeting [15:44:02] I know AndyRussG and a couple of other folks have expressed interest in the ML and ChatGPT chats [15:45:04] some interesting notes on this doc https://docs.google.com/document/d/1GB8PS26xJV2OR46UO5_6JyVLGh2HX_3l5VprBjXpvTQ/edit#heading=h.z6lpsr34inf8 about the estimated costs to run a ChatGPT-like service [15:45:10] back later! [15:58:09] thanks damilare [15:58:38] np ejegg. thanks for adding that [16:23:36] hiii fr-tech jgleeson|away ejegg damilare :) [16:24:05] ejegg: as noted above, I was indeed starting to work on the Civi phonetic name consumption [16:24:57] if you'd really like to do it, ofc you're welcome to, though your working on DjangoBS might be more inline with the FeatureChaos roles this sprint? [16:25:25] ah yeah, i can do the django chaos [16:25:44] please don't hesitate to ask if you want any pointers to mapping those custom fields [16:26:22] do you see in the code where we are already mapping a few custom fields from the messages? [16:26:30] ejegg: oki thanks so much for sure will do! [16:26:57] it feels like a good straightforward-ish task to get my brain back in that space :) [16:27:12] thx for understanding [16:41:55] ejegg: _wmf_civicrm_message_contribution_insert() in wmf_civicrm/wmf_civicrm.module ? [16:43:58] AndyRussG: that's the top-level function, yep [16:44:13] most of the guts have migrated into WmfContact\Save [16:44:26] in the wmf-civicrm civi extension [16:44:34] ah nice thanks [16:44:40] yeah just debugging thru [16:46:13] ejegg: you saw the nice Django patch from Jeff_Green? https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/tools/DjangoBannerStats/+/885435 [16:48:33] i did! [16:48:37] looking at it now [16:49:34] cool! [16:50:35] reminder of the testing Python debugging setup we have , in case you'd like to give it a spin: https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/dev/+/841441 [16:51:07] I will confess stopping on a Python breakpoint in silverpop export brought me joy [16:51:46] though only very fleeting heheh [16:51:57] thanks [16:55:58] hi AndyRussG [16:56:05] heyy jgleeson :) [16:56:17] what is Civi using Redis for besides the queues again? some kind of caching thing? [16:56:34] I see these GET and SETEX calls in the redis monitor [16:56:58] or are those due to queue consumption? [16:59:02] AndyRussG: yep, caching, exactly [17:00:42] hmmm so we should rename the queues service to redis under fr-dev? [17:01:00] oh probably [17:34:11] oops, net hiccup here [17:34:17] will restart the router [18:30:07] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: No entries in pgehres.bannerimpressions since 25 Jan - https://phabricator.wikimedia.org/T328460 (10Ejegg) Yes, the ancient code that populates that table doesn't run on the new OS. @Jgreen has a patch to modernize it a bit, tracked at T301905. I'm curren... [18:30:22] 10Fundraising-Backlog, 10FR-India, 10FR-dlocal, 10Recurring-Donations: Support India Network Tokenization (recurring credit cards) for DLocal - https://phabricator.wikimedia.org/T324304 (10jgleeson) dlocal mentioned that the recurring start date needs to be an Indian date set to India's time zone. [18:30:45] 10Fundraising Sprint Can't touch $this, 10Fundraising-Backlog, 10FR-India, 10FR-Smashpig, and 2 others: Implement backend for DLocal recurring UPI payments in SmashPig - https://phabricator.wikimedia.org/T324299 (10jgleeson) dlocal mentioned that the recurring start date needs to be an Indian date set to I... [18:31:06] 10Fundraising-Backlog, 10FR-India, 10FR-dlocal, 10MediaWiki-extensions-DonationInterface, 10Recurring-Donations: Support DLocal recurring UPI payments in DonationInterface - https://phabricator.wikimedia.org/T324300 (10jgleeson) dlocal mentioned that the recurring start date needs to be an Indian date se... [18:34:50] 10Fundraising Sprint Can't touch $this, 10Fundraising-Backlog, 10FR-India, 10FR-Smashpig, and 2 others: Implement backend for DLocal recurring UPI payments in SmashPig - https://phabricator.wikimedia.org/T324299 (10Ejegg) [18:35:32] 10Fundraising Sprint Can't touch $this, 10Fundraising-Backlog, 10FR-India, 10FR-Smashpig, and 2 others: Implement backend for DLocal recurring UPI payments in SmashPig - https://phabricator.wikimedia.org/T324299 (10Ejegg) [18:59:41] 10Fundraising-Backlog, 10FR-dlocal, 10MediaWiki-extensions-DonationInterface: Support DLocal recurring cards in DonationInterface - https://phabricator.wikimedia.org/T324297 (10Ejegg) [19:00:18] /ac/ac [19:01:24] 10Fundraising-Backlog, 10FR-dlocal, 10MediaWiki-extensions-DonationInterface: Support DLocal recurring cards in DonationInterface - https://phabricator.wikimedia.org/T324297 (10Ejegg) @EMartin any suggestions for the UI / messsaging when a Brazil donor who wants to make a recurring payment enters a debit car... [19:36:51] rebooting, brb [19:41:32] back [19:43:58] 10Fundraising-Backlog, 10FR-dlocal, 10MediaWiki-extensions-DonationInterface: Support DLocal recurring cards in DonationInterface - https://phabricator.wikimedia.org/T324297 (10EMartin) How about: Thank you for your one time donation. Due to bank regulations, recurring donations are only possible on cred... [19:56:41] fr-tech I'll be 5 min or so late to tech talk [20:22:14] PROBLEM - check_puppetrun on frdev1002 is CRITICAL: CRITICAL: Puppet has 1 failures. Last run 5 minutes ago with 1 failures. Failed resources (up to 3 shown): Package[pylint3-pylint-django] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdev1002&service=check_puppetrun [20:27:14] PROBLEM - check_puppetrun on frdev1002 is CRITICAL: CRITICAL: Puppet has 1 failures. Last run 3 minutes ago with 1 failures. Failed resources (up to 3 shown): Package[pylint3-pylint-django] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdev1002&service=check_puppetrun [20:32:15] RECOVERY - check_puppetrun on frdev1002 is OK: OK: Puppet is currently enabled, last run 50 seconds ago with 0 failures https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frdev1002&service=check_puppetrun [21:12:32] Jeff_Green: I'm getting issues running that updated djang code - even though I have python3-mysql installed I still get ModuleNotFoundError: No module named '_mysql_exceptions' [21:12:54] did you ever see that? [21:12:57] ejegg: I think that's part of python3-mysqldb [21:13:06] huh, I do have that installed [21:13:14] thinking [21:13:30] well, I'll poke around locally some more [21:13:31] it runs on frdev1002 so that apparently has all the things [21:13:36] ah thanks [21:14:51] I've gotta run to drive my daughter somewhere b/c her carpool partner just cancelled, but I'll be back and timeshifting starting in about an hour [21:15:41] ejegg: one thing to pay particular attention to--I haven't figured out how django caching works, and whether it is working as intented. it seem like we're missing config that would be necessary to declare where to cache. [21:15:53] "intended" [21:15:59] ok Jeff_Green [21:16:13] so we're not really using any part of django outside of the ORM [21:16:26] and I guess the command pattern from manage.py [21:16:51] the caching might only be relevant to the web layer, in which case it makes sense we wouldn't configure it [21:16:55] ok so I'll go back and rip out the caching stuff too [21:18:45] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Patch-For-Review: GatewayChooser: No supported gateway for parameters - https://phabricator.wikimedia.org/T326666 (10Pcoombe) @AnnWF Okay, think I've got this sorted and removed all the unsupported ones. We do have a special "PayPal USD" template for... [21:21:00] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Patch-For-Review: GatewayChooser: No supported gateway for parameters - https://phabricator.wikimedia.org/T326666 (10AnnWF) Sounds good, and I just use some random link which could lead to PayPal error now not shown from donate wiki, thanks! [21:22:00] * Jeff_Green driving daughter to another thing, back in about an hour and here/timeshifting afterward [21:28:47] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/885889 [21:28:56] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/885889 (owner: 10Ejegg) [21:29:32] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/885889 (owner: 10Ejegg) [21:39:27] (03PS1) 10Ejegg: Update vendor and composer.lock [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/885893 [21:39:52] (03CR) 10Ejegg: [C: 03+2] Update vendor and composer.lock [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/885893 (owner: 10Ejegg) [21:42:56] (03CR) 10CI reject: [V: 04-1] Update vendor and composer.lock [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/885893 (owner: 10Ejegg) [21:53:13] 10Fundraising Sprint Amazing grep, 10Fundraising Sprint Bridge over troubled Wifi, 10Fundraising Sprint Can't touch $this, 10Fundraising-Backlog, and 4 others: Update CI config for new fundraising/REL1_39 branch of mediawiki/core under php7.4 - https://phabricator.wikimedia.org/T326372 (10Ejegg) Thanks aga... [21:53:19] 10Fundraising Sprint Amazing grep, 10Fundraising Sprint Bridge over troubled Wifi, 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface, 10MW-1.40-notes (1.40.0-wmf.19; 2023-01-16): Make DonationInterface compatible with MediaWiki 1.39 - https://phabricator.wikimedia.org/T324154 (10Ejegg) [21:53:27] 10Fundraising Sprint Amazing grep, 10Fundraising Sprint Bridge over troubled Wifi, 10Fundraising Sprint Can't touch $this, 10Fundraising-Backlog, and 4 others: Update CI config for new fundraising/REL1_39 branch of mediawiki/core under php7.4 - https://phabricator.wikimedia.org/T326372 (10Ejegg) 05Open→... [21:55:32] (03PS2) 10Ejegg: Fix comma to full stop in a message [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/885462 (https://phabricator.wikimedia.org/T322793) (owner: 10Amire80) [21:55:37] (03CR) 10Ejegg: [C: 03+2] Fix comma to full stop in a message [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/885462 (https://phabricator.wikimedia.org/T322793) (owner: 10Amire80) [21:57:14] (03Merged) 10jenkins-bot: Fix comma to full stop in a message [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/885462 (https://phabricator.wikimedia.org/T322793) (owner: 10Amire80) [22:03:13] (03PS4) 10Ejegg: Add process-control and fresh to docker [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/883524 (https://phabricator.wikimedia.org/T326974) (owner: 10Wfan) [22:03:22] (03CR) 10Ejegg: [C: 03+2] "Thanks!" [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/883524 (https://phabricator.wikimedia.org/T326974) (owner: 10Wfan) [22:04:33] Thanks ejegg :) [22:07:14] :) [22:12:32] 10Fundraising-Backlog, 10fundraising-tech-ops: Issue new SSL Client Certificate for msuijkerbuijk - https://phabricator.wikimedia.org/T328611 (10Dwisehaupt) [22:49:43] (03PS1) 10Ejegg: Log rawResponse on recurring charge failure [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/885902 (https://phabricator.wikimedia.org/T239770)