[00:03:27] (03Merged) 10jenkins-bot: Fix the 'All' jackpot thank-you test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777032 (owner: 10Ejegg) [00:14:55] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Document new Acoustic apis etc on wikitech - https://phabricator.wikimedia.org/T305009 (10Eileenmcnaughton) https://wikitech.wikimedia.org/wiki/Fundraising/Data_and_Integrated_Processes/Acoustic_Integration... [00:21:33] (03CR) 10Ejegg: Fix the 'All' jackpot thank-you test (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777032 (owner: 10Ejegg) [00:21:52] (03PS9) 10Ejegg: Add Omnigroup.create api to create group at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/772968 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [00:23:29] (03CR) 10Ejegg: [C: 03+2] "Looks good, has unit test, works in local testing!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/772968 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [00:23:44] ejegg: I've started updating the docs for them - https://wikitech.wikimedia.org/wiki/Fundraising/Data_and_Integrated_Processes/Acoustic_Integration?veaction=edit§ion=22#Pushing_group_data_to_Acoustic [00:24:10] great! [00:25:03] eileen: is there any scenario where production would need to pull a remote group ID to store in that custom group_metadata field outside of the Omnigroup create action? [00:25:27] in our local testing it would be handy since we're all using the same remote database [00:25:40] and we all have the same local group names [00:25:55] but we only have the metadata for the ones we personally have pushed [00:26:34] oh wait, you were adding that as 'sync' in another patch, weren't you? [00:26:37] sorry [00:30:52] (03PS9) 10Ejegg: Store remote group ID to the custom field [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774582 (https://phabricator.wikimedia.org/T304475) (owner: 10Eileen) [00:31:13] (03CR) 10Ejegg: [C: 03+2] "Looks good, just another unused use statement to prune later" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774582 (https://phabricator.wikimedia.org/T304475) (owner: 10Eileen) [00:36:15] (03Merged) 10jenkins-bot: Add Omnigroup.create api to create group at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/772968 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [00:43:06] (03Merged) 10jenkins-bot: Store remote group ID to the custom field [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774582 (https://phabricator.wikimedia.org/T304475) (owner: 10Eileen) [00:44:12] (03PS8) 10Ejegg: Add groups to Acoustic data fetch output [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774587 (https://phabricator.wikimedia.org/T304475) (owner: 10Eileen) [00:45:59] (03CR) 10Ejegg: [C: 03+2] "Works locally once I manually set up the group_metadata row. Now that I see it in the display, that {{group.id}} seems maybe unnecessary o" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774587 (https://phabricator.wikimedia.org/T304475) (owner: 10Eileen) [00:46:35] (03CR) 10AndyRussG: [C: 03+2] "Fantastic, thanks for working on this!!!" [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/774560 (https://phabricator.wikimedia.org/T304769) (owner: 10Samwalton9) [00:49:00] (03Merged) 10jenkins-bot: Fix use of errorbox [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/774560 (https://phabricator.wikimedia.org/T304769) (owner: 10Samwalton9) [00:51:35] AndyRussG: I want my bash history to survive container rebuilds :) [00:51:58] so we just need to mount an external file... [00:52:58] ejegg: ah yeah we can do that! yeah just mount the fake home directory somewhere... I'd been thinking of that as a nice-to-have but never got to it [00:56:48] (03CR) 10Eileen: Add groups to Acoustic data fetch output (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774587 (https://phabricator.wikimedia.org/T304475) (owner: 10Eileen) [00:58:12] ejegg: I didn't have a plan to fix the 'local sites conflict on remote groups' issue - I suspect once this round is done we won't care much [00:58:27] eileen: ok, got it [00:58:48] we should probably delete the groups before we 'finish' though so next time we don't get confused [00:59:29] (03Merged) 10jenkins-bot: Add groups to Acoustic data fetch output [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774587 (https://phabricator.wikimedia.org/T304475) (owner: 10Eileen) [01:01:21] hmm I just tried running a "Sir Edgar J Higgenbotham III" through a full_name field and the middle initial dropped out. Will debug through and see where that gets lost [01:03:16] (03PS2) 10Eileen: Move variable declaration [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777033 [01:04:12] I think ^^ should be able to be merged by itself without messing with the chain at all [01:04:25] cool cool [01:04:27] Sir Edgar lost his J [01:04:32] so he did [01:05:21] if you +2 that minor one I'll rebase the last 2 & see if the use statement still needs to come out [01:07:01] (03CR) 10Ejegg: [C: 03+2] Move variable declaration [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777033 (owner: 10Eileen) [01:09:09] (03CR) 10Ejegg: "Smoke tested locally and it works. The full name is added to the queue message in full_name. The middle initial seems to get lost on the Q" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/774574 (https://phabricator.wikimedia.org/T300691) (owner: 10Ejegg) [01:17:27] (03CR) 10Eileen: [C: 03+2] "Yep that does the trick - alert is not super elegant but this is not something we hope to see often!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/776057 (https://phabricator.wikimedia.org/T304465) (owner: 10Ejegg) [01:17:44] thanks eileen [01:18:37] (03PS8) 10Eileen: Add api to create contacts at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) [01:18:39] (03PS8) 10Eileen: Add Omnimail push action [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774591 (https://phabricator.wikimedia.org/T305006) [01:19:04] ejegg: good to get that exception where it should be before we build that expectation in further! [01:19:35] yep yep [01:20:01] (03PS1) 10AndyRussG: Fix typo in impression-events-sample-rate-field [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/777036 (https://phabricator.wikimedia.org/T304638) [01:21:35] (03PS9) 10Eileen: Add api to create contacts at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) [01:21:36] ejegg fr-tech anyone for a quick +2 before I add stuff to the train ^ [01:21:37] (03PS9) 10Eileen: Add Omnimail push action [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774591 (https://phabricator.wikimedia.org/T305006) [01:21:58] sure AndyRussG [01:22:00] ok - that use statement is used in the end :-) [01:22:08] ejegg: thx! [01:22:22] (03CR) 10Ejegg: [C: 03+2] Fix typo in impression-events-sample-rate-field [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/777036 (https://phabricator.wikimedia.org/T304638) (owner: 10AndyRussG) [01:22:30] ejegg: yay thx! [01:23:39] (03PS1) 10Eileen: Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/777038 [01:24:08] ejegg: I pushed up a deploy - but it includes your thank you from - which means we just have to make sure we have the right settings [01:24:41] (03Merged) 10jenkins-bot: Fix typo in impression-events-sample-rate-field [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/777036 (https://phabricator.wikimedia.org/T304638) (owner: 10AndyRussG) [01:31:17] (03PS1) 10AndyRussG: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/777039 [01:31:48] eileen: should be right! [01:31:49] let' [01:32:01] s stop queues and do a flush in any case though [01:32:15] ejegg: so the current values are Jimmy for thank you [01:32:30] Katherine for EOY [01:32:40] and Lisa for endowment [01:33:00] I guess we should keep those for now & they can change when ready [01:33:05] (03CR) 10AndyRussG: [C: 03+2] Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/777039 (owner: 10AndyRussG) [01:33:17] well for Katherine it doesn't matter - it's bascally the Jimmy one [01:33:36] eileen oh, I thought they updated the TY text to be from Maryana already [01:34:09] (03CR) 10Eileen: "Just noting there is still more to do - this is a stage not the destination for the queue part" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774591 (https://phabricator.wikimedia.org/T305006) (owner: 10Eileen) [01:34:20] ejegg: the setting still says Jimmy [01:34:39] (03CR) 10Eileen: [C: 03+2] Merge branch 'master' of https://gerrit.wikimedia.org/r/wikimedia/fundraising/crm into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/777038 (owner: 10Eileen) [01:35:22] eileen: oh ok, let me see [01:35:31] (03Merged) 10jenkins-bot: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/777039 (owner: 10AndyRussG) [01:35:34] ahh, it was only the endowment one that they deployed [01:36:04] OK, so we can update the values in the UI to be Jimmy's name+address before we start the queues [01:36:28] yep - just turning off jobs [01:37:38] !log config revision changed from bb0e1af3 to 06379640 [01:37:39] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [01:39:40] jobs slow to stop.... [01:40:37] just ingenico audit running now - ejegg that maybe doesn't send any thank yous? [01:42:12] eileen: nope, any donations it finds just get pushed to the queue [01:42:48] !log civicrm revision changed from 84c737b6 to 87bc3114 [01:42:49] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [01:42:50] ok done ^^ [01:43:12] woohoo, let's do the system.flush to make sure those settings are created [01:43:31] (logging in now_ [01:43:32] ) [01:44:11] ok, flushed, going into the UI to change the annual fund ty from name+addy [01:44:25] think I'll keep the addy at donate@wikimedia.org for now [01:45:08] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-AutoTY-Email, 10FR-Email: From Name in Auto Thank You Emails needs update - https://phabricator.wikimedia.org/T304909 (10Eileenmcnaughton) Elliott has fixed this to be UI configurable - https://civicrm.wikimedia... [01:45:20] ok, that's saved [01:45:50] let's do a slow-start TY run to see what it sends [01:46:45] ejegg: I think I might have saved something too - do you want to just refresh & check it hasn't changed from what you set it to? [01:47:23] looks fine to me [01:47:31] cool [01:48:21] ejegg: slow start doesn't error - is there somewhere we should check the from? [01:48:56] huh, I thought it would show up in the mailing events tab [01:49:53] no, silly me, the activities tab [01:50:00] and the activity is there, but it doesn't show the from [01:50:36] maybe in the mail logs on civi box, lemme see if I can read those [01:52:39] I'll test send myself from the ui [01:53:21] eileen via your record, right? The TY test form still uses the arbitrary from values [01:53:43] ejegg: yep - I just got a nice thank you letter in some slavic language [01:53:59] from Jimmy? [01:54:01] Ukrainian [01:54:03] yep [01:54:21] great! [01:55:50] and I just thanked myself for an endowment gift and got an email from Lisa, so that's working too [01:56:11] cool [01:56:16] I'll turn queues back on [01:56:21] thanks eileen [01:57:41] !log process control config revision changed from 06379640 to 25728a0e [01:57:42] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [02:02:27] ahh, eileen, the parser spits out the J. in the 'initials' field but leaves 'middle_name' blank [02:03:30] (03PS10) 10Eileen: Add Omnimail push action [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774591 (https://phabricator.wikimedia.org/T305006) [02:03:32] seems like noplace else in Civi uses 'initials' though [02:04:52] ejegg: we could concat that into middle name I think - possibly without the . - unsure about that [02:04:55] I guess in the normalize function we could move it over [02:06:07] yeah [02:06:27] or even in https://github.com/eileenmcnaughton/deduper/blob/master/Civi/Api4/Action/Name/Parse.php#L77 - since that is civi integration & initials isn't a civi think [02:06:29] thing [02:06:35] lemme test a bit more and see what it does for e.g. 'J.W. Fields' - would somebody that writes that in their full name field expect to be addressed as "J.W." in a thank you letter? [02:07:30] quite nice being an API action, super easy to test [02:07:32] yeah no idea [02:07:54] I mean it should be J.W or just J. that would be right [02:08:33] ah boo, can't do it through the api explorer because it wants an array [02:08:48] oh right, it accepts [] [02:09:16] ok, that gets parsed to first_name J, initials W [02:09:41] so I guess initials being mapped to the middle_name bit makes sense in general [02:11:12] huh, "J. W. Fields" gets the dots passed through to first_name and initials [02:11:22] but "J.W. Fields" just gets the letters [02:11:27] * ejegg shrug [02:12:37] hmm, one other spot in deduper that looks at 'initials' [02:12:48] ah no, that's a different context [02:13:56] yeah there are probably lots of gotchas here - on the bright side we started from a very low bar [02:14:03] :) [02:19:10] whoa, "J. William Fields" parses William to the first_name and J. to initial [02:19:33] I guess writing your name like that means you go by William [02:19:50] but putting J in the middle_name field feels a bit off in that context [02:20:46] ehh, better than losing it altogether [02:24:05] (03PS1) 10Ejegg: Save initials to middle_name when blank [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777042 (https://phabricator.wikimedia.org/T300690) [02:24:58] (03PS11) 10Eileen: Add Omnimail push action [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774591 (https://phabricator.wikimedia.org/T305006) [02:25:10] (03CR) 10Ejegg: "See https://gerrit.wikimedia.org/r/777042 for the fix to save the middle initial on the Civi side." [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/774574 (https://phabricator.wikimedia.org/T300691) (owner: 10Ejegg) [02:42:44] (03CR) 10Eileen: [C: 03+2] Save initials to middle_name when blank [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777042 (https://phabricator.wikimedia.org/T300690) (owner: 10Ejegg) [02:42:55] seems fine to me [02:43:14] eileen: ahh running a bit behind schedule, available for a call in about 45 min if that's useful? [02:43:26] if that's late for u also it could be sooner [02:43:49] AndyRussG: 45 mins is fine [02:45:01] cool thx! [02:52:15] (03CR) 10Ejegg: "The existing form logic can validate minimums when the currency is not given in the URL and is selected from the dropdown. Can we keep tha" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/773840 (https://phabricator.wikimedia.org/T261436) (owner: 10Jgleeson) [02:54:57] (03Merged) 10jenkins-bot: Save initials to middle_name when blank [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777042 (https://phabricator.wikimedia.org/T300690) (owner: 10Ejegg) [03:30:16] eileen: heyyy howsie goesies? [03:30:27] just sitting down at the keyboard again here.... [03:32:26] AndyRussG: yep - I have a cuppa [03:32:30] so ready [03:32:40] cool! [03:32:56] ah actually just let me get my external monitor, one sec [03:35:30] (03PS10) 10Eileen: Add api to create contacts at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) [03:35:51] (03PS12) 10Eileen: Add Omnimail push action [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774591 (https://phabricator.wikimedia.org/T305006) [03:37:37] https://meet.google.com/mkm-dptq-dqq?authuser=0 [04:16:00] (03PS1) 10Eileen: Make new remote group identifier field read only [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777044 [05:17:04] (03CR) 10jerkins-bot: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_36) - 10https://gerrit.wikimedia.org/r/777089 (owner: 10L10n-bot) [05:29:38] (03CR) 10Nikerabbit: [V: 03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_36) - 10https://gerrit.wikimedia.org/r/777089 (owner: 10L10n-bot) [15:18:11] 10Fundraising-Backlog: Importing the Email Greeting field from Civi to Acoustic - https://phabricator.wikimedia.org/T305483 (10RLewis) [15:31:44] 10Fundraising-Backlog, 10FR-Email: Importing the Email Greeting field from Civi to Acoustic - https://phabricator.wikimedia.org/T305483 (10DStrine) [15:32:20] 10Fundraising-Backlog, 10FR-Email: Importing the Email Greeting field from Civi to Acoustic - https://phabricator.wikimedia.org/T305483 (10DStrine) This isn't a huge request but we will need to do this in coordination with @KHaggard [15:36:00] 10Fundraising-Backlog, 10FR-Email: Importing the Email Greeting field from Civi to Acoustic - https://phabricator.wikimedia.org/T305483 (10RLewis) Thanks David and yes @KHaggard is looped in on the task and the request. [15:39:00] 10Fundraising-Backlog: Added new fields to the Benefactor Page Listing section in Civi - https://phabricator.wikimedia.org/T305485 (10RLewis) [15:42:52] 10Fundraising-Backlog: Adding a new field to the Endowment Benefactor Page listing - https://phabricator.wikimedia.org/T305486 (10RLewis) [15:48:39] Hi fr-tech still no internet but I'll be back at Panera for meetings [15:49:27] (03CR) 10Wfan: [C: 03+2] "print result, and do tested 'OK (1 test, 7 assertions)'" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/776251 (https://phabricator.wikimedia.org/T300689) (owner: 10Damilare Adedoyin) [15:49:56] uff. [15:51:03] (03Merged) 10jenkins-bot: Add unit tests for Google Pay [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/776251 (https://phabricator.wikimedia.org/T300689) (owner: 10Damilare Adedoyin) [15:59:00] hey cstone dwisehaupt fr-tech! sorry to hear your internet is still being naughty cstone ! [16:00:17] hiii jgleeson cstone dwisehaupt fr-tech :) [16:05:56] 10Fundraising-Backlog, 10FR-Email: Importing the Email Greeting field from Civi to Acoustic - https://phabricator.wikimedia.org/T305483 (10EYener) Thank you, and thanks to Thea! We do want to keep in coordination with data and import / export requests, but since this one involves obvious PII, we (Analytics) wo... [16:12:07] hey AndyRussG! [16:15:09] 10Fundraising-Backlog, 10FR-Email: Importing the Email Greeting field from Civi to Acoustic - https://phabricator.wikimedia.org/T305483 (10KHaggard) Thanks Rosie, Thea, and Erin! @EYener - do you have a preference for how we name the field? Considering this for all future field additions until we have time to... [16:24:01] 10Fundraising-Backlog, 10Wikipedia-Android-App-Backlog (Android Release FY2021-22): English fundraising message test for Android app users in India in June 2022? - https://phabricator.wikimedia.org/T305405 (10LGoto) p:05Triage→03High [16:31:21] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-AutoTY-Email, 10FR-Email: From Name in Auto Thank You Emails needs update - https://phabricator.wikimedia.org/T304909 (10KHaggard) Oh nice, thanks @Ejegg and @Eileenmcnaughton ! @TSkaff - Looks like we can edit... [16:36:57] 10Fundraising-Backlog, 10FR-Email: Importing the Email Greeting field from Civi to Acoustic - https://phabricator.wikimedia.org/T305483 (10EYener) Thanks @KHaggard : I'd suggest `email_greeting`. In the UI, it is listed as `Email Greeting` and in the database it's listed as `email_greeting_display` so for the... [16:38:00] 10Fundraising-Backlog, 10FR-Email: Importing the Email Greeting field from Civi to Acoustic - https://phabricator.wikimedia.org/T305483 (10KHaggard) Awesome. Thanks for your input, and I agree! [16:57:54] (03CR) 10Wfan: "based on YYMMDD SSSS CAZ, the pattern 's first part 'YYMMDD SSSS' seems not right?" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/759004 (owner: 10Ejegg) [17:07:33] 10Fundraising-Backlog, 10Wikipedia-Android-App-Backlog (Android Release FY2021-22): English fundraising message test for Android app users in India in June 2022? - https://phabricator.wikimedia.org/T305405 (10JTannerWMF) TLDR: We can accommodate this request. We will put it on our board once we have the verbia... [17:41:47] fr-tech for ssh tunneling to cloud what is the ssh config we need? I don't have it on my mac [17:55:15] cstone ah one sec [17:56:46] also flaw of only having one usbc to usb adapter and my headset and yubikey need it haha [18:19:31] cstone: if your trying to fix the issue with regards to reoslving the hostname when tunneling with fundrasing-dev this patch will do it https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/dev/+/771446 [18:19:44] 10Fundraising-Backlog, 10Fundraising-Backlog-Old, 10MediaWiki-extensions-DonationInterface: General DonationInterface code cleanup - https://phabricator.wikimedia.org/T87043 (10Umherirrender) 05Open→03Resolved [18:20:01] AndyRussG: do you think that ones worth merging to save folks needing the custom ssh config? [18:21:38] I was working on adding some options to that script so that we could stop the service also [18:26:36] unless folks know of a nicer way to stop the service once it's running? I had to kill the pid as I accidentally ran it more than once [18:44:43] (03PS2) 10Ejegg: Add validation pattern for ZA National ID [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/759004 [18:45:17] (03CR) 10Ejegg: "Thanks Wfan, I updated the regex for the two digit YY" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/759004 (owner: 10Ejegg) [18:49:13] (03PS2) 10Ejegg: Make new remote group identifier field read only [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777044 (owner: 10Eileen) [18:49:18] (03CR) 10Ejegg: [C: 03+2] Make new remote group identifier field read only [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777044 (owner: 10Eileen) [19:00:34] (03CR) 10Wfan: [C: 03+2] "looks good 😊" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/759004 (owner: 10Ejegg) [19:02:33] (03Merged) 10jenkins-bot: Make new remote group identifier field read only [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/777044 (owner: 10Eileen) [19:02:41] thanks wfan [19:03:59] 😀 [19:04:14] (03Merged) 10jenkins-bot: Add validation pattern for ZA National ID [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/759004 (owner: 10Ejegg) [19:10:25] (03CR) 10Jgleeson: "Thanks for feedback! I've replied inline and tried my best to split the big one up to make it clear. Happy to chat more on a call also if " [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/773840 (https://phabricator.wikimedia.org/T261436) (owner: 10Jgleeson) [19:11:14] (03CR) 10AndyRussG: "Cool, thanks for this!!! Just two minor comments so far. :)" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [19:12:48] thanks for the review ejegg [19:13:04] sure thing jgleeson [19:16:34] 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice: MediaWiki:Centralnotice: contradiction for upper bound: Centralnotice-banner-history-logger-rate-help vs. Centralnotice-impression-events-sample-rate-field - https://phabricator.wikimedia.org/T304638 (10AndyRussG) Hi! Thanks for finding this. Fixed i... [19:30:09] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Document new Acoustic apis etc on wikitech - https://phabricator.wikimedia.org/T305009 (10DStrine) p:05Triage→03Medium [19:30:16] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Add API to push a group to Acoustic - https://phabricator.wikimedia.org/T305006 (10DStrine) p:05Triage→03Medium [19:30:29] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Figure out how to batch tasks within Acoustic push - https://phabricator.wikimedia.org/T305007 (10DStrine) p:05Triage→03Medium [19:30:57] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog: Brief Civi Deadlock 3/30/22 - https://phabricator.wikimedia.org/T305089 (10DStrine) 05Open→03Resolved p:05Triage→03High [19:33:28] 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, 10Fundraising-Backlog: Refactor ConfigurationReader - https://phabricator.wikimedia.org/T305152 (10DStrine) p:05Triage→03Medium [19:38:56] (03CR) 10AndyRussG: "Quick question--is it expected that we see the same result output if the contact is already in contact list on Acoustic?" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [19:39:09] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10FR-Adyen: Enable Google Pay on Production - https://phabricator.wikimedia.org/T304991 (10DStrine) [19:43:15] (03PS11) 10Eileen: Add api to create contacts at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) [19:45:18] (03CR) 10Eileen: Add api to create contacts at Acoustic (033 comments) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [19:45:56] 10Fundraising-Backlog, 10Continuous-Integration-Config: mediawiki/extensions/DonationInterface/vendor repo needs CI V+2 jobs - https://phabricator.wikimedia.org/T143025 (10Umherirrender) [19:46:37] 10Fundraising Sprint English Cuisine, 10Fundraising Sprint Far Beer, 10Fundraising Sprint Gondwanaland Reunification Engine, 10Fundraising Sprint Homebrew Hadron Collider, and 4 others: Clean up confusing and repeated code in the PayPal express adapter - https://phabricator.wikimedia.org/T134445 (10Umherirr... [19:47:00] 10Fundraising-Backlog, 10FR-Ingenico, 10MediaWiki-extensions-DonationInterface: Orphan rectifier is silent about communication failures - https://phabricator.wikimedia.org/T161160 (10Umherirrender) 05Open→03Resolved [19:47:10] 10Fundraising-Backlog, 10FR-dlocal, 10MediaWiki-extensions-DonationInterface: Client-side validation messages are not country specific - https://phabricator.wikimedia.org/T133706 (10Umherirrender) [19:47:30] 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface: Refactor: Sort out transaction_option vs getData* vs getTransactionSpecificValue vs account* constellation - https://phabricator.wikimedia.org/T137164 (10Umherirrender) 05Open→03Resolved [19:48:00] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Fix malformed transaction IDs for recent refunds - https://phabricator.wikimedia.org/T115511 (10Umherirrender) 05Open→03Resolved [19:48:10] 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface: Mustache forms should provide server-side error map to client-side script - https://phabricator.wikimedia.org/T112323 (10Umherirrender) [19:48:14] 10Fundraising-Backlog, 10FR-Adyen, 10FR-Smashpig: IPN listener should be able to update payments_final_status - https://phabricator.wikimedia.org/T137590 (10Umherirrender) 05Open→03Resolved [19:48:32] 10Fundraising Sprint Uptight Piano, 10Fundraising Sprint Vaporwerewolf, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, and 2 others: Epic: Don't duplicate contact for email donations - https://phabricator.wikimedia.org/T177663 (10Umherirrender) [19:48:56] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Investigate use of strtotime() in Civi - https://phabricator.wikimedia.org/T177640 (10Umherirrender) [19:49:04] 10Fundraising Sprint Anti-matter doesn't matter, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint Xenomorph Petting Zoo, 10Fundraising Sprint e^🥧👀=yum, and 5 others: Enable South Africa through Dlocal - https://phabricator.wikimedia.org/T293508 (10DStrine) @EMartin this is ready to test again. Can you... [19:49:28] 10Fundraising-Backlog, 10FR-Ingenico, 10FR-Smashpig, 10MediaWiki-extensions-DonationInterface, 10Technical-Debt: [Tech Debt] Separate gateway constructor, addData, and validate - https://phabricator.wikimedia.org/T159910 (10Umherirrender) [19:49:35] 10Fundraising-Backlog, 10FR-Smashpig, 10Epic: [EPIC] Reduce use of SmashPig\Core\Configuration where possible - https://phabricator.wikimedia.org/T158374 (10Umherirrender) 05Open→03Resolved [19:49:38] 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice: Inline help on all CentralNotice forms - https://phabricator.wikimedia.org/T134958 (10Umherirrender) [19:50:23] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: CiviCRM bounce import job is failing - https://phabricator.wikimedia.org/T122265 (10Umherirrender) [19:50:25] 10Fundraising-Backlog: Accept incoming unsubscribe emails - https://phabricator.wikimedia.org/T93427 (10Umherirrender) [19:50:27] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Epic: Make CiviMail useful - https://phabricator.wikimedia.org/T91710 (10Umherirrender) [19:50:29] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Speed up CiviCRM queue processing - https://phabricator.wikimedia.org/T119986 (10Umherirrender) 05Open→03Resolved [19:50:33] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Fix performance problems recording thank you emails to CiviMail records - https://phabricator.wikimedia.org/T86863 (10Umherirrender) 05Open→03Resolved [19:51:28] 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice: CentralNotice: Remove unused code for banner preview in banner editor - https://phabricator.wikimedia.org/T161907 (10Umherirrender) 05Open→03Resolved [19:52:05] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10FR-Adyen: Enable Google Pay on Production - https://phabricator.wikimedia.org/T304991 (10Cstone) a:03Cstone [19:56:13] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Add ability to view acoustic data in CiviCRM - https://phabricator.wikimedia.org/T304465 (10Eileenmcnaughton) [19:56:26] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Add ability to view acoustic data in CiviCRM - https://phabricator.wikimedia.org/T304465 (10Eileenmcnaughton) [19:56:58] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Add ability to view acoustic data in CiviCRM - https://phabricator.wikimedia.org/T304465 (10Eileenmcnaughton) 05Open→03Resolved [19:57:00] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Civi Searchkit should allow user to export list to acoustic - https://phabricator.wikimedia.org/T301514 (10Eileenmcnaughton) [19:57:27] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Prospect: Three new custom fields on the prospect tab - https://phabricator.wikimedia.org/T304479 (10Eileenmcnaughton) 05Open→03Resolved [19:57:47] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Civi-Prospect, 10Patch-For-Review: New custom field - Prospect Insight - https://phabricator.wikimedia.org/T304268 (10Eileenmcnaughton) 05Open→03Resolved [19:58:04] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Add api to allow creation of group at Acoustic - https://phabricator.wikimedia.org/T304472 (10Eileenmcnaughton) 05Open→03Resolved [19:58:06] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Civi Searchkit should allow user to export list to acoustic - https://phabricator.wikimedia.org/T301514 (10Eileenmcnaughton) [19:59:26] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10FR-Adyen, 10FR-Gpay-Integration: Add GPay to CivICRM - https://phabricator.wikimedia.org/T303623 (10Eileenmcnaughton) @AnnWF I think this is resolved - we don't have an external person who needs to check this one [19:59:50] 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice: MediaWiki:Centralnotice: contradiction for upper bound: Centralnotice-banner-history-logger-rate-help vs. Centralnotice-impression-events-sample-rate-field - https://phabricator.wikimedia.org/T304638 (10DStrine) 05Open→03Resolved [20:00:05] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, 10Fundraising-Backlog: time spent on CN SEO meetings and related work - https://phabricator.wikimedia.org/T302346 (10DStrine) 05Open→03Resolved [20:26:40] 10Fundraising-Backlog, 10fr-donorservices: Agents in need of additional Civi permission - https://phabricator.wikimedia.org/T305505 (10SHust) [21:11:41] (03CR) 10Wfan: [C: 03+2] Send full_name for Amazon donations [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/774574 (https://phabricator.wikimedia.org/T300691) (owner: 10Ejegg) [21:12:05] We updated/deployed the new TY emails and things are looking good so far. Thank you eileen, dstrine, and any others that helped make this new tool possible for us to use! [21:12:31] great tskaff! [21:12:45] Do you need the From name/email updated too? [21:13:36] I see we can now do that ourselves too, awesome. The only weird thing is that the from name updates immediately, whereas there's a delay with content refreshing, but we can deal! [21:14:05] right tskaff, we have a task filed to make the content refresh more quickly [21:14:38] Oh great ok [21:14:38] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-AutoTY-Email, 10FR-Email: From Name in Auto Thank You Emails needs update - https://phabricator.wikimedia.org/T304909 (10TSkaff) This is great news! Things are looking good so far (I updated the one-time/recurri... [21:23:16] (03Merged) 10jenkins-bot: Send full_name for Amazon donations [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/774574 (https://phabricator.wikimedia.org/T300691) (owner: 10Ejegg) [21:32:46] 10Fundraising Sprint Anti-matter doesn't matter, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint Xenomorph Petting Zoo, 10Fundraising Sprint e^🥧👀=yum, and 5 others: Enable South Africa through Dlocal - https://phabricator.wikimedia.org/T293508 (10EMartin) We just tested the links. Did something chan... [21:33:56] 10Fundraising Sprint Anti-matter doesn't matter, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint Xenomorph Petting Zoo, 10Fundraising Sprint e^🥧👀=yum, and 5 others: Enable South Africa through Dlocal - https://phabricator.wikimedia.org/T293508 (10EMartin) @RKumar fyi. @Frtech, WMF We had tested the S... [21:36:06] Woo there is internet [21:36:11] nice! [21:52:53] 10Fundraising Sprint Anti-matter doesn't matter, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint Xenomorph Petting Zoo, 10Fundraising Sprint e^🥧👀=yum, and 5 others: Enable South Africa through Dlocal - https://phabricator.wikimedia.org/T293508 (10DStrine) 05Open→03Resolved Ah ok sorry never mind.... [21:55:35] 10Fundraising-Backlog, 10fundraising-tech-ops: Deactivate fundraising accounts for jrobell - https://phabricator.wikimedia.org/T305513 (10Dwisehaupt) [21:56:21] 10Fundraising Sprint Anti-matter doesn't matter, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint Xenomorph Petting Zoo, 10Fundraising Sprint e^🥧👀=yum, and 5 others: Enable South Africa through Dlocal - https://phabricator.wikimedia.org/T293508 (10EMartin) Great, thanks! @Rakhi Kumar 10fundraising-tech-ops, 10Infrastructure-Foundations, 10Mail, 10SRE, 10Znuny: move donation,donate, donations (otrs, wikimania) exim aliases from SRE to ITS - https://phabricator.wikimedia.org/T297915 (10Dzahn) 05Open→03Stalled ACK, I am setting this to stalled until May. [22:18:12] 10fundraising-tech-ops, 10Infrastructure-Foundations, 10Mail, 10SRE, 10Znuny: move donation,donate, donations (otrs, wikimania) exim aliases from SRE to ITS - https://phabricator.wikimedia.org/T297915 (10Dzahn) [22:24:25] 10Fundraising-Backlog, 10fundraising-tech-ops: Deactivate fundraising accounts for jrobell - https://phabricator.wikimedia.org/T305513 (10Dwisehaupt) [22:27:17] 10Fundraising-Backlog, 10fundraising-tech-ops: Deactivate fundraising accounts for jrobell - https://phabricator.wikimedia.org/T305513 (10Dwisehaupt) [22:29:53] 10Fundraising-Backlog, 10fundraising-tech-ops: Deactivate fundraising accounts for jrobell - https://phabricator.wikimedia.org/T305513 (10Dwisehaupt) Commits related to deactivation: ` [frack::puppet::private] 8eb7341 deactivate jrobell account. [frack::puppet::private] 50f81a5 Revoke jrobell ssl client cert [... [22:38:54] 10Fundraising-Backlog, 10fundraising-tech-ops: Deactivate fundraising accounts for jrobell - https://phabricator.wikimedia.org/T305513 (10Dwisehaupt) [23:41:47] (03CR) 10Wfan: "Seems like we now can call the contact create api multiple times with different group id (as long as the Group_Metadata.remote_group_ident" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [23:42:50] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Add API to push a group to Acoustic - https://phabricator.wikimedia.org/T305006 (10AnnWF) https://gerrit.wikimedia.org/r/c/wikimedia/f... [23:44:08] (03CR) 10Wfan: Add api to create contacts at Acoustic (031 comment) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [23:45:37] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Document new Acoustic apis etc on wikitech - https://phabricator.wikimedia.org/T305009 (10AnnWF) I think this document looks pretty good :) could move to done [23:46:07] (03CR) 10Eileen: Add api to create contacts at Acoustic (032 comments) [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [23:47:13] (03PS12) 10Eileen: Add api to create contacts at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) [23:49:28] thanks for checking those docs wfan [23:51:09] 10Fundraising Sprint Cheese String Theory, 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising Sprint Fibonachos, 10Fundraising Sprint e^🥧👀=yum, and 3 others: Figure out how to move batch processing to a background task - https://phabricator.wikimedia.org/T305516 (10Eileenmcnaughton) [23:52:09] 10Fundraising Sprint Fibonachos, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Email: Figure out how to batch tasks within Acoustic push - https://phabricator.wikimedia.org/T305007 (10Eileenmcnaughton) [23:53:50] ejegg: I was looking at updating smashpig in DI and when i run composer update it updates it to 0.7.9.5 but it should be updating it to 0.7.9.6 [23:53:51] (03CR) 10jerkins-bot: [V: 04-1] Add api to create contacts at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) (owner: 10Eileen) [23:53:55] which civi is already updated to [23:55:36] (03PS13) 10Eileen: Add api to create contacts at Acoustic [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/774590 (https://phabricator.wikimedia.org/T304473) [23:59:05] /wg 13