[00:04:23] (03PS1) 10Cstone: Add missing types for the Payments Accounting Report [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/843587 (https://phabricator.wikimedia.org/T314753) [00:11:02] (03PS2) 10Cstone: Add missing types for the Payments Accounting Report [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/843587 (https://phabricator.wikimedia.org/T314753) [00:49:22] (03CR) 10Eileen: [C: 03+2] "I was able to see this successfully logging the contact ID." [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/843523 (https://phabricator.wikimedia.org/T320991) (owner: 10Wfan) [00:54:06] Thanks Eileen, and since we already have this locale negotiate check, should we add another function to map those 2 chars local to the right format? like sl to sl_SI? [00:54:17] locale * [00:54:55] I mean I kinda like the idea of just fixing the data in our db in the first instance - hopefully we aren't creating new ones & it is just a historical data cleanup [00:57:35] ok, if all historical data not been introduced by contact import, then it's fine, but I saw one failmail related with contact import: https://phabricator.wikimedia.org/T320836 isn't this means we are importing new contacts that is using two chars for locale? [00:58:40] 10Fundraising Tech - Chaos Crew: 'is' is not a valid option for field preferred_language - https://phabricator.wikimedia.org/T320836 (10AnnWF) [01:00:43] yeah - we need to fix that [01:01:04] do we have an id? [01:01:23] (03Merged) 10jenkins-bot: Also log the contact id for thank you email failmail [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/843523 (https://phabricator.wikimedia.org/T320991) (owner: 10Wfan) [01:01:26] MariaDB [civicrm]> select preferred_language, count(*) FROM civicrm_contact WHERE LENGTH(preferred_language) < 4 group by preferred_language; [01:01:26] +--------------------+----------+ [01:01:26] | preferred_language | count(*) | [01:01:26] +--------------------+----------+ [01:01:26] | en | 597032 | [01:01:26] | fr | 501 | [01:01:26] | nl | 273 | [01:01:27] +--------------------+----------+ [01:05:20] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Clean up our two-character locale data - https://phabricator.wikimedia.org/T320829 (10AnnWF) Fix language negotiation exception to be standard://github.com/civicrm/civicrm-core/pull/24764/commits/b3ba53889f1ae98aed397e23004648cdde801406 [01:06:06] 10Fundraising Tech - Chaos Crew: 'is' is not a valid option for field preferred_language - https://phabricator.wikimedia.org/T320836 (10Eileenmcnaughton) I just figured out the problem - for some reason the right option (is_IS) is disabled - see below ` MariaDB [civicrm]> select * from civicrm_option_value whe... [01:06:07] Ah, that’s many of them, do you feel we can log this update to this ticket? https://phabricator.wikimedia.org/T320829 [01:06:26] Oh ok, 320836 also make sense πŸ˜„ [01:10:44] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Clean up our two-character locale data - https://phabricator.wikimedia.org/T320829 (10Eileenmcnaughton) select preferred_language, count(*) FROM civicrm_contact WHERE LENGTH(preferred_language) < 4 group by preferred_language; 2:01 PM +------------------... [01:10:46] wfan: I'm gonna cleanup the icelandic records - I guess I can close then & yep that other ticket for the historical ones [01:11:27] Great 😊 thanks eileen:! [01:23:28] (03PS1) 10AndyRussG: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/843589 [01:24:40] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Make it easier for us to edit our languages - https://phabricator.wikimedia.org/T321025 (10Eileenmcnaughton) [01:28:00] (03CR) 10AndyRussG: [C: 03+2] Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/843589 (owner: 10AndyRussG) [01:28:29] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Make it easier for us to edit our languages - https://phabricator.wikimedia.org/T321025 (10Eileenmcnaughton) [01:30:01] (03CR) 10CI reject: [V: 04-1] Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/843589 (owner: 10AndyRussG) [01:32:12] 10Fundraising Tech - Chaos Crew: 'is' is not a valid option for field preferred_language - https://phabricator.wikimedia.org/T320836 (10Eileenmcnaughton) I enabled `is_IS` (Icelandic), updated the 9 contacts with `is_US` or `is_DK` to `is_IS` and deleted `is_US` and `is_DK` - I think we can close this now [01:35:06] (03CR) 10AndyRussG: [C: 03+2] "resubmit" [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/843589 (owner: 10AndyRussG) [01:37:41] (03Merged) 10jenkins-bot: Merge branch 'master' into wmf_deploy [extensions/CentralNotice] (wmf_deploy) - 10https://gerrit.wikimedia.org/r/843589 (owner: 10AndyRussG) [02:11:19] fr-tech CentralNotice ESI test has purchased its ticket for this week's deploy change, with related config change booked for a backport window tomorrow (UTC afternoon)! https://wikitech.wikimedia.org/wiki/Deployments#Tuesday,_October_18 [02:19:18] I'm watching the recorrded engagement meeting I missed while I was away - has anyone watched it - I feel like they maybe meant to stop recording when people broke into private conversations? [02:27:08] I emailed Jayde - she is checking it out [02:34:48] hmmm no didn't watch it eileen, rather I attended live [04:31:39] (03PS2) 10Abijeet Patro: BannerMessageGroup: Ensure proper columns are in the GROUP BY clause [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/841568 [04:31:41] (03PS2) 10Abijeet Patro: Remove usage of deprecated MessageIndexRebuildJob::insertIntoJobQueue [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/841569 [04:31:53] (03CR) 10Abijeet Patro: BannerMessageGroup: Ensure proper columns are in the GROUP BY clause (031 comment) [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/841568 (owner: 10Abijeet Patro) [10:08:39] (03CR) 10Nikerabbit: [C: 03+2] BannerMessageGroup: Ensure proper columns are in the GROUP BY clause [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/841568 (owner: 10Abijeet Patro) [10:20:15] (03Merged) 10jenkins-bot: BannerMessageGroup: Ensure proper columns are in the GROUP BY clause [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/841568 (owner: 10Abijeet Patro) [10:20:17] (03Merged) 10jenkins-bot: Remove usage of deprecated MessageIndexRebuildJob::insertIntoJobQueue [extensions/CentralNotice] - 10https://gerrit.wikimedia.org/r/841569 (owner: 10Abijeet Patro) [11:45:11] 10Fundraising Sprint Solid State Spaghetti, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-Smashpig: Move damaged message browser to SmashPig CiviCRM extension - https://phabricator.wikimedia.org/T295412 (10Damilare) a:03Damilare [13:45:38] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10MBeat33) [13:57:15] 10Fundraising-Backlog, 10FR-Email: Click-tracking in Acoustic - https://phabricator.wikimedia.org/T316573 (10Aklapper) [14:50:39] hey fr-tech. I've pushed up the finished version of the fr-tech stack diagram here https://phabricator.wikimedia.org/T321089 [14:51:40] If anyone has a few minutes to check over it and confirm it makes sense and looks accurate I'd appreciate it! thanks in advance [14:52:57] I've also pushed up a branch containing the same diagram to our fr-tech-diagrams repo https://gitlab.wikimedia.org/repos/fundraising-tech/fr-tech-diagrams/-/commit/f862400e54451f861b17206ac8178bddd42f4484 [14:54:28] hmm I just realised that there's no line from the email-pref-wiki to the queues. lemme add that [15:05:42] jgleeson: hiii! thxso much, yes for sure I'll comment a bit later :) [15:06:05] thanks! [15:06:31] also fr-tech how do folks feel about moving today's tech-talk so it doesn't overlap with Architecture event? [15:14:34] works for me AndyRussG. I won't be able to attend if it's afterwards though as it goes beyond 10pm my time. [15:15:34] actually if it's immediately after I might stay up to see eileen :) we don't get much overlap [15:23:50] hi ejegg [15:24:10] when you get some time, could you check if https://phabricator.wikimedia.org/F35596665 looks right to you? [15:25:54] I don't know much about how privatebin works or what it speaks to so it would be good to get your feedback on that bit too [15:34:16] hi jgleeson [15:34:18] will do! [15:35:48] so there are some acoustic import processes that run directly as drush jobs (not python) and talk to Acoustic [15:35:56] also the forgetme button makes Acoustic calls [15:36:17] so there should be some arrow from drupal / civi to Acoustic [15:36:24] or back and forth [15:40:49] thanks ejegg [15:45:36] ejegg: what drupal modules are those? I'm looking over the code and can't see anything [15:46:03] for the acoustic stuff [15:47:20] oh is this the omnimail-silverpop thing [15:47:32] the civi ext [15:47:34] 10Fundraising Sprint Discworld reformatted as ntfs, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Request to make $1k donation bot for END gifts only - https://phabricator.wikimedia.org/T301982 (10RLewis) 05Resolvedβ†’03Open [15:50:20] jgleeson: yep, exactly [15:50:27] the omnimail civi ext [15:51:23] now to look at privatebin - it doesn't do any queue stuff, and I'm trying to remember if it does any db connections. I think it might just store stuff on the filesystem [15:52:29] great [15:57:57] that's updated now ejegg https://phabricator.wikimedia.org/T321089#8325561 [15:58:11] cool cool [15:58:18] I've struggle to keep the 'data lines' from crossing [15:58:23] struggled* [15:59:40] 10Fundraising-Backlog, 10fr-donorservices: Fundraising access request for Donor Relations agent Salvatore Borriello - https://phabricator.wikimedia.org/T321100 (10krobinson) [16:00:41] 10Fundraising-Backlog, 10fr-donorservices: Fundraising access request for Donor Relations agent Salvatore Borriello - https://phabricator.wikimedia.org/T321100 (10krobinson) [16:01:14] 10Fundraising-Backlog, 10fr-donorservices: Fundraising access request for Donor Relations agent Salvatore Borriello - https://phabricator.wikimedia.org/T321100 (10krobinson) [16:01:22] 10Fundraising-Backlog, 10fr-donorservices: Fundraising access request for Donor Relations agent Salvatore Borriello - https://phabricator.wikimedia.org/T321100 (10krobinson) [16:02:47] hmm maybe we should make it clear that donation interface exists in email-prefs-wiki [16:02:59] that might be a candidate for a refactor in the future [16:09:34] fr-tech, does FundraisingEmailUnsubscribe still get used or did we move away from that after introducing the email-prefs-wiki? [16:09:39] (03PS1) 10Wfan: Check if $mail->from exist and if not set reply to to blank [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/843990 (https://phabricator.wikimedia.org/T318837) [16:09:48] jgleeson: if only... [16:09:53] (it still exists) [16:09:54] ah [16:12:35] 10Fundraising Tech - Chaos Crew: Also log contact_id for thank you email send failed failmail log - https://phabricator.wikimedia.org/T320991 (10AnnWF) https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/843523 [16:15:58] hey fr-tech can anyone review this one its quick https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/SmashPig/+/843587/ just 2 missing types for the audit to ignore (discovered them when testing yesterday) [16:16:03] (03Abandoned) 10Wfan: Check if $mail->from exist and if not set reply to to blank [wikimedia/fundraising/crm/civicrm] - 10https://gerrit.wikimedia.org/r/837728 (https://phabricator.wikimedia.org/T318837) (owner: 10Wfan) [16:16:24] looking cstone [16:16:51] (03CR) 10Ejegg: [C: 03+2] "Looks good" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/843587 (https://phabricator.wikimedia.org/T314753) (owner: 10Cstone) [16:17:06] oops jgleeson just saw you were looking too [16:17:12] np [16:17:28] thanks! [16:17:46] (03Merged) 10jenkins-bot: Add missing types for the Payments Accounting Report [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/843587 (https://phabricator.wikimedia.org/T314753) (owner: 10Cstone) [16:17:48] (03CR) 10Ejegg: [C: 03+2] Check if $mail->from exist and if not set reply to to blank [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/843990 (https://phabricator.wikimedia.org/T318837) (owner: 10Wfan) [16:22:07] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Clean up our two-character locale data - https://phabricator.wikimedia.org/T320829 (10AnnWF) today the data updated to | preferred_language | count(*) | | en | 597014 | | fr | 501 | | nl | 273 |... [16:33:24] (03Merged) 10jenkins-bot: Check if $mail->from exist and if not set reply to to blank [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/843990 (https://phabricator.wikimedia.org/T318837) (owner: 10Wfan) [16:53:20] (03PS1) 10Wfan: update smash-pig requirement to 0.8.3.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/844008 [16:54:13] (03CR) 10CI reject: [V: 04-1] update smash-pig requirement to 0.8.3.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/844008 (owner: 10Wfan) [16:55:16] fr-tech have we had any issues with packagist not updating even though the tag is there and the green update button is clicked? [17:09:22] cstone: yeah I think I've had that before [17:09:46] some magic cache in the ether maybe [17:09:47] seems like wikimedia/smash-pig did not picked the new tag [17:10:25] lemme pull smashpig [17:10:27] what's the tag? [17:10:33] 0.8.3.2? [17:10:35] =v0.8.3.2 [17:10:36] yep! [17:10:37] yes [17:11:49] yeah so I see it [17:11:52] locally [17:12:02] but ci isn't seeing it? [17:12:11] packagist isnt updating [17:12:16] ohh [17:12:24] when we click the green update it just does nothing [17:12:43] ah is it on the github mirror [17:12:58] if you check the log from packgist, it did try to read the cache of wikimedia/smash-pig, while no 8.3.2 there~ [17:12:59] is it using that? I forget [17:13:02] lemme login [17:13:03] i do not know [17:13:38] https://github.com/wikimedia/wikimedia-fundraising-SmashPig [17:13:49] looks like that's the repo it's using [17:14:01] and I don't see your tag there [17:14:13] or my code update that i want to deploy [17:15:03] maybe ask releng if they can poke the thing that pushes updates our to the github mirroe [17:15:06] mirror* [17:16:02] gonna grab food [17:17:04] hmm that repo doesnt even have any commits from me [17:23:10] 10Fundraising-Backlog, 10fundraising-tech-ops, 10fr-donorservices: Fundraising access request for Donor Relations agent Salvatore Borriello - https://phabricator.wikimedia.org/T321100 (10Dwisehaupt) [17:25:58] 10Fundraising-Backlog, 10fundraising-tech-ops, 10fr-donorservices: Fundraising access request for Donor Relations agent Salvatore Borriello - https://phabricator.wikimedia.org/T321100 (10Dwisehaupt) [17:36:12] (03CR) 10Jforrester: "check experimental" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/843587 (https://phabricator.wikimedia.org/T314753) (owner: 10Cstone) [17:49:03] do we store trxn with payments_final_status failed to civi? [17:51:08] hmm, based on what we have at pendingQueueConsume, should skip, [17:51:27] cstone: sorry, just seeing this now [17:51:35] I was able to push a tag directly to github once [17:51:41] oh ejegg theres a replag issue [17:51:49] i talked to releng about it [17:52:05] 6-12 hours! [17:52:09] ah yeah, I see in the other channel [17:52:25] if it's for somethign urgent I think we can push the tag ourselves [17:52:44] and it hopefully won't cause any problems if it's the same tag on the same revision [17:53:02] sounds sensible to me ejegg [17:53:07] same hashes right [17:53:07] (03PS13) 10Ejegg: Add support for SMTP to SmashPig's MailHandler [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/837136 (https://phabricator.wikimedia.org/T318831) (owner: 10Jgleeson) [17:53:23] (03CR) 10Ejegg: [C: 03+2] "Looks good, works in smoke tests!" [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/837136 (https://phabricator.wikimedia.org/T318831) (owner: 10Jgleeson) [17:53:33] (03CR) 10Ejegg: [V: 03+2 C: 03+2] Update global config 'email' to the new 'mail-backend' key [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/843566 (owner: 10Jgleeson) [17:53:42] ty ejegg [17:53:46] :) [17:53:57] (03Merged) 10jenkins-bot: Add support for SMTP to SmashPig's MailHandler [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/837136 (https://phabricator.wikimedia.org/T318831) (owner: 10Jgleeson) [17:54:29] i guess for github the change isnt there yet either [17:54:41] not just the tag is missing [17:59:33] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10AnnWF) payments_final_status is failed and the payment_status is Denied, while we still have 2022-10-10 01:32:00,781 I... [17:59:35] 10Fundraising-Backlog, 10FR-Email: Click-tracking in Acoustic - https://phabricator.wikimedia.org/T316573 (10DBu-WMF) Does this mean that this particular task is now moving forward? Is there an ETA? [17:59:43] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10AnnWF) a:03AnnWF [18:03:21] 10Fundraising Sprint Solid State Spaghetti, 10Fundraising-Backlog: Turn on and test Adyen pending transaction resolver - https://phabricator.wikimedia.org/T320571 (10Cstone) a:03Cstone [18:26:43] fr-tech, AndyRussG asked about moving tech-talk earlier but not many folks responded. Any thoughts? [18:28:35] (03PS1) 10Wfan: Clean up historical two-character locale data to default one [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844047 (https://phabricator.wikimedia.org/T320829) [18:28:35] im fine with whenever [18:32:12] earlier as in over midsprint or another time? [18:38:58] my guess is later due to it already being early for eileen [18:55:22] 10Fundraising Sprint Solid State Spaghetti, 10Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig: Implement PayPal EC DoExpressCheckoutPayment in SmashPig - https://phabricator.wikimedia.org/T318756 (10jgleeson) Gonna move this out of sprint for now and pick up {T318759} which I think is the o... [18:55:30] 10Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig: Implement PayPal EC DoExpressCheckoutPayment in SmashPig - https://phabricator.wikimedia.org/T318756 (10jgleeson) [18:55:50] 10Fundraising Sprint Solid State Spaghetti, 10Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig: Implement PayPal EC SetExpressCheckout in SmashPig - https://phabricator.wikimedia.org/T318759 (10jgleeson) a:03jgleeson [18:56:19] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Ingenico FIELD_LENGTH_INVALID error when firstName is too long - https://phabricator.wikimedia.org/T317355 (10AnnWF) Firstname for this donation is only 7 characters, and this donor tried 18 times (accumulated to riskscore 1000) for the same donation, I b... [18:57:22] (03PS7) 10Ejegg: Add Adyen data deletion requests to forgetme [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/832580 (https://phabricator.wikimedia.org/T311741) [18:58:29] (03CR) 10Ejegg: "Thanks for the CR AndyRussG! I addressed the comments in PS7 and also made some fixes so it doesn't break when you run it on a donor whose" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/832580 (https://phabricator.wikimedia.org/T311741) (owner: 10Ejegg) [19:34:25] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Patch-For-Review: Clean up our two-character locale data - https://phabricator.wikimedia.org/T320829 (10AnnWF) a:03AnnWF [19:35:04] 10Fundraising Tech - Chaos Crew: 'is' is not a valid option for field preferred_language - https://phabricator.wikimedia.org/T320836 (10AnnWF) a:03Eileenmcnaughton [19:44:52] 10Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig: Implement PayPal EC SetExpressCheckout in SmashPig - https://phabricator.wikimedia.org/T318759 (10jgleeson) a:05jgleesonβ†’03None [19:45:11] 10Fundraising Sprint Solid State Spaghetti, 10Fundraising-Backlog, 10FR-PayPal-ExpressCheckout, 10FR-Smashpig: Implement PayPal EC DoExpressCheckoutPayment in SmashPig - https://phabricator.wikimedia.org/T318756 (10jgleeson) [19:49:54] 10Fundraising Sprint Quarkery, 10Fundraising Sprint Recursion Excursion, 10Fundraising Sprint Solid State Spaghetti, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Way to send Civi TY email to a group? - https://phabricator.wikimedia.org/T314525 (10Eileenmcnaughton) @MDemosWMF @RLewis I think th... [19:50:56] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10Ejegg) Looks like these were recorded in response to an IPN message, not a donation direct from the front-end. Let's se... [19:55:27] 10Fundraising Sprint NaN is a Number, 10Fundraising Sprint Overused petting Zoo Memetics, 10Fundraising Sprint Potato, 10Fundraising Sprint Quarkery, and 4 others: Get imports running in the background - https://phabricator.wikimedia.org/T312166 (10Eileenmcnaughton) OK so this is needed https://github.com/... [19:56:47] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Adyen getAccountConfig() call failures - https://phabricator.wikimedia.org/T313441 (10AnnWF) a:03AnnWF [20:00:59] 10Fundraising Sprint Quarkery, 10Fundraising Sprint Recursion Excursion, 10Fundraising Sprint Solid State Spaghetti, 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Way to send Civi TY email to a group? - https://phabricator.wikimedia.org/T314525 (10MDemosWMF) Yep, looks like I'm able to select e... [20:01:10] dami want to do that screenshare in the original tech talk meeting [20:01:13] ? [20:05:12] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10AnnWF) Ha, find out the reason, that we have the payments_init table later than the time we were checking for that dona... [20:12:54] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10Ejegg) I think it's a bigger issue that we're not actually doing anything with the valid_statuses key in the paypal con... [20:29:53] (03PS1) 10Ejegg: Drop PayPal IPNs outside of valid_statuses list [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/844059 (https://phabricator.wikimedia.org/T320989) [20:30:28] (03CR) 10CI reject: [V: 04-1] Drop PayPal IPNs outside of valid_statuses list [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/844059 (https://phabricator.wikimedia.org/T320989) (owner: 10Ejegg) [20:34:50] (03PS2) 10Ejegg: Drop PayPal IPNs outside of valid_statuses list [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/844059 (https://phabricator.wikimedia.org/T320989) [20:39:44] wfan I think ^^^ will fix the 'Denied' ones being recorded [20:41:08] and we can find how many there have been with [20:41:21] zgrep -F '[payment_status] => Denied' payments-listener-smashpig- [20:49:33] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices, 10Patch-For-Review: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10Ejegg) @EMartin We're making a pretty small code change to make this drop 'Denied' donations in t... [21:11:44] (03CR) 10Wfan: [C: 03+2] "We are going to use other tickets to cover the session cleared and new orderid created bug, and this way works for now :) Thanks elliott!" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/839766 (https://phabricator.wikimedia.org/T318769) (owner: 10Ejegg) [21:13:16] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices, 10Patch-For-Review: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10AnnWF) a:05AnnWFβ†’03Ejegg [21:13:50] (03Merged) 10jenkins-bot: Stop redirecting on Adyen failure [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/839766 (https://phabricator.wikimedia.org/T318769) (owner: 10Ejegg) [21:41:48] 10Fundraising-Backlog: Gpay transactions stopped erroneously for AVS/CVV scores in non-US markets - https://phabricator.wikimedia.org/T321136 (10EMartin) [21:47:57] 10Fundraising-Backlog: Gpay transactions stopped erroneously for AVS/CVV scores in non-US markets - https://phabricator.wikimedia.org/T321136 (10EMartin) Count of scores by country on stopped Gpay transactions. {F35597417} [21:52:20] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices, 10Patch-For-Review: Civi showing completed donations when they were rejected at Paypal - https://phabricator.wikimedia.org/T320989 (10EMartin) @Ejegg. Is it a great deal of work? If so, I say don't bother if it's fixed going forwa... [22:05:54] ok - the thing I didn't check on during the call has 4 legs & too many teeth - off to check if it was causing chaos [22:06:36] chaos bunny is cuter than the chaos monkey. [22:19:04] 10Fundraising Sprint Solid State Spaghetti, 10Fundraising-Backlog, 10MediaWiki-extensions-CentralNotice, 10Readers-Web-Backlog, 10MW-1.40-notes (1.40.0-wmf.6; 2022-10-17): CentralNotice: Add ESI test string to base HTML - https://phabricator.wikimedia.org/T320734 (10AndyRussG) These changes are live on p... [22:29:48] lol [22:36:22] (03PS1) 10Damilare Adedoyin: WIP: Move Damage table to CiviCRM extension [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844072 (https://phabricator.wikimedia.org/T295412) [22:37:11] thanks all, the damaged entity is now returning data. I'd do some tidy up and set up the pages tomorrow [22:37:19] bye for now [23:03:20] nice [23:42:50] smashpig got the tag 0.8.3.2 [23:43:07] (03PS2) 10Wfan: update smash-pig requirement to 0.8.3.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/844008 [23:43:58] (03CR) 10CI reject: [V: 04-1] update smash-pig requirement to 0.8.3.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/844008 (owner: 10Wfan) [23:44:52] (03PS3) 10Wfan: Update smashPig to 0.8.3.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/844008 [23:47:59] (03CR) 10Wfan: [C: 03+2] Update smashPig to 0.8.3.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/844008 (owner: 10Wfan) [23:50:18] (03Merged) 10jenkins-bot: Update smashPig to 0.8.3.2 [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/844008 (owner: 10Wfan) [23:59:36] (03PS1) 10Wfan: Update SmashPig to v0.8.3.2 and DonationInterface to v2.5.8.2 [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/844077