[00:09:20] (03PS1) 10Wfan: Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) [00:09:32] (03CR) 10CI reject: [V: 04-1] Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) (owner: 10Wfan) [00:13:24] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10Patch-For-Review: mc-debit not found for ingenico - https://phabricator.wikimedia.org/T334769 (10AnnWF) [00:15:26] (03CR) 10Ejegg: "Looks pretty good! Let's also fix the wmf_contribution_extra.gateway_txn_id where gateway=dlocal" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910099 (https://phabricator.wikimedia.org/T335057) (owner: 10Wfan) [00:15:59] (03PS7) 10Wfan: Add missing submethods for ingenico [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/908868 (https://phabricator.wikimedia.org/T334769) [00:16:04] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-eqiad: Q1:rack/setup/install frbast1002, frmon1002, frpig1002 - https://phabricator.wikimedia.org/T319460 (10Dwisehaupt) @Cmjohnson I did some debugging with @papaul online and we think there may be a crossed cable issue between frbast1002 and frpig1002. Coul... [00:16:53] (03CR) 10Ejegg: [C: 03+2] Add missing submethods for ingenico [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/908868 (https://phabricator.wikimedia.org/T334769) (owner: 10Wfan) [00:17:03] (03PS2) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [00:17:11] Thx ejegg: will update wmf_contribution_extra [00:18:35] (03Merged) 10jenkins-bot: Add missing submethods for ingenico [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/908868 (https://phabricator.wikimedia.org/T334769) (owner: 10Wfan) [00:19:58] (03CR) 10CI reject: [V: 04-1] Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) (owner: 10Eileen) [00:20:00] (03PS3) 10Ejegg: Split out card-specific property mapping [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/909430 (https://phabricator.wikimedia.org/T260725) [00:20:02] (03PS3) 10Ejegg: Add amount and currency to Paypal detail response [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/909431 [00:20:04] (03PS3) 10Ejegg: Catch exceptions in createPaymentSession [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/909799 (https://phabricator.wikimedia.org/T260725) [00:22:26] (03PS4) 10Ejegg: Catch exceptions in createPaymentSession [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/909799 (https://phabricator.wikimedia.org/T260725) [00:25:55] (03PS3) 10Wfan: Revert trxn_id to default case for dlocal [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910099 (https://phabricator.wikimedia.org/T335057) [00:28:53] (03PS1) 10Ejegg: Handle ApiException in getLatestPaymentStatus [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910106 (https://phabricator.wikimedia.org/T260725) [00:29:20] (03CR) 10CI reject: [V: 04-1] Handle ApiException in getLatestPaymentStatus [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910106 (https://phabricator.wikimedia.org/T260725) (owner: 10Ejegg) [00:38:40] (03PS3) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [00:44:04] (03PS5) 10Ejegg: Catch exceptions in createPaymentSession [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/909799 (https://phabricator.wikimedia.org/T260725) [00:44:08] (03PS2) 10Ejegg: Handle ApiException in getLatestPaymentStatus [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910106 (https://phabricator.wikimedia.org/T260725) [00:45:21] (03PS2) 10Ejegg: Use NationalCurrencies to replace dlocal default country array [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/908645 (https://phabricator.wikimedia.org/T334416) (owner: 10Wfan) [00:46:07] (03CR) 10Ejegg: [C: 03+2] "Looks good for the current dlocal countries, at least! Things will get tricky if we start adding EUR or USD countries." [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/908645 (https://phabricator.wikimedia.org/T334416) (owner: 10Wfan) [00:46:17] (03PS2) 10Ejegg: add getSimpleSubmethods for DI pair if valid submethod [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910093 (https://phabricator.wikimedia.org/T334937) (owner: 10Wfan) [00:46:23] (03CR) 10Ejegg: [C: 03+2] add getSimpleSubmethods for DI pair if valid submethod [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910093 (https://phabricator.wikimedia.org/T334937) (owner: 10Wfan) [00:46:40] (03Merged) 10jenkins-bot: Use NationalCurrencies to replace dlocal default country array [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/908645 (https://phabricator.wikimedia.org/T334416) (owner: 10Wfan) [00:46:55] (03Merged) 10jenkins-bot: add getSimpleSubmethods for DI pair if valid submethod [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910093 (https://phabricator.wikimedia.org/T334937) (owner: 10Wfan) [00:50:25] (03PS1) 10Ejegg: Adyen audit: handle secondchargeback [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910108 [00:50:48] wfan or cstone I think we need ^^^ to keep adyen audit from failing again tonight [00:51:16] I'll tag a smashpig version once we get that merged [00:54:35] looking ejegg [00:56:50] (03CR) 10Cstone: [C: 03+2] Adyen audit: handle secondchargeback [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910108 (owner: 10Ejegg) [00:56:58] thanks! [00:57:13] (03Merged) 10jenkins-bot: Adyen audit: handle secondchargeback [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910108 (owner: 10Ejegg) [00:57:40] ejegg: so I've put up the fields - https://gerrit.wikimedia.org/r/c/wikimedia/fundraising/crm/+/910103 - now testing upgrading the population script [00:57:51] looking eileen [00:57:59] (03PS4) 10Wfan: Revert trxn_id to default case for dlocal [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910099 (https://phabricator.wikimedia.org/T335057) [00:58:48] oh hang on - some stuff missing [00:59:40] no - I was confused [01:00:15] ah - I didn't regenerate - doing that now [01:00:40] ok, i'm going to get this audit fix out in any case [01:01:25] latest SmashPig tag is a .13 version, hope it's lucky! [01:01:28] (03PS4) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [01:02:45] (03PS1) 10Ejegg: Update SmashPig [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910111 [01:02:47] (03CR) 10Ejegg: [C: 03+2] Update SmashPig [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910111 (owner: 10Ejegg) [01:03:15] that version brings in all the money! [01:03:33] (03PS5) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [01:06:18] (03CR) 10CI reject: [V: 04-1] Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) (owner: 10Eileen) [01:06:34] this is a pretty cool walk through of the art centre I was involved in getting built - https://www.youtube.com/watch?v=l_QkNrdZITE [01:06:34] (it's only mildly annoying that the 2 words they keep repeating they pronounce badly...) [01:06:57] In ContributionTracking.php line 820: [01:06:57] 21:06:13 [01:06:57] 21:06:13 Undefined class constant 'T_INTEGER [01:10:37] (03PS6) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [01:10:56] ejegg: yeah I just spotted that [01:10:59] & pushed it [01:11:25] am doing a rebuild now to check - I didn't run the last regen before & hence it 'worked' without working last time [01:15:40] (03Merged) 10jenkins-bot: Update SmashPig [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910111 (owner: 10Ejegg) [01:22:29] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: dLocal one-hour test failmail: "Merchant has no authorization to use this API" / "Token not found or inactive" - https://phabricator.wikimedia.org/T334945 (10AnnWF) [01:22:35] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal, 10Patch-For-Review: Set error for Dlocal unsupported card types in India - https://phabricator.wikimedia.org/T334937 (10AnnWF) [01:25:51] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: dLocal one-hour test failmail: "Merchant has no authorization to use this API" / "Token not found or inactive" - https://phabricator.wikimedia.org/T334945 (10AnnWF) [01:25:53] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal, 10Patch-For-Review: Set error for Dlocal unsupported card types - https://phabricator.wikimedia.org/T334937 (10AnnWF) 05duplicate→03Open a:05jgleeson→03AnnWF [01:27:27] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal: dLocal one-hour test failmail: "Merchant has no authorization to use this API" / "Token not found or inactive" - https://phabricator.wikimedia.org/T334945 (10AnnWF) [01:27:30] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal, 10Patch-For-Review: Set error for Dlocal unsupported card types - https://phabricator.wikimedia.org/T334937 (10AnnWF) [01:28:17] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal, 10Patch-For-Review: Set error for Dlocal unsupported card types - https://phabricator.wikimedia.org/T334937 (10AnnWF) [01:42:48] (03CR) 10Ejegg: "We could add a boolean 'recurring' flag parsed out of the utm_source as well. When there is an 'r' at the start of the payment_method that" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) (owner: 10Eileen) [01:43:14] hmm, donations queue consumer locked up? [01:43:56] no big queries in the processlist [01:44:15] it did it earlier and also no big queries when i checked [01:44:31] ejegg: ok re adding the extra field. Re payment_method using an option group - what are the possible options? [01:44:33] there was one instance before where the big query didnt show up all the time [01:46:26] yeah - I checked the read only db & didn't see something there either [01:48:09] eileen: possible options are cc, dd, bt, cash, paypal, obt, rtbt, apple, google, amazon, ew [01:48:33] I only see ew, obt, and dd for old-old Ingenico though [01:49:02] so what are bt obt rtbt and ew? [01:49:14] bt=Bank Transfer [01:49:34] obt=Online Bank Transfer (no idea why it's different - I think we only ever got them from Australia) [01:49:57] rtbt=Real-time bank transfer (again, no idea why it's different - we only code iDEAL this way as far as I know) [01:50:06] ew = E-wallet [01:50:28] why is paypal a top-level method and not a submethod of e-wallet? I dunno [01:51:17] lol paypal via globalcollect is supposedly ew_paypal [01:51:23] yeah we never got any of those [01:53:10] ok, so there are a ton of submethods in that FinanceInstrument list [01:54:40] I'm going to prune a couple that I know we're not getting any more [01:54:57] ewww [01:56:07] ok - so what I'll do is update up the patch now with the OptionGroups defined & the values for payment_method but just a couple for payment_submethod otherwise I'll be on that for a while & I want to get the new fields into the script [02:01:03] eileen: ejegg for naming things like upsell, do we want to leave it that or give it some other name? as of right now we have two different things that have both been called upsell in the past [02:01:26] cstone: ah.. is there a clearly name you can think of? [02:01:35] no thats the problem I guess hah [02:01:36] hmm right we call it monthly convert in DonationInterface [02:01:48] just the first upsell = monthly convert and now this new upsell is still upsell [02:01:50] ejegg: arg so we do have these in the db already - https://civicrm.frdev.wikimedia.org/civicrm/admin/options/payment_instrument?reset=1 [02:02:07] are they payment method or submethod or .... neither [02:02:44] those are mostly submethods [02:03:24] ejegg: could we use that as the option group? [02:04:04] for submethod [02:04:28] eileen: so we would just need to add a string value to each that corresponds to the value as configured in DonaionInterface and SmashPig [02:04:37] I've actually wanted to do that for a while [02:04:40] i guess if that upsell is going to refere to monthly convert maybe we name it that in the db? [02:04:44] we can use the name field [02:05:15] cstone: the more you talk about upsell the less I know what it means [02:05:16] to get rid of this huge mapping mess: https://phabricator.wikimedia.org/diffusion/WFCG/browse/master/drupal/sites/default/civicrm/extensions/wmf-civicrm/Civi/WMFHelpers/FinanceInstrument.php [02:05:22] ah [02:05:40] well it was called upsell, the renamed updonate, and then renamed again monthly convert [02:06:01] ok & monthly donate is def the type of upsell here? [02:06:10] sorry monthly_convert? [02:06:44] I think that upsell is monthly convert it just never got renamed [02:06:53] ok cool [02:07:26] So I'm gonna link payment_submethod to payment_instrument option group on the assumption we wil work through that & just add a new group for method [02:08:19] ok... that'd be a heck of a sql query, but I guess we can get the greatest hits in for starters [02:09:47] oh i see, name and label are idential there [02:10:26] ah wait, for contribution_tracking rows I bet it only has rtbt_ideal in that last field [02:10:41] because they don't send any other specific types in the banner clicks [02:10:56] so sure, that won't really slow down our data import [02:11:40] so does rbt_ideal map to ideal here https://civicrm.frdev.wikimedia.org/civicrm/admin/options/payment_instrument?reset=1 [02:12:13] I wonder where it would show up if we changed the 'name' in those instruments to the code from the messages [02:12:22] eileen: yep, maps to 'iDeal' [02:12:31] hah, still inaccurately described as Ideal from Global Collect [02:13:03] sorrt, that's rtbt_ideal [02:13:48] yeah changing 'name' should be OK - but just need to be careful if we are using MGD system (which we should) cos name + option_group_id are the identifier [02:15:31] ah yeah, we'd have to change this file and also have a one-time update script I think: https://phabricator.wikimedia.org/diffusion/WFCG/browse/master/drupal/sites/default/civicrm/extensions/wmf-civicrm/Managed/PaymentInstruments.mgd.php [02:16:04] ah phooey [02:16:12] are those just front end ones ejegg our our old friend sepa isnt there [02:16:39] i guess if they are just the direct debit maybe [02:16:49] "Apple Pay: Visa" and "Credit Card: Visa" both have payment_submethod=visa [02:17:05] we need both payment_method and payment_submethod to distinguish some of those eileen [02:17:09] hmm [02:17:22] & we can't consider the combo I guess [02:17:32] or pre-pend something [02:17:57] re the difference between Bank Transfer & Online Bank Transfer - yeah that makes sense to me [02:18:24] again, for contribution_tracking, payment_submethod only has a value when a donor selects a specific submethod in a banner [02:18:36] and afaik that's only rtbt_ideal and maybe now upi [02:18:59] it would be great to clean up the mess in general [02:19:10] but we might not need to do much of it for this project [02:19:16] If I want to pay an invoice I can either do a bank transfer - ie I go into my bank account & transfer the money to the given account OR I select bank transfer on their site instead of credit card & then I get the bank login popup etc - is that the difference? [02:19:38] eileen: your guess is as good as mine! [02:19:38] ejegg: yeah we mostly want the fields & existing data in place huh [02:19:56] I think we got sloppy about categorizing bt/obt [02:20:29] or rather whoever came up with the distinction was gone by the time we added all the south american bank transfer methods and they all just got called 'bt' [02:20:50] even though I think they're all pretty online [02:21:37] how old are these maybe online bank transfer was a cool new way? [02:22:22] obt was 'old' when I got here in 2014 [02:22:26] lol [02:22:51] ejegg: are you on-board with renaming upsell_type-id to monthly_convert_type_id as cstone suggested [02:22:51] cool new way in like 2007 [02:23:00] and we added the astropay bt ones in 2015 [02:23:21] eileen: oh ok [02:23:37] were just going to be stuck with these column names right? [02:23:45] eileen: ugh, maybe upsell is more generic? [02:24:04] like, I don't know if monthly convert is just one type of upsell [02:24:04] upsell is just confusing ejegg especially when we are going to use it otherwise for that recurring thing [02:24:11] is pay-the-fee also an upsell? [02:24:16] hmmmm [02:24:37] pay-the-fee does get it's own field [02:24:51] ok, right [02:25:17] so if they invent new ways of asking people for more money after they click, should we add more columns, or more upsell types? [02:25:30] joseph was talking about making things less confusing earlier and i just think the upsell term is confusing [02:25:39] for how we use it/dont use it [02:26:45] i guess if we separate this upsell from MC/others and its just adding something to a donation? [02:28:32] ah well, I guess they could both ask for pay the fee and also monthly convert [02:29:02] so that's a reason to have a column for each [02:30:51] ohh hah its monthlyUpsell in the utm_key even worse name [02:31:03] yay [02:32:14] i dont know if we are having it mean monthly convert just seems like a future point of confusion when its still called upsell here [02:33:31] but then is that more or less confusing when its still called monthlyUpsell in the params being sent over [02:33:52] oh wait, looking at the two options it looks like that should be recurring_type [02:34:07] if they're 'upsell' vs 'organic' [02:34:24] (03PS7) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [02:34:26] 'organic' means there was no upsell at all, the donor just chose monthly on the first step [02:34:35] ah is that what it means [02:34:38] oooh [02:35:04] so it's almost like upsell_method rather than type [02:35:16] or even recurring_upsell_method_id [02:35:54] eileen: but there's no upsell at all for 'organic' [02:36:08] hmm - can we go back to a boolean? [02:36:10] I think it's just recurring_type_id [02:36:27] recurring_reason_id? [02:36:50] I guess type is fine [02:36:57] well, we could distinguish between no-upsell, upsell-on-banner and monthly-convert-on-payments-form, i guess [02:37:30] but for the backfill we only have those first two [02:37:45] ok - so recurring_type_id - does that work for you cstone [02:37:47] i dont think we have anything else called recurring_type [02:38:02] yeah thanks for listening to my name requests hah just knowing we have something coming thats also called upsell... [02:38:37] yeah sometimes when I don't really understand something I just use what I've been told but it's usually better to understand [02:39:07] and in the rest of our code this upsell got renamed monthly convert [02:39:08] so for those option values I guess we can fix name + label later if we need to [02:39:20] yep [02:39:22] ah nice yeah [02:39:25] as long as we get the column names right [02:39:54] recurring_type gives us room for more types too [02:40:36] I still slightly wonder if it is a reason or motive rather than type [02:40:45] (cos it doesn't 'stay with the recurring') [02:41:25] (03PS8) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [02:42:00] recurring_source_id? (hah to bring in that word) recurring_style? [02:42:12] oh I know... source_code [02:42:37] I kinda like recurring_source_id tho [02:46:03] oh you named it choice recurring_choice_id kind of makes sense [02:46:57] oh right - yeah that was the label - so we are recurring_choice_id now? I'm good with that - ejegg ? [02:47:55] https://www.helen.co.nz/digital-prints/kiwi-slang-choice/#:~:text=Choice%20is%20a%20kiwi%20slang,%2C%20fine%2C%20awesome%20or%20cool. [02:48:55] its cool to start a recurring payment [02:49:37] :-) [02:52:12] im gona head out im ok with choice or type or even source hah [02:54:14] (03PS9) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [02:54:23] just_another_source [02:54:46] source_2_final_use_this_one [02:56:21] oh sure, recurring_choice_id is fine [02:57:10] ejegg: not source_2_final_really_final_version_3 [02:57:12] (03CR) 10CI reject: [V: 04-1] Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) (owner: 10Eileen) [02:59:03] (03CR) 10Ejegg: "ok, now that we have banner_variant I want to change the old 'variant' field to payments_form_variant" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) (owner: 10Eileen) [02:59:24] also a comment on mailing_identifier length ^^ [03:00:07] ejegg: so minor point payments_form or payment_form? [03:00:30] hah, payments_form as in payments-wiki form [03:00:58] let's leave that one as-is [03:01:36] ok [03:02:48] (03Abandoned) 10Ejegg: Use ReferenceData to decode Ingenico submethods [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/790788 (https://phabricator.wikimedia.org/T308088) (owner: 10Ejegg) [03:07:40] woohoo, googlepay working on new Adyen JS after their account fix! [03:10:16] so our new contribution tracking queue consumer will do a lot more db option lookups, but I guess they'll all be cached [03:10:25] it runs for like 10 minutes at a time [03:11:06] ejegg: yeah - they will all be cached [03:15:54] (03PS10) 10Eileen: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) [03:19:28] ok eileen that is looking pretty good [03:19:57] i guess maybe banner_variant could be 32 max as well [03:21:49] but I'm ok +2in just to get it moving [03:22:17] ejegg: how sure are you on 32 for banner_variant - feels like the sort of thing that might mess with us [03:23:42] ehh, not entirely [03:23:48] so let's go with what we have [03:23:55] (03PS11) 10Ejegg: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) (owner: 10Eileen) [03:23:59] (03CR) 10Ejegg: [C: 03+2] Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) (owner: 10Eileen) [03:27:28] ok - it's merged - now I need to try another populate-script pass! [03:29:56] nice! Shall we deploy? [03:30:07] I want to get that smashpig version bump out in any case [03:30:29] I can update the vendor submodule [03:33:42] (03PS1) 10Ejegg: Delete a couple of unused payment methods from mapper [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910114 [03:34:43] (03CR) 10CI reject: [V: 04-1] Delete a couple of unused payment methods from mapper [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910114 (owner: 10Ejegg) [03:36:08] (03Merged) 10jenkins-bot: Additional contribution_tracking fields proposed by analytics [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910100 (https://phabricator.wikimedia.org/T334532) (owner: 10Eileen) [03:36:54] (03PS1) 10Ejegg: Update smashpig [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/910115 [03:36:59] (03CR) 10Ejegg: [C: 03+2] Update smashpig [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/910115 (owner: 10Ejegg) [03:38:31] (03PS1) 10Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/910116 [03:38:55] there's the merge + submodule bump eileen ^^ [03:39:00] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - 10https://gerrit.wikimedia.org/r/910116 (owner: 10Ejegg) [03:39:45] ejegg: cool - you don't need me to +2 anything? [03:40:53] I had a funny moment with the extension upgrade on staging via the CLI - no problem via UI so I ignored but it might happen on prod - no idea [03:47:43] yeah it happened again - it said UserJob api doesn't exist [03:49:10] (03Merged) 10jenkins-bot: Update smashpig [wikimedia/fundraising/crm/vendor] - 10https://gerrit.wikimedia.org/r/910115 (owner: 10Ejegg) [03:49:23] !;og civicrm upgraded from efdf9434 to a2e7c079 [03:49:26] !log civicrm upgraded from efdf9434 to a2e7c079 [03:49:30] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [04:01:16] k, i have to hang it up for the night [04:55:11] recur [05:34:02] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-Phase 1 Create new CiviCRM contribution tracking entity - https://phabricator.wikimedia.org/T334532 (10Eileenmcnaughton) OK - we added fields more-or-less like above with some changes to what fr-tech felt worked. Note that we went w... [06:07:03] (03CR) 10CI reject: [V: 04-1] Localisation updates from https://translatewiki.net. [extensions/DonationInterface] (REL1_40) - 10https://gerrit.wikimedia.org/r/910204 (owner: 10L10n-bot) [07:12:21] (03PS3) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910103 (https://phabricator.wikimedia.org/T334534) [07:17:49] (03PS4) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910103 (https://phabricator.wikimedia.org/T334534) [07:49:28] (03PS5) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910103 (https://phabricator.wikimedia.org/T334534) [07:55:59] (03PS1) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910416 (https://phabricator.wikimedia.org/T334534) [07:58:38] (03PS2) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910416 (https://phabricator.wikimedia.org/T334534) [08:00:24] (03PS3) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910416 (https://phabricator.wikimedia.org/T334534) [08:17:36] (03PS4) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910416 (https://phabricator.wikimedia.org/T334534) [08:21:24] (03PS5) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910416 (https://phabricator.wikimedia.org/T334534) [08:23:57] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-Phase 1 Create new CiviCRM contribution tracking entity - https://phabricator.wikimedia.org/T334532 (10Eileenmcnaughton) @JMando the fieilds are somewhat populated on staging. Can you take look. The sql it is using is below - you c... [08:29:17] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-Phase 1 - expose in the UI - https://phabricator.wikimedia.org/T334540 (10Eileenmcnaughton) @Damilare is your hook firing? The import hook is injecting the search into a form using civiimport_civicrm_buildForm (even though it looks... [09:54:48] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-Phase 1 - expose in the UI - https://phabricator.wikimedia.org/T334540 (10Eileenmcnaughton) @Damilare the other thing you can do is jump on chat.civicrm.org & ping colemanw if your hook is definitely firing - he should be around for... [11:31:01] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-Phase 1 - expose in the UI - https://phabricator.wikimedia.org/T334540 (10Damilare) Sounds good, thanks @Eileenmcnaughton. [11:33:57] (03CR) 10Damilare Adedoyin: [C: 03+2] Make Adyen setup function async [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/908318 (https://phabricator.wikimedia.org/T334999) (owner: 10Ejegg) [11:35:49] (03Merged) 10jenkins-bot: Make Adyen setup function async [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/908318 (https://phabricator.wikimedia.org/T334999) (owner: 10Ejegg) [14:55:12] 10Fundraising-Backlog, 10fr-donorservices: iDeal recurring donation continued after cancellation - https://phabricator.wikimedia.org/T335140 (10SHust) [14:56:19] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-eqiad: Q1:rack/setup/install frbast1002, frmon1002, frpig1002 - https://phabricator.wikimedia.org/T319460 (10Jclark-ctr) Verified cables and corrected them @Dwisehaupt [14:59:49] thanks damilare [15:00:03] np [15:01:20] (03PS1) 10Ejegg: Update SmashPig [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910510 [15:02:30] (03CR) 10Ejegg: [C: 03+2] Update SmashPig [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910510 (owner: 10Ejegg) [15:06:12] ok, I noticed one more change in the changelog that I think we need to update our code for to use the new Adyen SDK - the validation errors supposedly have different properties for the different components [15:08:09] (03PS1) 10Ejegg: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/910514 [15:08:11] (03CR) 10Ejegg: [C: 03+2] Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/910514 (owner: 10Ejegg) [15:16:05] (03Merged) 10jenkins-bot: Update SmashPig [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910510 (owner: 10Ejegg) [15:16:07] (03Merged) 10jenkins-bot: Merge branch 'master' into deployment [extensions/DonationInterface] (deployment) - 10https://gerrit.wikimedia.org/r/910514 (owner: 10Ejegg) [15:17:10] (03PS1) 10Ejegg: Update composer.lock for new SmashPig [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/910516 [15:17:16] (03CR) 10Ejegg: [C: 03+2] Update composer.lock for new SmashPig [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/910516 (owner: 10Ejegg) [15:25:05] (03PS1) 10Ejegg: Ignore another type of Adyen IPN [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910518 [15:25:16] (03Merged) 10jenkins-bot: Update composer.lock for new SmashPig [core] (fundraising/REL1_39) - 10https://gerrit.wikimedia.org/r/910516 (owner: 10Ejegg) [15:31:14] oops, forgot to !log [15:31:32] !log payments-wiki upgraded from 66be66e0 to 744d82c6 [15:31:35] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log [15:32:06] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-Phase 1 Create new CiviCRM contribution tracking entity - https://phabricator.wikimedia.org/T334532 (10JMando) That code looks correct to me. I do not see any data in the table, but I do see the new columns in there. Once there is d... [15:54:04] (03CR) 10Cstone: [C: 03+2] Ignore another type of Adyen IPN [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910518 (owner: 10Ejegg) [15:54:36] (03Merged) 10jenkins-bot: Ignore another type of Adyen IPN [wikimedia/fundraising/SmashPig] - 10https://gerrit.wikimedia.org/r/910518 (owner: 10Ejegg) [15:58:47] 10Fundraising Sprint Highway to Hello World, 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal, 10MediaWiki-extensions-DonationInterface: "Order id is duplicated" error from dlocal - https://phabricator.wikimedia.org/T334905 (10jgleeson) a:03jgleeson [15:59:16] i wonder why that chargeback continued we refunded them like 2 hours after the notification of fraud [16:06:13] (03PS1) 10Wfan: Update link for all form chooser supported dlocal countries [wikimedia/fundraising/dev] - 10https://gerrit.wikimedia.org/r/910525 (https://phabricator.wikimedia.org/T334937) [16:29:52] https://phabricator.wikimedia.org/T335140 [16:32:34] 175438503.2 [16:46:30] ok, that's the debug logging reverted [16:47:56] (03PS5) 10Wfan: Add css for other amount and the larger amount class updatet [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/908662 (https://phabricator.wikimedia.org/T334308) [16:47:58] (03PS2) 10Wfan: Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) [16:49:09] 10Fundraising-Backlog: Check recurring status later in recurring charge job - https://phabricator.wikimedia.org/T335152 (10Cstone) [16:49:45] (03CR) 10CI reject: [V: 04-1] Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) (owner: 10Wfan) [16:50:07] 10Fundraising-Backlog, 10fr-donorservices: iDeal recurring donation continued after cancellation - https://phabricator.wikimedia.org/T335140 (10Cstone) Ahh this was an edge case bug. I made a ticket to fix the issue here: {T335152} What happened is it was mid process of charging them when Seilo cancelled the... [16:53:33] Thanks cstone for working on this ideal recurring cancellation issue [16:54:49] np i put the query on this one https://phabricator.wikimedia.org/T335152 [17:02:09] (03PS4) 10Ejegg: Send initial scheme txn ID to Civi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T331754) [17:03:03] ok, that has a test now ^^^ [17:03:22] goin to lunch [17:03:57] (03CR) 10CI reject: [V: 04-1] Send initial scheme txn ID to Civi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [17:12:22] PROBLEM - check_log_messages on frav1002 is CRITICAL: CRITICAL: check_endpoints_critical (Amazon:1) 1 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1002&service=check_log_messages [17:17:23] PROBLEM - check_log_messages on frav1002 is CRITICAL: CRITICAL: check_endpoints_critical (Amazon:1) 1 [=1] https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1002&service=check_log_messages [17:19:12] ACKNOWLEDGEMENT - check_log_messages on frav1002 is CRITICAL: CRITICAL: check_endpoints_critical (Amazon:1) 1 [=1] Jeff_Green known https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1002&service=check_log_messages [17:38:30] I can fix that pedantic CI failure ejegg|food [17:41:12] oh i see, I'll assertSame [17:41:42] (03PS5) 10Ejegg: Send initial scheme txn ID to Civi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T331754) [17:41:45] thanks! [17:45:45] huh, adyen's docs site has been down all day [17:45:57] ah yeah i noticed that when trying to figure out that chargeback [17:55:46] ejegg: happy to +2 that patch. could we make the fixme a phab task? feels worthy of doing sooner rather than later that one [17:57:12] ah right, let me remind myself what that was [17:58:16] the ingenico-specific stuff in the gateway adapter [17:58:54] oh right, actually, with the refactor of Ingenico it's not using curl_transaction at all [17:59:14] so that's a good reminder to do that sanitizing before logging the raw response [17:59:47] I guess I'll delete that fixme (and that ingenico-specific code) once I do that [18:00:12] are you working on that now? [18:00:19] yep! [18:00:22] ah cool [18:00:26] so it's gotten to be a really big patch [18:00:34] but i just now realized the way to split it out [18:00:51] is probably one patch to refactor the initial hosted checkout setup [18:01:03] and then another to refactor the processDonorReturn [18:02:13] (03CR) 10Jgleeson: [C: 03+2] "Code looks good. I guess we can't test it fully but the test coverage gives me confidence that if Ingenico do send it, we will capture the" [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [18:02:23] thanks for the CR! [18:02:30] np [18:02:44] ooh, looking back at that task I realize there was one more step [18:02:53] I need to backfill the data for existing recurrings! [18:03:19] looking at that task I just merged? [18:03:28] https://phabricator.wikimedia.org/T331754 [18:03:43] yep [18:03:48] so now the code is all done [18:04:06] we pick up the IDs for new txns and store them in a new custom field [18:04:14] and then use them when we make recurring charges [18:04:37] I just need to populate that custom field for all the existing recurring charges where I can find one of those IDs in the logs [18:05:35] time for some grepping [18:08:19] gotcha [18:17:13] ok, so we're not going to be able to get the very first of these for some recurrings [18:17:34] but I think they say we can use a more recent one if needed [18:17:47] so I'll just snag a recent one [18:22:51] ah damn, in the nightly charge we've only been logging the full response when there's an error [18:22:54] boo [18:23:10] ok, I'll try just populating the ones I can find on the initial txn [18:23:53] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-eqiad: Q1:rack/setup/install frbast1002, frmon1002, frpig1002 - https://phabricator.wikimedia.org/T319460 (10Dwisehaupt) @Jclark-ctr Thanks! I can verify that worked and I have been able to start building out the hosts. [18:25:02] 10fundraising-tech-ops, 10DC-Ops, 10SRE, 10ops-eqiad: Q1:rack/setup/install frbast1002, frmon1002, frpig1002 - https://phabricator.wikimedia.org/T319460 (10Dwisehaupt) a:05Cmjohnson→03Dwisehaupt [18:25:39] well, that'll leave 14k non-USD recurrings without it [18:26:14] (03Merged) 10jenkins-bot: Send initial scheme txn ID to Civi [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/906673 (https://phabricator.wikimedia.org/T331754) (owner: 10Ejegg) [18:39:25] :O [18:39:36] does that mean they will fail ejegg ? [18:41:38] hopefully not [18:42:01] according to Adyen's docs we shouldn't even be subject to this rule [18:42:16] but ingencio says there's an increase in rejections without this param [18:42:42] anyway, i'm having a hard time getting these IDs from the logs [18:42:57] I wonder if I can look any up via API [18:43:20] https://phabricator.wikimedia.org/T239770 for fixing that recurring charge job log, it really isnt helpful to print out the previous payment [18:45:46] hah we had more recurring adyens yesterday than paypals [18:47:31] wow [18:52:22] hmm, ok, I can get some info via API [18:52:41] I think I'll have to write a lil action to do this in code then [18:52:52] gonna relocate and then get going on it [19:22:19] hmm so it looks like this duplicate order ID bug might be happening when a donor is redirected to paytm and doesn't come back [19:23:07] in one instance the donor was redirect but the payment sat at pending and didn't move. the same donor then comes back an hour later and tries to donate again but is assigned the order ID of the earlier PENDING attempt [19:23:20] redirected* [19:23:34] damilare: ^^ it happened again :) [19:24:27] so I wonder if we're resetting things correctly [19:26:19] haha you're definitely not getting the "no text errors" award this year jgleeson [19:26:29] :) [19:29:24] ah well, the ingenico API call I need is still returning just the raw array [19:29:28] * TODO: make this return a normalized PaymentStatusResponse or the like [19:29:36] s'ok, I can work with that [19:29:40] it has the scheme ID in it [19:34:34] (03PS1) 10Ejegg: WIP fill scheme ID for ingenico recurrings [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910577 [19:47:21] RECOVERY - check_log_messages on frav1002 is OK: OK https://icinga.wikimedia.org/cgi-bin/icinga/extinfo.cgi?type=2&host=frav1002&service=check_log_messages [20:01:22] 10Fundraising Sprint Highway to Hello World, 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10FR-dlocal, 10MediaWiki-extensions-DonationInterface: "Order id is duplicated" error from dlocal - https://phabricator.wikimedia.org/T334905 (10jgleeson) I looked into `175199518:175199518.1` / `17520629... [20:06:38] I wonder if the order id coming back on the return URL has anything to do with it [20:06:57] although based on the logs, the donor never came back from the external paytm flow [20:10:49] have a good long weekend fr-tech [20:10:52] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog, 10fr-donorservices: adjust audit to catch Adyen script refunds? - https://phabricator.wikimedia.org/T334503 (10MBeat33) I shared these transactions with Jake & he's looking into the gap in timing for refunds... [20:10:54] bye for now [20:35:53] (03PS1) 10Damilare Adedoyin: CT-to-Civi-Phase 1 - expose in the UI [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910589 (https://phabricator.wikimedia.org/T334540) [20:40:19] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog, 10Patch-For-Review: CT-to-Civi-Phase 1 - expose in the UI - https://phabricator.wikimedia.org/T334540 (10Damilare) I've been able to render the ContributionTracking search in the Contribute Dashboard with some help from Coleman. Here's what... [21:10:05] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-Phase 1 Create new CiviCRM contribution tracking entity - https://phabricator.wikimedia.org/T334532 (10Eileenmcnaughton) @jmando there is data imported with it on staging - although I'm trying to get code review so I can push the pa... [21:10:39] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-Phase 1 Create new CiviCRM contribution tracking entity - https://phabricator.wikimedia.org/T334532 (10Eileenmcnaughton) @jmando there is data imported with it on staging - although I'm trying to get code review so I can push the pa... [21:13:06] so joseph looked at the populate sql & thought it looked ok but didn't see the data cos he looked on live not staging [21:14:11] given we can do what I've been doing on staging (truncate & start again) maybe if we get the script out without too much review then I can run a few batches & joseph can look at the data [21:18:36] (03PS3) 10Wfan: Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) [21:20:16] (03CR) 10CI reject: [V: 04-1] Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) (owner: 10Wfan) [21:24:01] (03PS4) 10Wfan: Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) [21:24:47] (03PS5) 10Wfan: Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) [21:26:51] (03CR) 10CI reject: [V: 04-1] Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) (owner: 10Wfan) [21:27:59] (03PS6) 10Wfan: Add DI error when dlocal smart field has unsupported card type [extensions/DonationInterface] - 10https://gerrit.wikimedia.org/r/910079 (https://phabricator.wikimedia.org/T334937) [21:31:02] (03CR) 10Eileen: [C: 03+2] "This looks good. Unfortunately there is a gotcha - that main contribution dashboard has a nasty summary query in it that means the page is" [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910589 (https://phabricator.wikimedia.org/T334540) (owner: 10Damilare Adedoyin) [21:35:17] eileen: will the columns be in that order on live as the are on staging right now? for the new contribution_tracking [21:35:52] cstone: within the table? Yeah - they are there in that order - but we could re-order [21:36:20] if i could request utm_source going by the other unbroken up ones [21:37:06] oh ok - the main reason I shoved it off to the end was that in theory it is obsolete - ejegg was really keen to drop it - but I can move if you want [21:37:13] ahh i see [21:45:11] (03Merged) 10jenkins-bot: CT-to-Civi-Phase 1 - expose in the UI [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910589 (https://phabricator.wikimedia.org/T334540) (owner: 10Damilare Adedoyin) [21:46:08] (03PS6) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910103 (https://phabricator.wikimedia.org/T334534) [21:46:35] (03Abandoned) 10Eileen: Populate contribution tracking fields [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910416 (https://phabricator.wikimedia.org/T334534) (owner: 10Eileen) [21:54:33] I just tinkered with the civi fields display on prod - specifically setting the custom fields for contribution extra to not expand on initial display - it makes the page more readable but I'm wondering who other than us would care what is in those fields & if I need to communicate they need to do an extra click (or if I should avoid moving anyone's cheese) [21:54:57] donor relations maybe? [21:56:09] yeah [22:16:52] have people read this - https://www.washingtonpost.com/investigations/2023/04/20/mike-lindell-prove-wrong-contest/ - it's kinda fascinating & mind boggling & I realise my non-tech friends won't be able to understand it [22:20:27] ok, I want to try to look up a bunch of those scheme ids on staging [22:20:35] he's gonna have to see a lot of pillows to cover that and the coming dominion suit [22:22:11] "At the symposium, participants interested in the contest were given a badge with a hot-pink dot to indicate they were cyber experts and could enter a room where Lindell’s data was shared." i dont know why this is the most ridiculous sentence in this article so far [22:22:32] im just imagining king of the mountain jersey hah [22:22:50] https://en.wikipedia.org/wiki/King_of_the_Mountains [22:23:02] yeah - the fact it was hot-pink was a nice detail [22:23:49] I love the fact he only had to disprove the files were nothing to do with 2020 election - they could literally have stuck in a photo of a bumper sticker & we would have been stymied [22:24:16] i like that some of the data was in pdfs [22:25:25] lol [22:29:17] glad my other laptop cant find my wifi but it can find fbivan [22:29:28] oh and fbivan-5G [22:29:35] and someones oven [23:06:26] 10Fundraising Tech - Chaos Crew, 10Fundraising-Backlog: Check recurring status later in recurring charge job - https://phabricator.wikimedia.org/T335152 (10AnnWF) [23:09:51] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-tangent remove queue2civicrm_log tables & writing to them 2 - https://phabricator.wikimedia.org/T334544 (10Eileenmcnaughton) 05Open→03Resolved a:03Eileenmcnaughton [23:09:53] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: Scope contribution tracking - https://phabricator.wikimedia.org/T182030 (10Eileenmcnaughton) [23:10:00] 10Fundraising Sprint Highway to Hello World, 10Fundraising-Backlog: CT-to-Civi-tangent remove queue2civicrm_log tables & writing to them 2 - https://phabricator.wikimedia.org/T334544 (10Eileenmcnaughton) [23:21:07] 10Fundraising-Backlog, 10MediaWiki-extensions-DonationInterface: Reintegrate Ingenico rtbt_enets payment method - https://phabricator.wikimedia.org/T141989 (10Eileenmcnaughton) @mbeat33 @Pcoombe - just searching for something in phab & hit this - I assume we can close? 2016 [23:29:50] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM: Move transaction ID field down for check donations - https://phabricator.wikimedia.org/T191851 (10Eileenmcnaughton) 05Open→03Declined [23:30:31] 10Fundraising-Backlog, 10Wikimedia-Fundraising-CiviCRM, 10FR-EN6C, 10FR-UK: UKFC as Payment Instrument option for foreign check import - https://phabricator.wikimedia.org/T167355 (10Eileenmcnaughton) 05Open→03Declined This seems no longer relevant [23:31:48] ejegg|food: I thought you opened a task to cleanup payment methods but I can't find it? [23:31:48] I did find some weird & wonderful (& likely closeable) things [23:31:48] https://phabricator.wikimedia.org/T131194 [23:31:48] https://phabricator.wikimedia.org/T131994 [23:31:48] https://phabricator.wikimedia.org/T167378 [23:43:42] (03PS1) 10Eileen: Remove some payment instruments that are not used [wikimedia/fundraising/crm] - 10https://gerrit.wikimedia.org/r/910608