[00:00:24] i think its when you make a paypal transaaction and its like want to donate to X too? [00:00:35] im not sure how it comes into us though like through a spreadsheet or from paypal itself [00:02:04] looks like from paypal [00:02:38] im guessing the name is different and thats why i wasnt deduped lemme dig up the logs [00:02:56] oh i take it back theres two for april [00:03:34] ok nevermind looks like sometimes theres two [00:13:13] so coming in via audit [00:13:25] (By the way there was a audit one in what I deployed before) [00:23:56] eileen: ah that paypal giving thing has come up before [00:24:11] I think there was one specific email address they used to use [00:24:31] oh wait, no, the one I'm thinking of was a different paypal money steering vehicle [00:24:44] yeah the name is diffferent on the cids but the email is the same [00:25:26] Paypal Grants I guess need to stay out of civi; https://phabricator.wikimedia.org/T349392 [00:26:39] i think it just needs to be deduped and the name updated [00:26:45] ahh cstone so it's the same hw_payout_omnibus? [00:27:09] OK, it would be better to have the name different [00:27:46] huh, 'No possible duplicates were found using Email is enuf rule.' [00:27:50] (03PS1) 10Eileen: Port s://github.com/civicrm/civicrm-core/pull/30082 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025489 (https://phabricator.wikimedia.org/T360139) [00:30:54] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 13Patch-For-Review: Civi batch data entry tool build out - https://phabricator.wikimedia.org/T360139#9756020 (10Eileenmcnaughton) This turns out to be an easy fix - https://github.com/civicrm/civicrm-core/pull/30082 @MDemosWMF try ag... [00:31:05] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 13Patch-For-Review: Civi batch data entry tool build out - https://phabricator.wikimedia.org/T360139#9756021 (10Eileenmcnaughton) a:03Eileenmcnaughton [00:31:15] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 13Patch-For-Review: Civi batch data entry tool build out - https://phabricator.wikimedia.org/T360139#9756022 (10Eileenmcnaughton) [00:32:52] 14Fundraising Sprint: didAnyoneTryThis(), 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 13Patch-For-Review: Consolidate rest of normalize into WMFMessage class - https://phabricator.wikimedia.org/T357471#9756025 (10Eileenmcnaughton) this one... [00:33:42] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 07Unplanned-Sprint-Work: Mapping Import Project - Engage import - https://phabricator.wikimedia.org/T360150#9756033 (10Eileenmcnaughton) @MDemosWMF the fix on thank you is now deployed on prod [00:39:29] (03PS1) 10Eileen: I am an IDE-pleaser [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025494 [00:49:11] ah i see, i was deduping the wrong one [00:49:39] so are those some kind of soft credit? [01:03:01] 06Fundraising-Backlog, 10FR-email-preferences, 10MediaWiki-extensions-DonationInterface: Update text on email preferences success page - https://phabricator.wikimedia.org/T363717#9756053 (10Ejegg) @MSuijkerbuijk_WMF and @AKanji-WMF : sorry, this might not actually need a change! This was prompted by @AKanji... [01:23:18] Are they from Melanie's imports? [01:23:57] I did ping Melanie to that effect (she is out) but I didn't want to sound too dumb on the actual phab [01:35:59] sorry went to eat dinner mid investigation but I think they are from ipns it says they are from smashpig [01:37:25] although the ids are weird [01:38:23] txn_type send_money [01:39:02] i wonder if we should have a list of magic emails in config to dedupe ignoring name? [01:39:48] (03PS1) 10Eileen: Override core mailing action urls with ones to our preference centre [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025498 (https://phabricator.wikimedia.org/T361630) [01:39:52] is that some weird like 3rd paypal thats not old or ec or this a old type one? [01:40:25] Maybe we should leave this one until we can check in with Melanie [01:40:36] on the happy side I think I have a fix for anilk [01:42:42] cstone we have send_money mapped to a payment in paypal/main.yaml https://phabricator.wikimedia.org/diffusion/WFSP/browse/master/config/paypal/main.yaml$113 [01:42:58] yeah i was looking at that but its on the same level as express_checkout [01:43:16] or do express checkouts come in with a different mapping? [01:43:26] ah yeah, i'm not sure [01:43:42] it's definitely not initiated via the express_checkout flow [01:44:13] im trying to hunt down its source maybe the audit did hand it off to the job runner and it came in that way [01:44:29] oh not an IPN? [01:44:41] havent found a log of it yet [01:45:10] but may be grep failing [01:48:22] failing at finding the id from the audit too haha does an import kick things over to the job runner too? [01:48:37] i don't think so [01:50:08] maybe its in the log rotation netherland? [01:50:16] netherworld? [01:50:20] weird, yeah, I can't find it either [01:50:28] ok hah makes me feel better if you cant too [01:50:42] from the civi UI I see enqueued at timestamp: Apr 29, 2024 14:22 [01:50:58] (03CR) 10CI reject: [V:04-1] Override core mailing action urls with ones to our preference centre [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025498 (https://phabricator.wikimedia.org/T361630) (owner: 10Eileen) [01:51:41] ahhhh [01:51:53] it's the date +1 thing [01:51:56] zgrep 8WP94555HU2635933 payments-listener-paypal-20240430.gz [01:52:00] in the futureeee [01:52:01] haha [01:52:23] ok sooooo [01:52:32] I guess we should normalize this at the incoming side? [01:52:34] was it ipn then? [01:52:39] yep it was IPN [01:52:53] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 07Unplanned-Sprint-Work: Matching EFT import error - but not duplicate - https://phabricator.wikimedia.org/T358143#9756125 (10Eileenmcnaughton) @MDemosWMF I think maybe mapping the gateway and letting it use the incoming transaction... [01:53:10] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 07Unplanned-Sprint-Work: Matching EFT import error - but not duplicate - https://phabricator.wikimedia.org/T358143#9756126 (10Eileenmcnaughton) [01:53:19] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 07Unplanned-Sprint-Work: Matching EFT import error - but not duplicate - https://phabricator.wikimedia.org/T358143#9756127 (10Eileenmcnaughton) a:03Eileenmcnaughton [01:53:41] so rather than making weird exceptions in the queue consumers I'd rather have a list of email addresses at the job runner [01:53:56] that get their name stripped out and maybe have the type changed to organization [01:54:09] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog: CiviMail Unsubscribe Footer Broken - https://phabricator.wikimedia.org/T361630#9756129 (10Eileenmcnaughton) a:03Eileenmcnaughton [01:54:21] i actually see THREE records in Civi with that email, 1 org and 2 individuals [01:55:01] ok that new name is the current CFO of paypal giving haha [01:55:07] ah man [01:55:24] ejegg: we don't need to touch the contact type if mapping to existing contacts - I was gonna tidy that up a little anyway [01:55:24] but has been CFO for 3 years soo [01:55:56] lol it's like our tle account - every account is a personal account! [01:56:00] rip [01:56:37] (03PS2) 10Eileen: Override core mailing action urls with ones to our preference centre [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025498 (https://phabricator.wikimedia.org/T361630) [01:57:00] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog: CiviMail Unsubscribe Footer Broken - https://phabricator.wikimedia.org/T361630#9756130 (10Eileenmcnaughton) [01:57:22] eileen do you know offhand if we send a contact in with no name and just an email that matches an existing, will it merge cleanly? [01:57:34] or will it create a new Anonymous Anonymous with that email? [01:58:02] if (empty($msg['first_name']) || empty($msg['last_name'])) { [01:58:02] return NULL; [01:58:02] } [01:58:27] so it will create a new contact - although I think we now do blank first name & last name [01:59:13] I'm really trying to wade through that new vs update stuff at the moment but it's hard to make sense of - I have some patches open towards that goal [01:59:52] ok so we need to give the job-runner (which reads the raw IPN and sends the message to civi) a map of email => org name [02:00:22] if you are gonna do that you might as well go as far as contact ID [02:00:39] hmm, I guess [02:01:39] previous name is their COO [02:02:06] cstone there's also the org record: https://civicrm.wikimedia.org/civicrm/contact/view?reset=1&cid=3729480 [02:02:28] that one probbably makes the most sense? [02:04:28] hmm cid=59606568 is another send_money [02:04:46] 06Fundraising-Backlog: Paypal Giving Fund donations added to Civi under incorrect CID - https://phabricator.wikimedia.org/T363768#9756144 (10Ejegg) @EMartin they also have an organization record, cid 3729480. Would it be better to add some logic to make sure all donations with that email go to the org record? [02:05:30] i dont even know whats going on with this record is this soft credits? [02:06:10] cstone we don't seem to get individual IPNs for each individual donation, do we? [02:06:25] I think it's just all given as a lump under their fund name [02:06:28] well for the send_money ones its just these big chunks [02:06:38] maybe I dont remember what a soft credit is [02:06:52] theres like 4 different companies listed in that cid I just linked [02:06:59] er in the ipns [02:07:09] Vanguard Charitable Endowment Program has sent you money [02:07:11] oh sorry i didn't see that other cid [02:07:20] Groundswell Charitable Foundation has sent you money [02:07:21] ahh yeah I think that one IS soft credits [02:07:40] there was some ticket for that [02:08:11] and we had to ignore one copy of those donations? [02:12:17] 06Fundraising-Backlog: Paypal Giving Fund donations added to Civi under incorrect CID - https://phabricator.wikimedia.org/T363768#9756149 (10Ejegg) Suggested tech fix: give IPN job runner a map of email address => contact_id for special cases like this one, and when the IPN's email is on the list have the runner... [02:13:51] ok, I need to hang it up [02:18:39] how have i broken just payments wiki haha [02:18:46] locally [02:27:07] there is special handling for Vanguard Charitable Endowment in ChecksFile -but I've long suspected it is unreachable [02:34:57] is it tied to the email address? [02:35:33] (03PS2) 10Eileen: Update OptInQueueConsuer to use a Message class, apiv4 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025472 (https://phabricator.wikimedia.org/T357471) [02:35:49] nope // Expand soft credit short names. [02:35:49] if (!empty($msg['soft_credit_to'])) { [02:35:49] $nickname_mapping = [ [02:35:49] 'Fidelity' => 'Fidelity Charitable Gift Fund', [02:35:49] 'Vanguard' => 'Vanguard Charitable Endowment Program', [02:35:50] 'Schwab' => 'Schwab Charitable Fund', [02:35:50] ]; [02:35:51] if (array_key_exists($msg['soft_credit_to'], $nickname_mapping)) { [02:35:51] $msg['soft_credit_to'] = $nickname_mapping[$msg['soft_credit_to']]; [02:35:52] } [02:35:52] } [02:36:23] ah okay the names [02:38:59] fix payments break civi cant win [02:41:03] fix civi break payments what is going onnn haha [02:42:10] has to be my docker override [02:45:12] cstone: you just have to prioritise... [02:54:04] haha [02:54:18] maybe try 4 will work [02:54:36] (03CR) 10CI reject: [V:04-1] Update OptInQueueConsuer to use a Message class, apiv4 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025472 (https://phabricator.wikimedia.org/T357471) (owner: 10Eileen) [02:54:41] ok nope [02:54:44] now it makes less sense [02:55:22] deleting everything [03:24:00] eileen: are you still around? [03:24:07] cstone: yep [03:24:25] i have a tiny DI patch if you could +2 ideal is currently broken on prod I think lemme push it up [03:24:59] ok [03:25:41] (03PS1) 10Cstone: Turn of monthly convert flag for rtbt [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1025501 [03:26:42] cstone: obviously I don't have much back ground here - if you are confident I will +2 on that [03:26:55] yeah its missing some other code I think [03:26:58] ok [03:27:01] (03CR) 10Eileen: [C:03+2] Turn of monthly convert flag for rtbt [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1025501 (owner: 10Cstone) [03:27:05] thanks! [03:27:42] ACH doesnt have a redirect but ideal does [03:29:17] (03Merged) 10jenkins-bot: Turn of monthly convert flag for rtbt [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1025501 (owner: 10Cstone) [03:32:06] hmm live transactions are coming in somehow still so I guess thats good but it wouldnt work locally [03:34:42] oh - does that mean we should not deploy it [03:35:10] i dont know I cant test that payment method on live I think I'll just leave it not deployed until tomorrow [03:35:30] and we can get someone to test [03:47:25] cool [03:48:48] (03PS7) 10Jgleeson: Add SEPA js [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1009950 (https://phabricator.wikimedia.org/T324520) (owner: 10Wfan) [03:51:07] (03CR) 10Cstone: [C:03+2] I am an IDE-pleaser [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025494 (owner: 10Eileen) [03:52:56] (03PS8) 10Jgleeson: Delay sending recurring SEPA payments to donations queue. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1020868 (https://phabricator.wikimedia.org/T324520) [04:01:54] writing a end of day cause i want to people to read it and its already way too long for anyone to reasonably read [04:04:14] im giving up for the night have a good rest of your day eileen [04:04:31] lol - yeah I'm writing my EOD too now [04:12:32] (03Merged) 10jenkins-bot: I am an IDE-pleaser [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025494 (owner: 10Eileen) [04:30:14] (03PS2) 10Eileen: Port https://github.com/civicrm/civicrm-core/pull/30082 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025489 (https://phabricator.wikimedia.org/T360139) [04:34:19] (03PS3) 10Eileen: Update OptInQueueConsuer to use a Message class, apiv4 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025472 (https://phabricator.wikimedia.org/T357471) [04:45:17] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 13Patch-For-Review: Enable 90 day snooze: Comms Preferences Centre update - https://phabricator.wikimedia.org/T358878#9756281 (10MSuijkerbuijk_WMF) @AKanji-WMF happy to meet to review these. [11:26:26] 06Fundraising-Backlog, 07Epic: Rename utm_ URL parameters through donatewiki and paymentswiki - https://phabricator.wikimedia.org/T351325#9757248 (10Pcoombe) @Ejegg There is some special-casing on payments forms which changes the logo and some of the link targets when [utm_medium=endowment](https://payments.wi... [13:05:14] (03CR) 10Jgleeson: [C:03+2] "re +2ing after fixing merge conflict" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1009950 (https://phabricator.wikimedia.org/T324520) (owner: 10Wfan) [13:07:20] (03Merged) 10jenkins-bot: Add SEPA js [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1009950 (https://phabricator.wikimedia.org/T324520) (owner: 10Wfan) [13:26:05] cstone: hmm, we're getting more errors with autorescues unable to find their rescue reference [13:26:17] could something have changed to stop us recording those correctly? [13:31:51] (03PS1) 10Ejegg: Update endowment hook to respect wmf_medium [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1025779 (https://phabricator.wikimedia.org/T351325) [13:32:09] tiny patch for the utm_ -> wmf_ switch fr-tech ^^^^ [13:43:14] (03CR) 10Jgleeson: [C:03+2] "lgtm!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1025779 (https://phabricator.wikimedia.org/T351325) (owner: 10Ejegg) [14:02:23] (03Merged) 10jenkins-bot: Update endowment hook to respect wmf_medium [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1025779 (https://phabricator.wikimedia.org/T351325) (owner: 10Ejegg) [14:10:57] thanks jgleeson|skrun ! [14:23:39] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1025794 [14:23:59] fr-tech any objection to deploying ^^^ ? [14:25:10] It's got the 'load SEPA js' patch but not the one to delay the queue message [14:43:57] Were you able to treat it ejegg it was breaking for me last night but a lot of other things were also breaking [14:47:11] Test it * [14:55:35] 06Fundraising-Backlog: Paypal Giving Fund donations added to Civi under incorrect CID - https://phabricator.wikimedia.org/T363768#9758123 (10MDemosWMF) @Ejegg @Cstone Thanks for that information! I wonder why it was set up that way in the first place? Having the gifts go to the organization record makes more sen... [15:12:28] oh it seemed to work fine for me last night cstone [15:12:30] trying again [15:13:24] you were using data from https://docs.adyen.com/development-resources/testing/test-other-payment-methods/ right? [15:14:02] it's getting me to the TY page just fine locally [15:14:09] was it breaking anything else? [15:14:40] not seeing any errors in payments-syslog [15:16:05] it was when getting it into civi [15:16:15] ahhh let me try that [15:17:14] Failure while processing message: INVALID_MESSAGE No payment type found for message. [15:17:26] OK, let's add a quick Civi patch for that [15:17:43] but I think it's OK to go out to payments if we're not steering anyone to it yet, right? [15:17:54] ah yeah ejegg [15:23:59] (03PS1) 10Ejegg: Add SEPA Direct Debit payment instrument [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025801 [15:24:15] lemme just try that locally ^^ [15:28:38] hmm, still failing [15:28:43] let's see what I missed [15:29:20] oh derp [15:29:28] needed to be a key => value [15:29:57] (03PS2) 10Ejegg: Add SEPA Direct Debit payment instrument [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025801 [15:40:28] cstone: is there anything obviously wrong with that as far as you can see? ^^^ [15:40:50] I may have messed something up locally by applying the broken earlier version [15:41:14] but I can't seem to make it re-create the correct SEPA payment_instument option value [15:42:42] sorry ejegg I was half here when responding yes to we can deploy DI since no one is using sepa right now and lemme try that patch [15:43:03] hi [15:43:12] hi jgleeson ! [15:43:17] what was the error you ran into? [15:43:26] (03CR) 10Ejegg: [C:03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1025794 (owner: 10Ejegg) [15:43:27] do we want this change here https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1010157/2/PaymentProviders/Adyen/ReferenceData.php [15:43:48] missing payment type jgleeson when importing into civi [15:44:12] did you have this checked out? https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/1010156 [15:44:37] ejegg: ^ [15:44:56] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/1025794 (owner: 10Ejegg) [15:45:01] ohhhh i hadn't seen that jgleeson [15:45:35] jgleeson: any reason to keep that in WIP? [15:45:52] Nice to have the instrument option even if the charging needs work, right? [15:46:06] will abandon the other one [15:46:18] I guess not now. that was the point it was broken so I marked it as that to prevent it merging [15:46:24] (03PS3) 10Jgleeson: Add SEPA as new instrument [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1010156 (https://phabricator.wikimedia.org/T324520) (owner: 10Wfan) [15:46:30] (03Abandoned) 10Ejegg: Add SEPA Direct Debit payment instrument [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1025801 (owner: 10Ejegg) [15:46:46] chckout the review comments [15:48:02] I'll +2 that one now [15:48:13] as the problem was unrelated to that patch [15:48:16] although it did highlight it [15:49:19] cool cool. I'll see if that one works for me to create the method [15:49:28] or if i jsut need a rebuild :P [15:49:37] I this you need to run the sync [15:49:59] yeah, i ran it but I had a messed-up version of the payment instrument before that I deleted through the UI [15:49:59] it's in the commit [15:50:07] ah [15:50:17] and it seems to be someplace stubborn in the caches [15:50:24] even system.flush doesn't let me recreate it [15:50:28] ehh [15:50:34] s/this/think/ [15:51:30] slightly distracted. someone just nearly back over me and Oscar in an electric car and when I pulled him back the driver said... "did you not hear the car?" [15:51:35] backed [15:51:51] ah damn [15:51:55] what a jerk [15:52:03] no, surprisingly. yeah [15:52:25] I replied with "do you not use your mirrors" [15:52:31] but yeah not great [15:59:05] ejegg: I think that DI patch can go out. I can push out the recurring fix once that's +2ed [16:08:04] (03Merged) 10jenkins-bot: Add SEPA as new instrument [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1010156 (https://phabricator.wikimedia.org/T324520) (owner: 10Wfan) [16:08:54] do we want to reclassify the ideal 2+s as sepa? [16:09:21] hmm [16:10:02] does Adyen describe them as that? [16:10:42] ok yeah it does become a sepa payment [16:11:34] "paymentMethod":{ [16:11:36] "type":"sepadirectdebit", [16:11:38] "storedPaymentMethodId":"7219687191761347" [16:11:40] }, [16:18:49] yeah so the 1st is ideal and the 2+ are sepas [16:18:59] i am not sure we want to make this change? but maybe? https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/1010157/2/PaymentProviders/Adyen/ReferenceData.php [16:21:04] hmm it looks half right to me but the old comment looks even closer [16:21:06] // 'sepadirectdebit' => [ [16:21:08] // 'method' => 'dd', [16:21:10] // 'submethod' => 'dd_sepa', [16:21:12] // ], [16:21:16] although would that even work [16:55:24] !log payments-wiki upgraded from c7ab847d to c4f43931 [16:55:27] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [17:33:33] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 07Unplanned-Sprint-Work: Matching EFT import error - but not duplicate - https://phabricator.wikimedia.org/T358143#9758872 (10MDemosWMF) @Eileenmcnaughton Ok, we can add the gateway to the import files for now. Is there anything that... [19:40:43] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 07Unplanned-Sprint-Work: Matching EFT import error - but not duplicate - https://phabricator.wikimedia.org/T358143#9759366 (10Eileenmcnaughton) @MDemosWMF " she no longer has to enter anything for transaction ID." - so Ellen you mean... [19:47:45] (03CR) 10Eileen: [C:03+2] Move Ingenico audit test settings to extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1024086 (https://phabricator.wikimedia.org/T270675) (owner: 10Cstone) [19:49:37] (03CR) 10Eileen: [C:03+2] "Looks like this just leaves Amazon & ripping out the settings forms, alternate functions" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1024086 (https://phabricator.wikimedia.org/T270675) (owner: 10Cstone) [19:50:09] 06Fundraising-Backlog, 07Epic, 10MW-1.43-notes (1.43.0-wmf.4; 2024-05-07): Rename utm_ URL parameters through donatewiki and paymentswiki - https://phabricator.wikimedia.org/T351325#9759391 (10Ejegg) >>! In T351325#9757248, @Pcoombe wrote: > @Ejegg There is some special-casing on payments forms which changes... [20:03:32] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 07Unplanned-Sprint-Work: Matching EFT import error - but not duplicate - https://phabricator.wikimedia.org/T358143#9759408 (10MDemosWMF) Yes, she was entering it into the spreadsheet, but we are ok with it not being in Civi if we wil... [20:08:04] (03Merged) 10jenkins-bot: Move Ingenico audit test settings to extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1024086 (https://phabricator.wikimedia.org/T270675) (owner: 10Cstone) [21:40:57] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 13Patch-For-Review: Civi batch data entry tool build out - https://phabricator.wikimedia.org/T360139#9759684 (10MDemosWMF) Thanks @Eileenmcnaughton I see the field now! I also added 'soft credit type' so we can continue to mark that... [21:41:39] 03Fundraising Sprint: hammertime($touch_this=false), 06Fundraising-Backlog, 07Unplanned-Sprint-Work: Mapping Import Project - Engage import - https://phabricator.wikimedia.org/T360150#9759686 (10MDemosWMF) @Eileenmcnaughton great! Will be testing this tomorrow. [21:53:38] (03PS1) 10Eileen: Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1025861 [22:00:39] (03PS1) 10Umherirrender: Migrate to IReadableDatabase::newSelectQueryBuilder [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/1025863 (https://phabricator.wikimedia.org/T312369) [22:36:50] cstone: shall I deploy the ingenico audit patch [22:37:24] (03CR) 10Eileen: [C:03+2] Merge branch 'master' of ssh://gerrit.wikimedia.org:29418/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1025861 (owner: 10Eileen) [22:51:44] oh sure eileen i think it has run for the day already