[00:01:43] (03PS1) 10Lars SG: Speed up Deduper testEmailHasCustomDataConflict test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299655 (https://phabricator.wikimedia.org/T406735) [00:11:33] eileen: not sure i follow you exactly, the ids array is per test and we want to avoid re-creating the payment processor for each test as that's what's expensive. [00:13:11] hmm - so we should be using setupBeforeClass & tearDownAfterClass more - or should we just create them as part of our build data? [00:18:06] Well, in this case this is used in three different test classes with three different payment processor names. So we need a way to handle that here, but in other cases I think maybe it could be in setupHeadless. [00:18:50] ok - I don't want to make to big a thing on this one - I just generally don't like having multiple ids arrays on test classes [00:21:44] totally fair in general, I just don't see a different way to handle it here. I guess we could do a different setupHeadless for each one. Or we could just make the test processor always have the same name across all the tests, unless it matters somewhere specifically in there [00:23:54] (03CR) 10Eileen: [C:03+2] Speed up SmashPig tests by not deleting PaymentProcessor each test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299633 (https://phabricator.wikimedia.org/T406735) (owner: 10Lars SG) [00:24:40] (03CR) 10Eileen: [C:03+2] Speed up Deduper testEmailHasCustomDataConflict test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299655 (https://phabricator.wikimedia.org/T406735) (owner: 10Lars SG) [00:25:13] well - let's take the win huh - I think the custom group install is also the slowness on the install [00:25:18] thanks eileen [00:25:43] (03PS2) 10Lars SG: Speed up Deduper testEmailHasCustomDataConflict test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299655 (https://phabricator.wikimedia.org/T406735) [00:26:11] that would make sense [00:31:04] 06Fundraising-Backlog, 10fundraising-tech-ops: Make the full list of languages available to custom fields on Community CiviCRM instance - https://phabricator.wikimedia.org/T343488#12002924 (10Dwisehaupt) 05Open→03Resolved a:03Dwisehaupt Given that it's almost 2 years since the last update, I'm going... [00:33:09] 10fundraising-tech-ops: Switch over to new payments servers - https://phabricator.wikimedia.org/T424582#12002929 (10Dwisehaupt) Looking to possibly tie this in with the switch to trixie. We can move forward now on bookworm or hold until the code is fully tested on trixie and php8.4. [00:33:30] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 07Epic, 13Patch-For-Review: CiviCRM Deployments taking over an hour to merge on CI. Let's speed it up! - https://phabricator.wikimedia.org/T406735#12002931 (10Lars) The above cut down nearly 3 minutes, which isn't a tonne, but better than nothing. I'm... [00:40:45] (03CR) 10Cstone: [C:03+2] "thanks for this, I was able to delete the pix token in the sandbox" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299501 (https://phabricator.wikimedia.org/T424245) (owner: 10Wfan) [00:40:59] (03CR) 10Cstone: [C:03+2] Add test for Gravy Pix subscription cancellation [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1295957 (https://phabricator.wikimedia.org/T424245) (owner: 10Wfan) [00:57:59] 10fundraising-tech-ops: Improve fundraising kafkatee monitoring - https://phabricator.wikimedia.org/T349249#12002946 (10Dwisehaupt) 05Open→03Resolved a:03Dwisehaupt Added new `KafkaJumboNoData` alert which tracks `rate(kafkatee_broker_topic_rxbytes{broker=~"kafka-jumbo.*"}[60m])`. This will alert if we... [00:59:22] (03PS1) 10Lars SG: Add time logging for query in set time zone [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299659 [01:03:12] 06Fundraising-Backlog, 10fundraising-tech-ops: Get coworker running on fundraising staging server - https://phabricator.wikimedia.org/T363928#12002953 (10Dwisehaupt) Capturing the fact that we set the queue release time into the future. This was done in commits but not added to the task: ` commit d04d7e92e1a80... [01:05:57] 10fundraising-tech-ops: Capture traceroute information on external endpoints for post failure diagnostics - https://phabricator.wikimedia.org/T263204#12002954 (10Dwisehaupt) 05Open→03Declined This hasn't been an issue in practice and in most cases we get the data we need from existing methods. The endpoi... [01:06:28] (03Merged) 10jenkins-bot: Return recurring token for gravy redirect methods when cancel recurring [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299501 (https://phabricator.wikimedia.org/T424245) (owner: 10Wfan) [01:06:53] (03Merged) 10jenkins-bot: Add test for Gravy Pix subscription cancellation [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1295957 (https://phabricator.wikimedia.org/T424245) (owner: 10Wfan) [03:20:59] (03PS1) 10Lars SG: Try with rpow master override [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299668 [03:22:45] 03Fundraising Sprint: Jack's Junk Drawer, 10Wikimedia-Fundraising-CiviCRM: Diagnose & resolve Civi 6.16 build speed issue - https://phabricator.wikimedia.org/T428686#12003096 (10Lars) I tried out some logging to see if there was something going wrong at that point, but what I got was 11 minute gap in a differe... [03:30:00] 03Fundraising Sprint: Jack's Junk Drawer, 10Wikimedia-Fundraising-CiviCRM: Diagnose & resolve Civi 6.16 build speed issue - https://phabricator.wikimedia.org/T428686#12003097 (10Lars) Also interesting to note that this step was already taking 2.5+ minutes on 6.13 [03:43:01] (03PS10) 10Eileen: Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) [03:43:01] (03PS1) 10Eileen: Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 [03:47:26] Thanks Christine! [03:54:48] (03PS1) 10Lars SG: Try with rpow debug [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299670 [03:56:15] eileen: still not sure what's going on with the build speed issue, but at least the tests are reliably not timing out now with 6.16, so there's that [03:56:46] larssandergreen: yeah I'm in a rabbit hole - but that is good! [04:09:05] eileen: are you looking at the rpow install? If not, I can take a look in the am [04:09:59] larssandergreen: im about to give up for the night but have you ever run into docker containers refusing to stop [04:10:21] no, that sounds spooky, zombie containers [04:10:30] yeah hah tomorrows zombie issue then [04:40:50] (03PS1) 10Eileen: See if we can get the timings on jenkins too [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299674 [04:40:50] (03PS1) 10Eileen: Test not doing the slow stuff [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 [04:46:21] larssandergreen: what is more spooky is that I started a new extension locally called 'spook' a couple of hours back and when I next checked irc you seem to be already referring to it [04:46:37] I think I found the slow down [05:28:33] (03CR) 10CI reject: [V:04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_44) - 10https://gerrit.wikimedia.org/r/1299750 (owner: 10L10n-bot) [06:25:37] (03PS2) 10Eileen: Skip slow hooks on custom field create for now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 (https://phabricator.wikimedia.org/T428686) [08:53:21] (03PS1) 10Wfan: Upgrade SmashPig to v1.2.4.13 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300046 [08:54:12] (03CR) 10Wfan: [C:03+2] Upgrade SmashPig to v1.2.4.13 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300046 (owner: 10Wfan) [09:16:30] (03Merged) 10jenkins-bot: Upgrade SmashPig to v1.2.4.13 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300046 (owner: 10Wfan) [09:38:21] (03PS1) 10Wfan: Smashpig [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1300062 [09:39:09] (03CR) 10Wfan: [V:03+2 C:03+2] Smashpig [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/1300062 (owner: 10Wfan) [09:39:55] (03PS1) 10Wfan: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1300063 [09:40:15] (03CR) 10Wfan: [V:03+2 C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/1300063 (owner: 10Wfan) [10:28:12] (03PS1) 10Jgleeson: Skip Base62 conversions for Adyen CaptureFailed webhooks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1300081 (https://phabricator.wikimedia.org/T428042) [10:28:22] we got another one damilare ^ [10:32:12] (03CR) 10Jgleeson: "Should we go back to checking this across them all rather than having subtype specifics?" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1300081 (https://phabricator.wikimedia.org/T428042) (owner: 10Jgleeson) [10:41:42] 06Fundraising-Backlog, 10fundraising-tech-ops, 06SRE Observability: Shift frack alerting to use prometheus-alertmanager instead of icinga - https://phabricator.wikimedia.org/T367370#12004052 (10hnowlan) [10:50:11] (03CR) 10Jgleeson: "Maybe not. For refunds and chargebacks, the ID needs to exist, so we can't return null as we're doing for capture handling. We could explo" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1300081 (https://phabricator.wikimedia.org/T428042) (owner: 10Jgleeson) [11:08:48] (03CR) 10Damilare Adedoyin: [C:03+2] Skip Base62 conversions for Adyen CaptureFailed webhooks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1300081 (https://phabricator.wikimedia.org/T428042) (owner: 10Jgleeson) [11:09:26] (03Merged) 10jenkins-bot: Skip Base62 conversions for Adyen CaptureFailed webhooks [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/1300081 (https://phabricator.wikimedia.org/T428042) (owner: 10Jgleeson) [11:09:33] ty damilare [11:09:55] np good fix, I think its a good idea to ask gravy why we are seeing this now [11:10:41] (03PS1) 10Jgleeson: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1300095 [11:10:48] right damilare [11:12:24] (03CR) 10Jgleeson: [C:03+2] Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1300095 (owner: 10Jgleeson) [11:12:54] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/SmashPig] (deployment) - 10https://gerrit.wikimedia.org/r/1300095 (owner: 10Jgleeson) [11:36:54] !log SmashPig upgraded from f6b24f7f to e82c2c5f [11:36:55] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [12:07:12] (03PS1) 10Damilare Adedoyin: Add problem login email subject to DonorPortal login help link [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1300106 (https://phabricator.wikimedia.org/T426345) [12:20:41] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy Recurring: Validation error during recurring charge, in field: processor_contact_id. Message: Input should be a valid UUID - https://phabricator.wikimedia.org/T424213#12004478 (10jgleeson) 05Resolved→03Open It looks like we have a few more of th... [12:47:49] (03CR) 10Jgleeson: [C:03+2] "LGTM!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1300106 (https://phabricator.wikimedia.org/T426345) (owner: 10Damilare Adedoyin) [12:49:06] thanks! [12:49:52] (03Merged) 10jenkins-bot: Add problem login email subject to DonorPortal login help link [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1300106 (https://phabricator.wikimedia.org/T426345) (owner: 10Damilare Adedoyin) [12:50:23] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10FR-Donor-portal, 13Patch-For-Review: [Donor Portal] Update subject line in 'Problems logging in?' dialog box (hardcoded) - https://phabricator.wikimedia.org/T426345#12004557 (10Damilare) [12:56:14] np [13:06:41] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy Recurring: Validation error during recurring charge, in field: processor_contact_id. Message: Input should be a valid UUID - https://phabricator.wikimedia.org/T424213#12004590 (10jgleeson) p:05Triage→03Medium a:05AnnWF→03jgleeson [13:07:03] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog: Gravy Recurring: Validation error during recurring charge, in field: processor_contact_id. Message: Input should be a valid UUID - https://phabricator.wikimedia.org/T424213#12004593 (10jgleeson) Ok, looks like one got cancelled already, so I've updated th... [13:09:46] fr-tech, we had another 9 gravy recurring charges with their processor_contact_id set to an invalid value. We fixed this last time we saw them, but a few new ones seem to have appeared. 1 of the 9 was cancelled due to max retires. I've fixed the other 8. These are related to the "Validation error during recurring charge, in field: processor_contact_id. Message: Input should be a valid UUID, invalid character: expected an [13:09:46] optional prefix of `urn:uuid:`" failmails. [13:10:07] original ticket is here https://phabricator.wikimedia.org/T424213 [13:13:26] (03PS1) 10Jgleeson: Test that payment_service_id is saved to contribution_recur_smashpig on import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299636 [13:16:51] (03CR) 10Damilare Adedoyin: [C:03+2] Test that payment_service_id is saved to contribution_recur_smashpig on import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299636 (owner: 10Jgleeson) [13:18:08] thanks damilare [13:18:49] np! [13:20:28] (03PS2) 10Damilare Adedoyin: Extend docker compose services in docker with volume mount [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1294423 [13:20:28] (03PS3) 10Damilare Adedoyin: Remove Civi DB backups from install scripts [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1298316 [13:22:12] (03Abandoned) 10Damilare Adedoyin: Add certificates for mysql ssl [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1297182 (owner: 10Damilare Adedoyin) [13:38:57] (03Merged) 10jenkins-bot: Test that payment_service_id is saved to contribution_recur_smashpig on import [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299636 (owner: 10Jgleeson) [13:45:26] jgleeson: what's the source_type on those? [13:45:46] I think we fixed it via one route, but there may be another route to fix [13:46:07] i.e. recurrings saved by IPN listener might be saving the wrong value [13:49:10] damilare: I'm looking at the 'extend' patch, and not having a mac with me to test it on, i'm just curious [13:49:21] how the extra civi volumes get populated [13:50:09] Ah, I guess on the initial run the volumes are empty, but then they are mounted to the directories where the other setup scripts will check stuff out [13:50:15] right, that makes sense [13:54:20] yea the volumes just replace the bind mounts and the setup script populates them during the setup process when run with --use-volume-mount [13:55:48] (03PS2) 10Damilare Adedoyin: Set Original country to billing_address_country if present for DonationQueueConsume [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1298914 [13:56:10] (03PS6) 10Damilare Adedoyin: Ensure the session country is saved to the billing_address_country field to be used in subsequent recurring payments. [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/1298912 (https://phabricator.wikimedia.org/T428465) [14:02:35] ejegg: good question. I'll dig into it when I get back [14:15:19] Fr-tech anyone else run into docker containers that won't stop ? Jack and I had fixed this before (I thought) but last night none of them would stop when I tried to rebuild, or delete [14:17:42] (03PS9) 10Ejegg: Add fundraising-ml db setup and config [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/1295075 [14:18:07] weird cstone [14:18:23] i would try the cli commands with --force [14:19:03] weird cstone, was this on a mac? [14:19:05] I'm wondering now if it's docker itself I even completely deleted the dev folder last night and tried a rebuild and it still happened [14:19:09] No this is on ubuntu [14:19:15] like docker container stop -t 1 XXX [14:19:26] (-t 1 is wait 1 sec before killing it outright) [14:19:57] then docker container rm -f XXX [14:20:03] ejegg: what version of docker do you have [14:20:13] Docker version 29.5.2, build 79eb04c [14:21:19] let's see, I think that's docker-ce from the debian repo [14:21:45] yeah [14:22:20] so the containers themselves live in /var/lib/docker and are owned by root [14:22:52] if you really need to kill everything you could 'service docker stop' [14:23:09] (also might hang) [14:23:35] okay mine is Docker version 29.5.3, build d1c06ef [14:23:48] gotta be basically the same as mine then [14:24:08] lemme see if I get that on a package upgrade [14:24:36] docker service stop doesnt stop them, last night i made sure the user was in the group and that the user id was right in the dev [14:24:43] lemme try a random other container [14:24:47] docker-ce is on the list, just gotta get 300MB of stuff with all the other updates [14:25:11] cstone and docker rm -f [14:25:13] ? [14:25:44] note no 'compose' so you need the fd- prefix on the container name [14:28:10] still permission issues [14:28:10] Error response from daemon: cannot remove container "fundraising-dev-queues-1": could not kill container: permission denied [14:28:20] whoa! [14:28:30] and if you do it with sudo? [14:29:06] same error with sudo [14:29:27] i was going to try just uninstall docker completely? [14:29:47] hah, i wouldn't go that far [14:30:01] well, i guess it might help with permissions... [14:30:34] but it would still have to stop the containers to uninstall the service [14:30:39] if its not the dev project i did something weird to setup i figured maybe i did a weird thing installing it in the first place? [14:31:07] you installed it from the package manager, right? [14:31:33] i don't remember what I did but probably? [14:33:13] i can stop them with kill [14:33:44] oh nice [14:34:03] then try docker container rm -f ? [14:34:35] well that removes them too [14:35:39] oh and you just wanted to stop it without removing? [14:35:47] no [14:36:02] but if its gone would i need to still docker container rm? [14:36:09] ahhh no i guess not [14:36:19] i thought kill just stopped the process [14:36:35] hmm maybe [14:36:39] its gone from ps [14:36:48] try docker container ls [14:37:05] they are gone from ls [14:37:17] ok, sounds like they're really gone then [14:38:15] huh now the mail catcher container i had to use sudo to kill [14:38:33] im just going to completely reinstall this [14:38:47] gl! [14:38:56] I just updated to the same version of docker [14:39:02] will let you know if I see any issues [14:39:08] hopefully not [14:39:10] hah [14:43:29] (03Abandoned) 10Lars SG: Try with rpow debug [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299670 (owner: 10Lars SG) [14:44:00] i see gemini now has a "use code with caution" disclaimer [14:48:17] (03CR) 10Lars SG: [C:03+2] Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 (owner: 10Eileen) [14:58:42] (03CR) 10Lars SG: [C:03+2] "Agree, we can definitely live without these for now" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 (https://phabricator.wikimedia.org/T428686) (owner: 10Eileen) [15:47:22] (03PS1) 10Lars SG: Speed up AdyenAuditTest by not creating unneeded entities for every test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300190 (https://phabricator.wikimedia.org/T406735) [16:03:29] (03PS2) 10Lars SG: Speed up AdyenAuditTest by not creating unneeded entities for every test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300190 (https://phabricator.wikimedia.org/T406735) [16:04:40] !log civicrm upgraded from d7113e04 to 819c4ede [16:04:41] Logged the message at https://wikitech.wikimedia.org/wiki/Fundraising/SAL [16:41:58] (03PS1) 10Lars SG: Fix PHP warnings in tests [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300203 [16:42:33] 06Wikimedia-Fundraising: test phab task for fr-email-builder - https://phabricator.wikimedia.org/T428469#12005775 (10ppenloglou) 05Open→03Resolved [16:57:21] 06Wikimedia-Fundraising: Highlight style when brand is "recurring" - https://phabricator.wikimedia.org/T428781 (10ppenloglou) 03NEW [16:58:24] 06Wikimedia-Fundraising: Highlight style when brand is "recurring" - https://phabricator.wikimedia.org/T428781#12005896 (10ppenloglou) p:05Triage→03Low [17:24:43] (03CR) 10Jgleeson: [C:03+2] Fix PHP warnings in tests [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300203 (owner: 10Lars SG) [17:36:00] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 07payments-orchestration: After July 2nd, remove suppression of Adyen error 65 for gravy recurring charges - https://phabricator.wikimedia.org/T428787 (10Ejegg) 03NEW [17:37:11] 06Wikimedia-Fundraising: Wrong root variable name reference. - https://phabricator.wikimedia.org/T428788 (10ppenloglou) 03NEW [17:37:27] 06Wikimedia-Fundraising: Wrong root variable name reference. - https://phabricator.wikimedia.org/T428788#12006139 (10ppenloglou) p:05Triage→03Low [17:49:52] (03Merged) 10jenkins-bot: Fix PHP warnings in tests [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300203 (owner: 10Lars SG) [17:54:40] 06Wikimedia-Fundraising: Highlight style when brand is "recurring" - https://phabricator.wikimedia.org/T428781#12006215 (10ppenloglou) [18:04:16] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-payments-general, 10FR-Smashpig, 07payments-orchestration: After July 2nd, remove suppression of Adyen error 65 for gravy recurring charges - https://phabricator.wikimedia.org/T428787#12006250 (10FR-Tech-Request-Bot) [18:04:54] 06Fundraising-Backlog, 10FR-Donor-portal, 10FR-email-preferences: Preference Centre - Make unsubscribed status clear for user - https://phabricator.wikimedia.org/T427288#12006253 (10FR-Tech-Request-Bot) [18:05:51] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Clean up emails where contact has two of the same type and address after backfilling location types - https://phabricator.wikimedia.org/T425190#12006255 (10Lars) @AMJohnson These have been cleaned up (fully this time). I... [18:06:54] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-payments-general, 07payments-orchestration: After July 2nd, remove suppression of Adyen error 65 for gravy recurring charges - https://phabricator.wikimedia.org/T428787#12006263 (10Ejegg) I don't think we need to do any updates to the SmashPig li... [18:14:51] jgleeson: if you still want to look at my cursed setup im around [18:15:08] but its having issues with rpow i think [18:15:19] sure i'll just grab a coffee and jump on a call [18:16:43] jgleeson: i made this https://meet.google.com/zjm-nnkq-zox?authuser=0 [19:32:05] (03CR) 10Lars SG: [C:03+2] Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) (owner: 10Eileen) [19:36:00] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Change default theme on our local CiviCRM - https://phabricator.wikimedia.org/T428804 (10Lars) 03NEW [19:58:53] 10Wikimedia-Fundraising-CiviCRM, 10FR-Donor-portal: New Consent Message on front page of Donor Portal - https://phabricator.wikimedia.org/T420577#12006773 (10SBurnett-WMF) 05Open→03Resolved no longer needed [20:03:14] (03CR) 10CI reject: [V:04-1] Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) (owner: 10Eileen) [20:03:16] (03CR) 10CI reject: [V:04-1] Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 (owner: 10Eileen) [20:03:17] (03CR) 10CI reject: [V:04-1] Skip slow hooks on custom field create for now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 (https://phabricator.wikimedia.org/T428686) (owner: 10Eileen) [20:08:39] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Donor-portal, 0725 Dec Q3 Planning: On give less page - no error message when inputting a gift amount too low - https://phabricator.wikimedia.org/T413111#12006823 (10SBurnett-WMF) PLease have text ready that includes donation threshold before working... [20:09:22] 03Fundraising Sprint: Jack's Junk Drawer, 06Fundraising-Backlog, 10FR-Donation-Processing, 0725 Dec Q3 Planning: Chariot DAFpay button integration - donation form - https://phabricator.wikimedia.org/T405043#12006829 (10XenoRyet) [20:13:06] 06Fundraising-Backlog, 07Fundraising-ComboWiki-Dependency, 06Wikimedia-Fundraising, 10MediaWiki-extensions-DonationInterface: Donors need an option to pay from another country - https://phabricator.wikimedia.org/T96047#12006842 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF I believe this functi... [20:14:05] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10MediaWiki-extensions-CentralNotice, 06SRE, 06Traffic: Set expiry time for GeoIP cookies - https://phabricator.wikimedia.org/T122097#12006862 (10AKanji-WMF) @Pcoombe could you please advise as to whether this is something we should/can resolve in the ne... [20:15:09] 06Fundraising-Backlog, 07Fundraising-ComboWiki-Dependency, 06Fundraising-Tech-Roadmap, 10donate.wikimedia.org, and 3 others: Provide a donate skin for donate.wikimedia.org - https://phabricator.wikimedia.org/T257862#12006866 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF Resolving based on @Pcoo... [20:16:29] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap: Add "one-time" to header on One-time Payments Wiki - https://phabricator.wikimedia.org/T269897#12006869 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF I'm pulling this into the ComboWiki PRD for consideration, and closing this. [20:16:59] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Docker, 0725 Dec Q3 Planning: Docker dev setup: add anonymous Civi user - https://phabricator.wikimedia.org/T282707#12006873 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF Assuming this is resolved @Eileenmcnaughton [20:18:18] 14Fundraising Sprint Can't touch $this, 14Fundraising Sprint Fish HEAD^, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, and 2 others: Docker dev setup: scope out use cases, requirements and command-line UI to replace setup.sh - https://phabricator.wikimedia.org/T287028#12006879 (10AKanji-WMF) 05Op... [20:18:29] 03Fundraising Sprint: Jack's Junk Drawer, 06Fundraising-Backlog, 07fr-current-sprint, 05FY25-26 WE3.5 Donor Identification and recognition: CiviCRM is connected to MediaWiki - https://phabricator.wikimedia.org/T416950#12006885 (10Jdlrobson-WMF) 05Open→03Resolved [20:19:58] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Adyen, 0725 Dec Q3 Planning, 07fr-combowiki: Optimization of Apple Pay Flow - https://phabricator.wikimedia.org/T295092#12006895 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF This is implemented AFAIK, adding note to ComboWiki PRD to sur... [20:20:37] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM, 0725 Dec Q3 Planning: Switch CiviCRM mailing to use smtp fallback, consolidate our smtp use across our codebase - https://phabricator.wikimedia.org/T297647#12006899 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF... [20:24:46] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM: Email edit ui: allow users to send draft MC emails to themselves - https://phabricator.wikimedia.org/T302444#12006910 (10AKanji-WMF) Is this still a requirement, and if so when is our next campaign where we'd want to use this... [20:27:44] 06Fundraising Tech - Chaos Crew, 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Gpay-Integration, and 2 others: Handle donor-cancelled GPay login/card selection - https://phabricator.wikimedia.org/T306814#12006929 (10AKanji-WMF) Old chaos - please review if still relevant or close. [20:32:10] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Smashpig, 0725 Dec Q3 Planning: PredisCache should serialize/deserialize values - https://phabricator.wikimedia.org/T318638#12006958 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF Looks like the patch does do the serialize/reserialize! [20:32:58] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM: Email edit ui: allow users to send draft MC emails to themselves - https://phabricator.wikimedia.org/T302444#12006962 (10Lars) @AKanji-WMF This is in our SOW with the Civi core team and is in progress, so I think we'll have i... [20:33:53] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Smashpig, 0725 Dec Q3 Planning: Rationalize use of ApiException vs PaymentProviderResponse error properties across all providers - https://phabricator.wikimedia.org/T323740#12006965 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF Superseded... [20:36:25] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM, 10FR-CiviCRM-Core-Team: Email edit ui: allow users to send draft MC emails to themselves - https://phabricator.wikimedia.org/T302444#12006985 (10Lars) I added a tag so we can keep track of the Core team work phabs. [20:36:33] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Smashpig, 0725 Dec Q3 Planning: PaymentProviders should be able to indicate required fields for each call - https://phabricator.wikimedia.org/T323966#12006988 (10AKanji-WMF) Is your view that Gravy covers this normalization requirement @Ejegg or is it... [20:37:45] (03PS11) 10Eileen: Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) [20:38:42] (03PS2) 10Eileen: Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 [20:38:58] (03PS3) 10Eileen: Skip slow hooks on custom field create for now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 (https://phabricator.wikimedia.org/T428686) [20:40:31] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig, 0725 Dec Q3 Planning: Implement cancelPayment for PayPal EC - https://phabricator.wikimedia.org/T324640#12007008 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:33] 06Fundraising Tech Chaos Holding Pen, 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout: Intermittent PayPal auth failures lead to bad payments-initial messages - https://phabricator.wikimedia.org/T400846#12007012 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:35] 06Fundraising Tech Chaos Holding Pen, 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig: Add PayPal IPN type for recurring_payment_outstanding_payment_failed - https://phabricator.wikimedia.org/T324510#12007010 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:37] 10Fundraising Refactor, 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig, 07payments-orchestration: Add Idempotency headers to PayPal EC API calls - https://phabricator.wikimedia.org/T317696#12007016 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:38] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-PayPal-ExpressCheckout, 10MediaWiki-extensions-DonationInterface: PayPal EC: Error 11581 - Profile description is Invalid - https://phabricator.wikimedia.org/T202644#12007021 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:39] 10Fundraising Refactor, 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig, 07payments-orchestration: Implement PayPal EC RefundTransaction in SmashPig - https://phabricator.wikimedia.org/T318882#12007018 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:41] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-PayPal-ExpressCheckout: Big batch of paypal_ec subscription payments with missing parent subscr_ids - https://phabricator.wikimedia.org/T181692#12007023 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:44] 10Fundraising Refactor, 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig, and 3 others: Move PayPal EC API calls to SmashPig - https://phabricator.wikimedia.org/T317245#12007014 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:48] 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout: Large recurring PayPal txn in Civi but no money collected - https://phabricator.wikimedia.org/T243627#12007025 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:54] 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10MediaWiki-extensions-DonationInterface: Merchant reference generation should have options for attempt number or API sequence - https://phabricator.wikimedia.org/T136733#12007027 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:40:58] 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout: Prevent users from accessing older paypal - https://phabricator.wikimedia.org/T165536#12007031 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:41:02] 06Fundraising-Backlog, 10FR-PayPal-ExpressCheckout: PayPal Express Checkout has problems when using a security key - https://phabricator.wikimedia.org/T149525#12007033 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:41:35] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Docker, 0725 Dec Q3 Planning: Docker: add Donut campaigns and banners - https://phabricator.wikimedia.org/T326968#12007039 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF Assuming this has been done by now. [20:42:06] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Docker, 0725 Dec Q3 Planning: Docker: sync mechanism for content from production wikis - https://phabricator.wikimedia.org/T326969#12007042 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF [20:43:01] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10a Wikimedia CLI, 10FR-Docker, 0725 Dec Q3 Planning: Docker: config to selectively enable/disable services - https://phabricator.wikimedia.org/T326972#12007046 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF I believe this has been superseded... [20:44:18] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Smashpig, 0725 Dec Q3 Planning: Discuss SmashPig IPaymentProvider Interface - https://phabricator.wikimedia.org/T327498#12007050 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF Please re-open if still relevant. [20:47:03] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 0725 Dec Q3 Planning: DI add a countdown to ask donor accept direct bt from their phone - https://phabricator.wikimedia.org/T334310#12007065 (10AKanji-WMF) 05Open→03Resolved a:03AKanji-WMF Closing! [20:49:59] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 0725 Dec Q3 Planning: Measure donation processing rate (once we decide what that is) - https://phabricator.wikimedia.org/T342013#12007088 (10AKanji-WMF) @JMando @Ejegg Relevant to our convos about identifying donation completion rate? [20:51:43] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 0725 Dec Q3 Planning: Batch refund recurring should cancel subscription - https://phabricator.wikimedia.org/T344750#12007100 (10AKanji-WMF) @AnnWF Do we still need to work on this? [20:52:13] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 0725 Dec Q3 Planning: braintree webhook for PaymentMethodCustomerDataUpdated - https://phabricator.wikimedia.org/T344935#12007103 (10AKanji-WMF) Checking in on this one too, @AnnWF [20:52:52] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Smashpig, 0725 Dec Q3 Planning: PaymentProviders should be able to indicate required fields for each call - https://phabricator.wikimedia.org/T323966#12007105 (10Ejegg) Hah, no, Gravy requires different params for different backends too and afaik does... [20:55:17] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10FR-Smashpig, 0725 Dec Q3 Planning: Rationalize use of ApiException vs PaymentProviderResponse error properties across all providers - https://phabricator.wikimedia.org/T323740#12007110 (10Ejegg) 05Resolved→03Open I think this is still a valid ticket... [20:55:21] 06Fundraising-Backlog, 06Fundraising-Tech-Roadmap, 10Wikimedia-Fundraising-CiviCRM, 10FR-Smashpig: Last Test - https://phabricator.wikimedia.org/T427647#12007113 (10AKanji-WMF) [21:05:51] 06Fundraising-Backlog, 10FR-Donor-portal, 06FR-donorrelations: Donor Portal/Civi visibility for Donor Relations - https://phabricator.wikimedia.org/T407782#12007176 (10SBurnett-WMF) Waiting for an upgrade from the Core team - [21:10:46] (03CR) 10Eileen: [C:03+2] Speed up AdyenAuditTest by not creating unneeded entities for every test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300190 (https://phabricator.wikimedia.org/T406735) (owner: 10Lars SG) [21:12:35] 06Fundraising-Backlog, 10FR-Donor-portal, 10FR-email-preferences: Preference Centre - Make unsubscribed status clear for user - https://phabricator.wikimedia.org/T427288#12007185 (10Lars) Part of the problem here is that while the unsub is pending, the user sees no state change at all until it has been proce... [21:13:29] 06Fundraising-Backlog, 10FR-Donor-portal, 10FR-email-preferences: Preference Centre - Make unsubscribed status clear for user - https://phabricator.wikimedia.org/T427288#12007202 (10AKanji-WMF) The other issue is that the language in the "unsubscribed" radio box is ambiguous. {F87856316} [21:17:42] 06Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Donor-portal: When a donor portal link is requested for an email that isn't a primary email in CiviCRM, send them some kind of email - https://phabricator.wikimedia.org/T426118#12007214 (10Lars) We should also indicate in the email in the single se... [21:33:24] (03Merged) 10jenkins-bot: Speed up AdyenAuditTest by not creating unneeded entities for every test [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300190 (https://phabricator.wikimedia.org/T406735) (owner: 10Lars SG) [21:45:09] (03CR) 10Lars SG: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 (https://phabricator.wikimedia.org/T428686) (owner: 10Eileen) [21:45:29] (03CR) 10Lars SG: "recheck" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 (owner: 10Eileen) [22:23:38] (03CR) 10Lars SG: [C:03+2] Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) (owner: 10Eileen) [22:24:14] (03CR) 10Lars SG: [C:03+2] Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 (owner: 10Eileen) [22:51:26] (03CR) 10Lars SG: [C:03+2] Skip slow hooks on custom field create for now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 (https://phabricator.wikimedia.org/T428686) (owner: 10Eileen) [22:54:53] (03CR) 10CI reject: [V:04-1] Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) (owner: 10Eileen) [22:54:55] (03CR) 10CI reject: [V:04-1] Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 (owner: 10Eileen) [22:54:56] (03CR) 10CI reject: [V:04-1] Skip slow hooks on custom field create for now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 (https://phabricator.wikimedia.org/T428686) (owner: 10Eileen) [23:04:55] (03CR) 10Lars SG: [C:03+2] Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) (owner: 10Eileen) [23:05:46] (03Abandoned) 10Eileen: See if we can get the timings on jenkins too [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299674 (owner: 10Eileen) [23:10:57] (03PS3) 10Eileen: Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 [23:11:03] (03CR) 10CI reject: [V:04-1] Skip slow hooks on custom field create for now [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299675 (https://phabricator.wikimedia.org/T428686) (owner: 10Eileen) [23:12:46] (03PS12) 10Eileen: Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) [23:13:39] (03CR) 10Eileen: "I just squashed the speed hack into the main commit - should finish quicker now. Also pulled in one minor upstream fix that was merged int" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) (owner: 10Eileen) [23:14:22] larssandergreen: ^^ I just squashed (means your +2 fell off but it should finish) [23:15:21] (it does mean we need to do more work to not lose the change if not fixed but it should be self-reminding....) [23:16:30] (03CR) 10Lars SG: [C:03+2] Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) (owner: 10Eileen) [23:17:18] (03PS1) 10Eileen: Toughen up our tests [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300269 [23:25:53] (03CR) 10Lars SG: [C:03+2] Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 (owner: 10Eileen) [23:31:40] (03CR) 10Lars SG: "Do we want notices and deprecations to cause test failures?" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1300269 (owner: 10Eileen) [23:36:26] (03Merged) 10jenkins-bot: Update Civi to 6.16rc [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1297264 (https://phabricator.wikimedia.org/T428122) (owner: 10Eileen) [23:45:33] (03Merged) 10jenkins-bot: Fix wrong type on cli_std_out_logger [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/1299669 (owner: 10Eileen)