[00:00:41] 10Fundraising-Backlog, 10FR-Ingenico, 10Epic: [Epic] Must open a new GlobalCollect account in order to escape Order ID apocalypse - https://phabricator.wikimedia.org/T121000 (10Ejegg) 05Open→03Declined We're doing them sequentially now, and have them in a new orderspace [00:00:45] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Ingenico, 10FR-Smashpig, 10MediaWiki-extensions-DonationInterface: Rename all references to GlobalCollect in code, to Ingenico. Determine if we need to update old database entries - https://phabricator.wikimedia.org/T123249 (10Ejegg) [00:03:50] oh hey, the bell has tolled [00:03:54] time to test! [00:04:26] k, that's an ingenico one completed happily [00:07:33] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster, 10Design-Systems-team-project, 10MediaWiki-extensions-CentralNotice, 10Patch-For-Review: Update CentralNotice admin UI not to use broken JQ multiselect widget - https://phabricator.wikimedia.org/T291431 (10AndyRussG) Hi Design System folks! Just... [00:09:48] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10Recurring-Donations, 10Technical-Debt: Use CiviCRM template contributions for recurring donations - https://phabricator.wikimedia.org/T291449 (10Ejegg) [00:10:46] hmm, --slow-start is always giving me ingenico [00:11:00] I think it starts with the oldest ones first [00:11:02] i guess it's going from earliest c_r.id to latest, so that would make sense [00:11:06] Yeah [00:11:07] heh yep [00:11:23] well, I guess we tested ours at least [00:11:43] and with that include_once we shouldn't hit any more class loading problems [00:12:37] dstrine: did you email that person from wmf uk ? [00:14:23] ejegg fr-tech what about just the CheckboxMultiselectInputWidget in a scrollable area with a list of selected items that automatically appears above or below? https://doc.wikimedia.org/oojs-ui/master/demos/?page=widgets&theme=wikimediaui&direction=ltr&platform=desktop#CheckboxMultiselectInputWidget [00:15:50] and I guess clear selection/select all buttons, and make the list of selected items able to say "all" or "all except this and that", like we have elsewhere in CN? [00:17:12] cool, that could be good enough AndyRussG [00:19:10] :) [00:19:39] (03CR) 10Ejegg: Replace obsolete jquery multiselect plugin with working one (031 comment) [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/722441 (https://phabricator.wikimedia.org/T291431) (owner: 10Ejegg) [00:20:41] 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface, 10Technical-Debt: Delete Globalcollect adapater, fold still-used code into Ingenico adapter - https://phabricator.wikimedia.org/T260725 (10Ejegg) [00:21:39] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Ingenico: Globalcollect recurring job config is screwy - https://phabricator.wikimedia.org/T117151 (10Ejegg) 05Open→03Resolved a:03Ejegg No longer using this separate job, have made most everything accept the command-line and config parameters [00:21:41] 10Fundraising Sprint William Shatner, 10Fundraising-Backlog, 10Fundraising-Backlog-Old, 10Wikimedia-Fundraising-CiviCRM, and 2 others: [Epic] Followup tasks after Civi 4.6 upgrade - https://phabricator.wikimedia.org/T116416 (10Ejegg) [00:23:33] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Tech-Analytics, 10Recurring-Donations: Recurring Payments Reporting - https://phabricator.wikimedia.org/T90630 (10Ejegg) @EYener does this have any interest beyond just historical for you? If not, would you please close it as 'declined' or 'inval... [00:24:11] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Skip tests when modules not enabled - https://phabricator.wikimedia.org/T88379 (10Ejegg) 05Open→03Declined [00:24:14] 10Fundraising Sprint Enya, 10Wikimedia-Fundraising, 10Wikimedia-Fundraising-CiviCRM, 10§ Fundraising Sprint Abba, and 3 others: Ressurect phpunit tests for the CRM modules - https://phabricator.wikimedia.org/T86686 (10Ejegg) [00:25:40] 10Fundraising-Backlog, 10Fundraising-Backlog-Old, 10Browser-Tests: Create unit and integration tests for Fundraising extensions to identify breaking MediaWiki changes - https://phabricator.wikimedia.org/T89404 (10Ejegg) 05Open→03Resolved a:03Ejegg Yay, we have unit tests now! We should still write som... [00:29:31] 10Fundraising-Backlog, 10Wikipedia-Android-App-Backlog, 10fr-donorservices: "No App" error from monthly convert on mobile - https://phabricator.wikimedia.org/T291199 (10Ejegg) >>! In T291199#7367131, @MBeat33 wrote: > 976704 provided more info, including a screenshot from when they tried the TY link Peter su... [00:34:35] cstone: well, the 'Undefined index' at least tells us the job is steadily chugging away :) [00:39:17] 10Wikimedia-Fundraising-CiviCRM, 10fundraising sprint Roller Toaster, 10FR-Adyen, 10FR-Smashpig, and 2 others: Update SmashPig + DonationInterface under CRM - https://phabricator.wikimedia.org/T291219 (10Ejegg) OK, this has been deployed and tested with a new Adyen CC recurring and a couple old Adyen CC re... [02:11:19] cstone: might be nice to get this out for Sandra - https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/civicrm/+/721467- Coleman did the fix & I reviewed it [02:21:45] (03PS1) 10Eileen: Fix bug where it attempts to create a log table for wmf_donor [wikimedia/fundraising/crm/civicrm] - 10https://gerrit.wikimedia.org/r/722482 (https://phabricator.wikimedia.org/T288721) [02:31:07] (03CR) 10Krinkle: Replace obsolete jquery multiselect plugin with working one (031 comment) [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/722441 (https://phabricator.wikimedia.org/T291431) (owner: 10Ejegg) [03:17:12] (03PS4) 10Eileen: Drop indexes on wmf_donor fields identified as not required to be searchable. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/719752 (https://phabricator.wikimedia.org/T288721) [03:28:14] (03PS4) 10Eileen: Add new wmf_donor fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/719163 (https://phabricator.wikimedia.org/T288721) [03:28:16] (03PS3) 10Eileen: Trigger updates [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/720909 (https://phabricator.wikimedia.org/T288721) [04:16:49] (03PS5) 10Eileen: Drop indexes on wmf_donor fields identified as not required to be searchable. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/719752 (https://phabricator.wikimedia.org/T288721) [04:47:26] (03PS6) 10Eileen: Drop indexes on wmf_donor fields identified as not required to be searchable. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/719752 (https://phabricator.wikimedia.org/T288721) [05:04:06] (03PS5) 10Eileen: Add new wmf_donor fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/719163 (https://phabricator.wikimedia.org/T288721) [05:04:34] (03PS4) 10Eileen: Trigger updates [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/720909 (https://phabricator.wikimedia.org/T288721) [05:35:36] (03PS1) 10Eileen: Fill new wmf_donor fields. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/722486 (https://phabricator.wikimedia.org/T288721) [05:40:06] (03PS2) 10Eileen: Fill new wmf_donor fields. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/722486 (https://phabricator.wikimedia.org/T288721) [05:59:06] (03PS1) 10Eileen: Extended reports update [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/722488 (https://phabricator.wikimedia.org/T290050) [06:11:36] (03Abandoned) 10Eileen: Move page ref to the template as it seems to be translated... [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/721242 (https://phabricator.wikimedia.org/T289617) (owner: 10Eileen) [06:12:09] (03PS3) 10Eileen: Update ca recurring failed message [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/721118 [06:17:36] 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 10MW-1.35-notes (1.35.0-wmf.40; 2020-07-07), 10Performance-Team (Radar): CentralNotice: Remove bannerController RL modules - https://phabricator.wikimedia.org/T224034 (10DannyS712) 05Open→03Resolved a:03AndyRussG Looks like this was done [06:17:44] 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 10Performance-Team (Radar): Reduce module registration overhead from CentralNotice - https://phabricator.wikimedia.org/T221805 (10DannyS712) [08:54:33] (03PS3) 10Eileen: Fill new wmf_donor fields. [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/722486 (https://phabricator.wikimedia.org/T288721) [12:20:53] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Tech-Analytics, 10Recurring-Donations: Recurring Payments Reporting - https://phabricator.wikimedia.org/T90630 (10EYener) 05Open→03Declined Thanks, @Ejegg, I haven't read through the comment thread thoroughly, but due to this being authored i... [13:42:32] PROBLEM - check_mysql on frdb1002 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 7318 [13:44:47] 10Fundraising-Backlog, 10FR-Tech-Analytics, 10acl*WMF-FR: Donor dupes - https://phabricator.wikimedia.org/T291439 (10EYener) [13:47:32] PROBLEM - check_mysql on frdb1002 is CRITICAL: SLOW_SLAVE CRITICAL: Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 1621 [13:52:38] RECOVERY - check_mysql on frdb1002 is OK: Uptime: 1871354 Threads: 11 Questions: 113408737 Slow queries: 953 Opens: 464574079 Flush tables: 1 Open tables: 200 Queries per second avg: 60.602 Slave IO: Yes Slave SQL: Yes Seconds Behind Master: 0 [14:09:39] 10Fundraising-Backlog, 10FR-Tech-Analytics, 10acl*WMF-FR: Donor dupes - https://phabricator.wikimedia.org/T291439 (10EYener) Thanks, all! This is great information to have. Realizing, of course, that we have no lead in this case, I think it's still informative to add some notes about this experience: - CID... [14:23:22] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster, 10MediaWiki-extensions-CentralNotice, 10JavaScript, 10Regression: Central notice projects and language choices not loading - https://phabricator.wikimedia.org/T291410 (10Zabe) p:05Unbreak!→03High This has been quickly fixed on meta. So I wou... [14:32:43] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Campaign Notification list - https://phabricator.wikimedia.org/T289478 (10RLewis) @Eileenmcnaughton is this working now or is it due to go into a sprint? The reason I ask is because we usually setup a donation email bot for when the major gifts annual ap... [15:37:19] hi fr-tech! [15:37:31] howdy [15:37:59] Hi [15:45:55] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising sprint Roller Toaster: Stripe import issues 9/16/21 - https://phabricator.wikimedia.org/T291220 (10jkim_wikimedia) Hooray, that worked! Thank you @Eileenmcnaughton !! [15:55:38] so I took Oscar to a local covid testing centre and got the result back which was negative but now his school is saying the rapid test result isn't good enough and he needs the more comprehensive test and the earliest I can get him in for one of those is tomorrow. [16:02:56] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Campaign Notification list - https://phabricator.wikimedia.org/T289478 (10DStrine) This is not in the current but it's next in line. [16:18:05] 10fundraising-tech-ops: Fundraising environment support for Debian 11 (Bullseye) - https://phabricator.wikimedia.org/T277184 (10Dwisehaupt) Builds mostly completed in the vm. build and pay-lvs are the 2 main roles that have not been tested. build is a bit complicated due to its role in puppet deploys. pay-lvs ha... [16:20:58] 10Fundraising-Backlog, 10Wikipedia-Android-App-Backlog, 10fr-donorservices: "No App" error from monthly convert on mobile - https://phabricator.wikimedia.org/T291199 (10MattCleinman) We discussed this in a team meeting this morning. Nothing has changed in how the app handles URLs. To help us get to the bot... [17:00:15] 10fundraising-tech-ops: Write alerts for missing or empty logfiles - https://phabricator.wikimedia.org/T87683 (10Dwisehaupt) 05Declined→03Resolved [17:00:17] 10fundraising-tech-ops, 10observability, 10Epic: [Epic] overhaul fundraising cluster monitoring - https://phabricator.wikimedia.org/T91508 (10Dwisehaupt) [17:02:36] dstrine: i'll be right there [17:14:45] 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface: Make it possible to turn off post-payments monthly convert by URL parameter - https://phabricator.wikimedia.org/T291507 (10Pcoombe) [17:23:19] 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface: Make it possible to turn off post-payments monthly convert by URL parameter - https://phabricator.wikimedia.org/T291507 (10Ejegg) For now, how about we just turn it off via the countries parameter and send the variant for when you want to turn it... [17:24:03] (03PS1) 10Pcoombe: Allow to turn off monthly convert with variant parameter [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/722662 (https://phabricator.wikimedia.org/T291507) [17:25:44] (03CR) 10jerkins-bot: [V: 04-1] Allow to turn off monthly convert with variant parameter [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/722662 (https://phabricator.wikimedia.org/T291507) (owner: 10Pcoombe) [17:28:18] 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface, 10Patch-For-Review: Make it possible to turn off post-payments monthly convert by URL parameter - https://phabricator.wikimedia.org/T291507 (10Pcoombe) @Ejegg I suppose that could work too, but we would need to make a bunch of changes to the de... [17:39:25] 10Fundraising-Backlog, 10fundraising-tech-ops, 10FR-Tech-Analytics: Create a table showing failed attempted recurring charges per month - https://phabricator.wikimedia.org/T291214 (10EYener) a:03EYener [17:56:18] !log payments-wiki revision is 23d0ffac66 [17:56:21] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [18:51:26] (03PS1) 10Ejegg: WIP pending resolver skeleton [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/722666 (https://phabricator.wikimedia.org/T291225) [18:52:16] 10Fundraising-Backlog, 10fundraising sprint Quantum Leapfrog, 10fundraising sprint Roller Toaster, 10fr-donorservices: Epic: Automatically send status 800 on prior donors stopped at 600 - https://phabricator.wikimedia.org/T276183 (10Ejegg) [19:28:38] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster, 10Design-Systems-team-project, 10MediaWiki-extensions-CentralNotice, 10Patch-For-Review: Update CentralNotice admin UI not to use broken JQ multiselect widget - https://phabricator.wikimedia.org/T291431 (10AndyRussG) Hi! Thanks @Ejegg for that p... [19:30:01] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster, 10Design-Systems-team-project, 10MediaWiki-extensions-CentralNotice, 10Patch-For-Review: Update CentralNotice admin UI not to use broken JQ multiselect widget - https://phabricator.wikimedia.org/T291431 (10DStrine) p:05Triage→03High [19:30:04] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster, 10FR-Smashpig: Add RiskScorer for Ingenico to SmashPig - https://phabricator.wikimedia.org/T291230 (10DStrine) p:05Triage→03Medium [19:30:09] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising sprint Roller Toaster: Update PaymentsInitQueueConsumer to leave FAILED/REJECTED pending rows for Ingenico - https://phabricator.wikimedia.org/T291228 (10DStrine) p:05Triage→03Medium [19:30:18] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising sprint Roller Toaster: Add existing donor matching to PendingTransaction::resolve - https://phabricator.wikimedia.org/T291226 (10DStrine) p:05Triage→03Medium [19:30:25] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising sprint Roller Toaster: Add PendingTransaction::resolve API call that replicates Ingenico orphan rectifier - https://phabricator.wikimedia.org/T291223 (10DStrine) p:05Triage→03Medium [19:30:46] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster, 10Recurring-Donations, 10fr-donorservices, 10Patch-For-Review: Civi: Auto Recurring Fail email, batch of copy corrections - https://phabricator.wikimedia.org/T289617 (10DStrine) p:05Triage→03Medium [19:30:52] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster: Disable post-payment (Ingenico) monthly convert during September 21 en6C pre-test - https://phabricator.wikimedia.org/T291429 (10DStrine) p:05Triage→03High [19:31:14] (03CR) 10AndyRussG: [C: 04-1] "Thanks much for this @Ejegg, and for the feedback @Krinkle!! Good to see this exploration... Apologies for the -1, just putting that there" [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/722441 (https://phabricator.wikimedia.org/T291431) (owner: 10Ejegg) [19:31:31] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster, 10fr-donorservices, 10Patch-For-Review: Civi Sidebar "recent items" --> no longer showing the top records - https://phabricator.wikimedia.org/T290767 (10DStrine) p:05Triage→03Medium [19:31:41] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising sprint Roller Toaster, 10Patch-For-Review: Add PendingTable::Consume API call that implements pending resolver outer loop - https://phabricator.wikimedia.org/T291225 (10DStrine) p:05Triage→03Medium [19:31:50] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10fundraising sprint Roller Toaster: Stripe import issues 9/16/21 - https://phabricator.wikimedia.org/T291220 (10DStrine) p:05Triage→03High [19:53:53] 10Fundraising-Backlog, 10Cloud-VPS (Project-requests): Request creation of fr-tech-dev VPS project - https://phabricator.wikimedia.org/T290098 (10AndyRussG) Thanks much @Andrew, @aborrero! We're working on updating the proposal--should be done soon! It's still a bit in flux. In summary, there are some things t... [20:13:31] gonna run an errand, back in a bit [20:20:03] ejegg|afk: I've added a reference to Apple Pay in the final bit of the proposal [20:20:28] fr-tech are folks ok with me submitting that update? [20:21:18] https://etherpad.wikimedia.org/p/fr-tech-dev-proposal here it is again in case the link got lost [20:27:09] Looks good to me, for my two cents. [20:27:58] 10Fundraising-Backlog, 10Wikipedia-Android-App-Backlog, 10fr-donorservices: "No App" error from monthly convert on mobile - https://phabricator.wikimedia.org/T291199 (10MBeat33) [20:29:26] 10Fundraising-Backlog, 10Wikipedia-Android-App-Backlog, 10fr-donorservices: "No App" error from monthly convert on mobile - https://phabricator.wikimedia.org/T291199 (10MBeat33) Than you @MattCleinman We will make sure to ask these donors for browser specs and app settings. [20:31:25] jgleeson: looks great, thanks for doing that! The only thing I might think to add is why it's an issue to set up on the FR cluster, e.g., limited public IP pool, more complex firefwall process with administrative overhead, PCI concerns... [20:34:01] yeah good point Andy let's see if we can work that in [20:34:09] AndyRussG* [20:34:27] oki cool thx! [20:35:46] maybe Jeff_Green dwisehaupt have details on that we'd like to mention? [20:49:13] 10Fundraising-Backlog, 10Editing-team, 10Performance-Team, 10Platform Engineering, and 5 others: RFC: Serve Main Page of Wikimedia wikis from a consistent URL - https://phabricator.wikimedia.org/T120085 (10Jdlrobson) [21:03:26] AndyRussG: I've added [21:03:28] We have deliberately kept this project outside the fundraising server cluster to allay any PCI concerns and remove the possibility of accidentally exposing donor data stored on the cluster. [21:04:14] jgleeson: lgtm! thx!!! [21:05:16] great. thanks for the suggestion also! [21:08:51] 10Fundraising-Backlog, 10Cloud-VPS (Project-requests): Request creation of fr-tech-dev VPS project - https://phabricator.wikimedia.org/T290098 (10jgleeson) [21:16:23] !log payments-wiki revision is 23d0ffac66 [21:16:26] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [21:18:27] 10Fundraising-Backlog, 10fundraising sprint Roller Toaster: Disable post-payment (Ingenico) monthly convert during September 21 en6C pre-test - https://phabricator.wikimedia.org/T291429 (10Cstone) Turned off for the test and has been renabled. [21:22:43] 10Fundraising-Backlog, 10Cloud-VPS (Project-requests): Request creation of fr-tech-dev VPS project - https://phabricator.wikimedia.org/T290098 (10jgleeson) [21:59:55] fr-tech if you want to do the pending transaction resolver in tech talk there's this patch we could start with: https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/722666 [22:04:16] (03PS2) 10Eileen: Ukranian text fix [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/721241 (https://phabricator.wikimedia.org/T289617) [22:33:49] 10Fundraising-Backlog, 10Wikipedia-Android-App-Backlog, 10fr-donorservices: "No App" error from monthly convert on mobile - https://phabricator.wikimedia.org/T291199 (10MBeat33) Actually, @MattCleinman could you let us know more specifically how donors can get the app settings? @KHancock99 did the long press... [22:51:36] (03PS1) 10AndyRussG: Close all tags in multiselect for jQuery update [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/722695 (https://phabricator.wikimedia.org/T291431) [23:45:16] 10Fundraising-Backlog, 10Wikipedia-Android-App-Backlog, 10fr-donorservices: "No App" error from monthly convert on mobile - https://phabricator.wikimedia.org/T291199 (10Dbrant) Breaking news: I've been able to reproduce this on one of my devices. However the actual cause is still a mystery. I'll keep invest... [23:50:58] ejegg: if you have time to talk through this silverpop a bit I think you wrote the logic I'm looking at [23:51:12] ut-oh :) [23:51:33] sure, one sec, let me commit a couple things [23:53:19] (03PS1) 10Eileen: Forget it [wikimedia/fundraising/tools] - 10https://gerrit.wikimedia.org/r/722702 (https://phabricator.wikimedia.org/T244811) [23:53:32] I think ^^ might be the fix? [23:53:35] (03PS2) 10Ejegg: Pending resolver outer loop + resolve skeleton [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/722666 (https://phabricator.wikimedia.org/T291225) [23:54:02] it just turns my head around the fact that things get suppressioned but... [23:55:28] ok, i have to read up on what this is solving [23:55:54] the issue is emails not being deleted out of silverpop tables when deleted [23:56:04] arr, ok [23:56:15] right, because we've got that running total [23:56:21] yeah [23:56:41] if you look at that patch the query is trying to do a delete but it's a bit messed up [23:56:51] and we can't even delete out of it on the forgetme action because it's in another DB way over there that we don't have write access to from the frontend [23:56:57] (also - the whole if not primary put on the master suppression list does my head in) [23:57:15] right - if we could just write to silverpop from civi... [23:57:35] - the query I changed it to is slow-ish on staging but I'm finding out how slow [23:57:55] log_date is indexed? [23:58:13] oh, we're already filtering on it [23:59:07] so the new case th handle is when log_date is null