[00:52:46] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Long-running tasks such as audit parsing can lose Redis connection and drop queue messages - https://phabricator.wikimedia.org/T266591 (10Cstone) This has been back twice from the donations queue consumer [01:37:19] (03PS3) 10Ejegg: Scripts for doing token migration in the db [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/965844 (https://phabricator.wikimedia.org/T344645) [01:53:54] (03PS4) 10Ejegg: Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [01:54:10] eileen: does that look OK? ^^ [01:54:31] I changed the name so we can store multiple external identifiers for the same donor. [01:55:22] Since some will have one ID at frup, one at paypal, maybe one at venmo, etc [02:03:26] (03CR) 10Ejegg: [C: 03+1] "C+1 for Dami's code, will wait for someone else to C+2" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [02:04:45] (03CR) 10Ejegg: [C: 03+1] "Oh, we probably want to add the external ID down in the handleUpdate function too, for donors whose email + name match existing donors." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [02:05:24] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Fr-tech chores list - https://phabricator.wikimedia.org/T258527 (10XenoRyet) [02:09:34] (03CR) 10CI reject: [V: 04-1] Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [03:00:15] (03PS2) 10Eileen: Do not log activity detail for bounce activities [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/965264 (https://phabricator.wikimedia.org/T347724) [03:00:17] (03PS2) 10Eileen: Fix CustomField handling to stop logging disabled custom fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/965274 (https://phabricator.wikimedia.org/T347724) [03:00:19] (03PS1) 10Eileen: Disable fields to be removed during maitenance window [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966318 (https://phabricator.wikimedia.org/T347724) [03:00:21] (03PS1) 10Eileen: Update calculation of WMF Donor fields to exclude fields for removal [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966319 (https://phabricator.wikimedia.org/T347724) [03:15:47] (03CR) 10CI reject: [V: 04-1] Update calculation of WMF Donor fields to exclude fields for removal [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966319 (https://phabricator.wikimedia.org/T347724) (owner: 10Eileen) [04:13:49] (03PS5) 10Eileen: Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [04:13:51] (03PS1) 10Eileen: Remove failed attempt at re-saving weight [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966323 (https://phabricator.wikimedia.org/T335070) [04:19:07] (03PS2) 10Eileen: Remove failed attempt at re-saving weight [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966323 (https://phabricator.wikimedia.org/T335070) [04:29:26] (03CR) 10CI reject: [V: 04-1] Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [04:30:12] (03CR) 10CI reject: [V: 04-1] Remove failed attempt at re-saving weight [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966323 (https://phabricator.wikimedia.org/T335070) (owner: 10Eileen) [04:31:02] (03PS3) 10Eileen: Remove failed attempt at re-saving weight [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966323 (https://phabricator.wikimedia.org/T335070) [04:37:03] (03CR) 10Eileen: "Opps - overwrote ejegg's patch - hopefully put it back (just had it in my tree when pushing another patch)" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [06:25:29] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Internal test: for validate all dlocal submethods - https://phabricator.wikimedia.org/T346814 (10RKumar_WMF) @AnnWF - The tester in Brazil tried to use Mastercard which serves both as Credit and Debit but when she completed the donation, there was no opti... [06:29:27] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Internal test: for validate all dlocal submethods - https://phabricator.wikimedia.org/T346814 (10RKumar_WMF) @AnnWF - For CL - The cash payment method is not appearing on the donation form. {F38465420} Please let me know how can we test that? Ty. [06:40:28] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_39) - 10https://gerrit.wikimedia.org/r/966365 (owner: 10L10n-bot) [08:18:59] 10Fundraising Sprint This LAN is your LAN, 10Fundraising-Backlog, 10Patch-For-Review: All Ingenico/Worldline active tokens will be migrated to Adyen - https://phabricator.wikimedia.org/T344645 (10RKumar_WMF) Adyen has responded to the query. Since FR-tech is already copied on the emails, I am not posting the... [11:33:22] (03CR) 10Nikerabbit: [V: 03+2] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_39) - 10https://gerrit.wikimedia.org/r/966365 (owner: 10L10n-bot) [11:54:21] (03PS6) 10Damilare Adedoyin: Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) [12:09:51] (03CR) 10CI reject: [V: 04-1] Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [12:13:38] (03PS7) 10Damilare Adedoyin: Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) [12:28:37] 10fundraising-tech-ops: Update the kafka-jumbo hosts for kafkatee - https://phabricator.wikimedia.org/T349088 (10brouberol) [12:37:09] 10fundraising-tech-ops: Update the kafka-jumbo hosts for kafkatee - https://phabricator.wikimedia.org/T349088 (10elukey) Not sure if already mentioned but the current list of brokers has been expanded up to kafka-jumbo1015.eqiad.wmnet, so now it should be: kafka-jumbo[1007-1015].eqiad.wmnet Added @Jgreen to th... [13:05:32] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10jgleeson) [13:11:47] (03CR) 10Damilare Adedoyin: "Thanks for the fixes ejegg and eileen, the patch now passes CI" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [13:15:10] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Handle Adyen non refund IPNs for non-USD donations - https://phabricator.wikimedia.org/T347422 (10Damilare) a:03Damilare [13:49:32] gonna pickup the kids. back soon! [13:59:03] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10Ejegg) Looks like this is searching for Group IDs rather than Contact IDs. @Eileenmcnaughton do you think something has changed in the UI so that the form field they're used to using... [14:13:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:7, minFraud-West:7) 14 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:18:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:7, minFraud-West:7) 14 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:23:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:7, minFraud-West:7) 14 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:28:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:7, minFraud-West:7) 14 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:28:51] ughhhh [14:29:43] multiple recurrings attached to the same ingenico token are sometimes getting mapped to a single invoice_id [14:29:47] in the Adyen migration [14:30:13] so that's going to violate the unique constraint on civicrm_contribution_recur.invoice_id [14:30:22] what to do.... [14:30:38] custom field on the payment_token or contribution_recur table? [14:31:41] ok, so we already have a contribution_recur_smashpig group [14:31:45] let's see what's in that now [14:32:12] just the initial scheme txn ID [14:32:22] OK, I guess we should put the shopperReference there [14:32:56] I'll use the 'normalized' name for it - processor_contact_id [14:33:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:7, minFraud-West:7) 14 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:33:34] huh, though that's unfortunately very similar to the external identifiers we're adding on Dami's table [14:33:55] This one really does need to be per-contribution_recur row though [14:38:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:7, minFraud-West:7) 14 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:43:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:5, minFraud-West:5) 10 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:48:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:7, minFraud-West:7) 14 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:51:26] 10Fundraising Sprint This LAN is your LAN, 10Fundraising-Backlog: Store Adyen ShopperReference (processor_contact_id) in new custom field - https://phabricator.wikimedia.org/T349107 (10Ejegg) [14:52:23] 10Fundraising Sprint This LAN is your LAN, 10Fundraising-Backlog: Store Adyen ShopperReference (processor_contact_id) in new custom field - https://phabricator.wikimedia.org/T349107 (10Ejegg) a:03Ejegg [14:52:34] ok, ^^^ describes the work [14:52:40] I hope that's a good idea [14:53:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:7, minFraud-West:7) 14 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [14:58:18] PROBLEM - check_log_messages on frav1003 is CRITICAL: CRITICAL: check_endpoints_critical (minFraud-East:4, minFraud-West:4) 8 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [15:03:18] RECOVERY - check_log_messages on frav1003 is OK: OK https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1003&service=check_log_messages [15:04:22] sounds plausible to me ejegg [15:04:24] reading ticket [15:05:59] reasonable would have been a better word [15:07:26] at first I was thinking why are people creating multiple recurrings but then it's not unthinkable however most of them are probably us! when testing [15:09:42] ejegg: how come we called that table contr...smashpig [15:09:59] (03PS1) 10Ejegg: Create new field for Adyen Shopper Reference [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966557 (https://phabricator.wikimedia.org/T349107) [15:10:01] is it holding stuff for the civicrm smashpig recurring charge ext [15:28:01] jgleeson: because it was for stuff we touch in the SmashPig extension, I guess [15:28:23] are we breaking that rule with this new one? [15:31:40] 10fundraising-tech-ops: Fundraising access request for Julie Ian-Wong - https://phabricator.wikimedia.org/T349110 (10ERoden-WMF) [15:35:16] 10fundraising-tech-ops: Fundraising Access Request for Lauren Nadai - https://phabricator.wikimedia.org/T349111 (10ERoden-WMF) [15:35:23] 10Fundraising-Backlog: Add non-native browser lack of support for Venmo to avoid donor confusion - https://phabricator.wikimedia.org/T349112 (10EMartin) [15:37:33] 10fundraising-tech-ops: Fundraising Access for Shaoyan Xu - https://phabricator.wikimedia.org/T349113 (10ERoden-WMF) [15:44:50] hmm, so, those temp tables I was working with assumed that we would have at least one row in the CSV per contribution_recur row [15:45:23] now that I've found multiple contribution_recurs associated with single CSV rows, I guess I need to change those temp tables [15:46:15] Would be better to put the new shopperReference column on the contribution_recur_copy [15:46:28] and the new Adyen token on the payment_tokens_copy table [15:46:51] plus the flag of which token rows need to be created [15:47:31] jgleeson it'll still hold stuff we touch in the SmashPig extension [15:47:56] the Adyen ShopperReference is only needed for recurring contributions that we charge via that SmashPig extension [15:48:23] so I should probably update it with Civix before I add an upgrader to move those values [15:58:15] oh fun, we have to run civix from within the docker container? [15:58:34] looks like its include paths are all gleaned from the settings, which i guess makes sense [15:58:38] ejegg: I guess I was thinking more for outside-of-wmf usage [15:58:39] but that's annoying [15:59:00] and whether it would make sense to rename the tbl [15:59:23] but nvm sounds like you figured something else out [15:59:32] jgleeson: right, so for outside-of-wmf usage this is actually better than stuffing the shopperReference in invoice_id [15:59:37] I'm curious, why is that annoying [15:59:52] running civix from inside the container? [16:00:03] I just didn't have it available there yet and had to copy it over [16:00:11] I'm used to editing code on the host machine [16:00:23] oh right, I think it should be installed with the buildkit install [16:00:32] ohhh [16:00:39] i didn't think to look there [16:00:47] i'd been running it standalone locally [16:00:53] thanks! [16:00:54] is it not in the path on the container [16:01:14] double-checking [16:01:52] hah, it totally is [16:01:56] ok, deleting my extra copy [16:02:07] and I guess I should make sure I have the latest civix [16:02:13] ok cool. I was just checking as if it wasn't, we could have added a script in the /scripts dir [16:02:18] good that it is [16:02:31] buildkit seems to set up a bunch of stuff which comes in handy [16:06:00] (03PS1) 10Ejegg: Regenerate SmashPig Civix files [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966566 (https://phabricator.wikimedia.org/T349107) [16:08:55] that looks straightforward ejegg but just wanted to check if it was you that added the smarty mixin? [16:09:11] oh that was just done by Civix [16:09:30] let's see if we have any templates in there [16:09:41] ah ok [16:10:13] yeah, it looks like there is one template that gets rendered by smarty [16:10:40] I think that mixin adds the extension's templates to the smarty include path [16:11:30] heh, I guess I can update that comment in info.xml [16:12:21] nice [16:13:55] oh lord, i made a gitlab for that extension a million years ago and have never updated it [16:14:01] 10fundraising-tech-ops: Update the kafka-jumbo hosts for kafkatee - https://phabricator.wikimedia.org/T349088 (10Jgreen) [16:14:01] https://gitlab.com/ejegg/org.wikimedia.smashpig [16:14:25] ok, 5 years, not a million [16:14:35] but that's a pretty rudimentary version there [16:15:40] (and still linked as the extension homepage in info.xml) [16:15:44] (03PS2) 10Ejegg: Regenerate SmashPig Civix files [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966566 (https://phabricator.wikimedia.org/T349107) [16:15:49] 10fundraising-tech-ops: Update the kafka-jumbo hosts for kafkatee - https://phabricator.wikimedia.org/T349088 (10Jgreen) >>! In T349088#9257624, @elukey wrote: > Not sure if already mentioned but the current list of brokers has been expanded up to kafka-jumbo1015.eqiad.wmnet, so now it should be: > > kafka-jumb... [16:15:58] 10fundraising-tech-ops: Update the kafka-jumbo hosts for kafkatee - https://phabricator.wikimedia.org/T349088 (10Jgreen) 05Open→03In progress p:05Triage→03Medium a:03Jgreen [16:16:29] 5 years ago! [16:16:37] that flew [16:17:27] (03CR) 10Jgleeson: [C: 03+2] "LGTM!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966566 (https://phabricator.wikimedia.org/T349107) (owner: 10Ejegg) [16:17:39] thanks! [16:18:00] np [16:30:50] ok it looks like I was wrong last night when saying that we're not logging the outbound requests for paypal. I'm testing locally and i can see the SetExpressCheckout nvp params getting logged. I wonder if it's the same for DoExpressCheckout [16:40:37] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10jgleeson) Good spot @Ejegg on the ID. I missed that earlier! [16:48:45] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10greg) p:05Triage→03High Feels high? Or is it "the big swath of medium"? [16:51:59] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: https://apple-pay-gateway.apple.com SSL certificate has expired - https://phabricator.wikimedia.org/T348972 (10greg) p:05Triage→03High [16:52:03] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: https://apple-pay-gateway.apple.com SSL certificate has expired - https://phabricator.wikimedia.org/T348972 (10greg) p:05High→03Unbreak! [16:52:16] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Add CSP for Fundraiseup on DonateWiki - https://phabricator.wikimedia.org/T345379 (10greg) p:05Triage→03High [16:58:44] (03PS3) 10Ejegg: Regenerate SmashPig Civix files [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966566 (https://phabricator.wikimedia.org/T349107) [16:58:46] (03PS2) 10Ejegg: Create new field for Adyen Shopper Reference [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966557 (https://phabricator.wikimedia.org/T349107) [16:58:48] (03PS1) 10Ejegg: Copy invoice_id to processor_contact_id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966579 (https://phabricator.wikimedia.org/T349107) [17:00:00] ok jgleeson I rebased the civix regenerate to the front of the chain [17:11:23] (03PS2) 10Ejegg: Copy invoice_id to processor_contact_id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966579 (https://phabricator.wikimedia.org/T349107) [17:14:31] ah gotcha [17:14:38] just sending this email, eating and then will check it out [17:14:53] this email is a piece of work [17:17:33] :) [17:17:56] yeah, I think the civix generate is safe enough to merge + deploy whenever [17:18:13] but i'll want to do a lot of testing on the column move [17:19:30] ejegg, would you be able to review the frup id patch again. I made a little modification to your last change [17:19:40] oh sure damilare [17:19:48] here's the patch: https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/966267 [17:20:19] basically a test failed because the gateway parameter was missing in the $msg object [17:20:26] so I added a check [17:20:58] ah yeah, safer that way [17:21:37] So I'll just C+2 that if you're happy with the new table/column names [17:22:34] This is unrelated, but I want to modify the Logdog log searching scripts to stop at the first match to make it easier to find a single instance of something. I'm pondering whether the flag to pass to the script should be --one, --limit, or --first. What do you think is best? [17:23:39] I think grep has the same thing, right? [17:23:50] --max-occurance [17:24:08] yep looks good ejegg [17:24:19] ah, but that's a bit more general than you want jgleeson, right? [17:24:28] --first seems easy to remember jgleeson [17:24:29] actually it's --max-count [17:24:41] yeah, that one takes an arg [17:24:44] (03CR) 10Ejegg: [C: 03+2] Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [17:24:46] a lil more work for now [17:24:55] cool I'll go with first, thanks [17:25:23] 10Fundraising-Backlog: Greece: Set Adyen with post-payment monthly convert as Default - https://phabricator.wikimedia.org/T349122 (10HNordeenWMF) [17:42:13] (03Merged) 10jenkins-bot: Save contact external_identifier in Custom field. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966267 (https://phabricator.wikimedia.org/T349015) (owner: 10Damilare Adedoyin) [17:47:41] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Internal test: for validate all dlocal submethods - https://phabricator.wikimedia.org/T346814 (10RKumar_WMF) For CL - When donor is trying to use Bank transfer, he is facing below issue: "Hi! I've tried, but it doesnt work with bank transfer, when I clic... [17:56:27] damilare: before we deploy that, let's delete the existing custom group/field so we end up with one that has a predictable name [17:56:44] (i.e. with no id numbers in the schema) [17:57:07] fr-tech I'm gonna miss some of tech-talk... gotta cook! back later [17:57:20] annoying paypal email sent [17:59:50] ejegg: do you mean in production?? [18:01:14] yep damilare [18:02:59] okk [18:05:20] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Internal test: for validate all dlocal submethods - https://phabricator.wikimedia.org/T346814 (10RKumar_WMF) Mexico - Bank transfer option is not showing up on the donation form: Payment methods shown are: Donate with a debit/credit card Cash PayPal Goo... [18:05:55] (03PS4) 10Ejegg: Regenerate SmashPig Civix files [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966566 (https://phabricator.wikimedia.org/T349107) [18:05:57] (03PS3) 10Ejegg: Create new field for Adyen Shopper Reference [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966557 (https://phabricator.wikimedia.org/T349107) [18:05:59] (03PS3) 10Ejegg: Copy invoice_id to processor_contact_id [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966579 (https://phabricator.wikimedia.org/T349107) [18:06:09] (03PS1) 10Ejegg: WIP store and retrieve ShopperReference from new field [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966590 (https://phabricator.wikimedia.org/T349107) [18:07:27] done [18:07:36] thanks! [18:11:11] 10Fundraising Sprint This LAN is your LAN, 10Fundraising-Backlog, 10Patch-For-Review: Removing some WMF donor fields - https://phabricator.wikimedia.org/T347724 (10KHaggard) Thanks @Eileenmcnaughton . Yes, just confirming that any fields currently in the export file that are being removed / changed / added s... [18:17:20] ugh, what do we do with the FRUP ID when we merge two contacts that have come in through FRUP? [18:28:43] (03PS2) 10Ejegg: Store and retrieve ShopperReference from new field [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966590 (https://phabricator.wikimedia.org/T349107) [18:42:28] (03PS1) 10Damilare Adedoyin: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/966601 [18:42:30] (03CR) 10Ejegg: [C: 04-1] "Ready for review, but not yet for deploy - want to do that when the follow-on patches are ready too and we can stop queues and charge job." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966579 (https://phabricator.wikimedia.org/T349107) (owner: 10Ejegg) [18:43:17] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/966601 (owner: 10Damilare Adedoyin) [18:45:38] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/966601 (owner: 10Damilare Adedoyin) [18:49:59] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Handle Adyen refund IPNs for non-USD donations - https://phabricator.wikimedia.org/T347422 (10Ejegg) [18:57:30] (03PS1) 10Ejegg: Update instructions for syncing custom fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966626 [19:14:50] (03CR) 10Damilare Adedoyin: [C: 03+2] Update instructions for syncing custom fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966626 (owner: 10Ejegg) [19:30:49] (03Merged) 10jenkins-bot: Update instructions for syncing custom fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966626 (owner: 10Ejegg) [20:01:50] 10Wikimedia-Fundraising-Banners, 10Wikipedia-Android-App-Backlog (Android Release - FY2023-24), 10Wikipedia-iOS-App-Backlog (Design), 10iOS Release FY2023-24: JSON for new app fundraising banners (multilingual, Italy and Netherlands) - https://phabricator.wikimedia.org/T348278 (10Tsevener) [20:02:04] 10Wikimedia-Fundraising-Banners, 10Wikipedia-iOS-App-Backlog, 10iOS Release FY2023-24: Netherlands fundraising messages for iOS app - https://phabricator.wikimedia.org/T344516 (10Tsevener) [20:59:04] (03PS4) 10Ejegg: Scripts for doing token migration in the db [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/965844 (https://phabricator.wikimedia.org/T344645) [21:02:18] 10Wikimedia-Fundraising-Banners, 10Wikipedia-Android-App-Backlog (Android Release - FY2023-24), 10Wikipedia-iOS-App-Backlog (Design), 10iOS Release FY2023-24: JSON for new app fundraising banners (multilingual, Italy and Netherlands) - https://phabricator.wikimedia.org/T348278 (10ABorbaWMF) Looks good on A... [21:24:33] (03CR) 10Jgleeson: "Am I right in thinking this change coupled us to the smashpig extension? Is there a way for us to mark in the RQC that the smashpig extens" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966590 (https://phabricator.wikimedia.org/T349107) (owner: 10Ejegg) [21:29:13] 10Fundraising Sprint This LAN is your LAN, 10Fundraising-Backlog, 10MediaWiki-extensions-FundraiserLandingPage, 10donate.wikimedia.org: donate.wikimedia.org landing pages don't have descriptions in search engine results - https://phabricator.wikimedia.org/T254808 (10EWilfong_WMF) I'll defer to Peter on how... [21:32:23] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10jgleeson) I'd go with medium @greg, as it looks like it needs user error to trigger it. @ejegg figured out that if you put invalid contact IDs into the search it defaults to a group... [21:33:25] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10greg) p:05High→03Medium [23:07:50] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10Eileenmcnaughton) Just an update to say this is an intermittent error - I have replicated elsewhere but it is now not happening on prod https://lab.civicrm.org/dev/core/-/issues/4705... [23:40:45] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10Eileenmcnaughton) Ok - not quite intermittent - it depends whether you want for the js action to resolve a contact or hit enter before it has - in the latter case it goes wrong [23:43:38] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: CiviCRM gid profile error - https://phabricator.wikimedia.org/T349091 (10Ejegg) It also seems to trigger if you paste a valid CID in and hit enter before the autocomplete comes back with a result. [23:49:34] (03PS4) 10Ejegg: Remove failed attempt at re-saving weight [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966323 (https://phabricator.wikimedia.org/T335070) (owner: 10Eileen) [23:49:41] (03CR) 10Ejegg: [C: 03+2] "Looks good!" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966323 (https://phabricator.wikimedia.org/T335070) (owner: 10Eileen) [23:57:46] (03PS5) 10Ejegg: Regenerate SmashPig Civix files [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/966566 (https://phabricator.wikimedia.org/T349107)