[03:07:15] (PS1) Ori.livneh: Remove dead code for targetting users based on UserDailyContribs data [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/225669 [03:08:00] (PS2) Ori.livneh: Remove dead code for targetting users based on UserDailyContribs data [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/225669 [12:59:40] atgo: Stop that immediately [13:00:53] awight: ...? [13:01:05] Aha! You're actually working, I saw it [13:01:18] last night! [13:01:32] i'm on vacation startingt omorrow :) [13:07:46] ©_© [13:08:06] Just trying to take it out on someone else that I'm working too [13:11:11] hehe i saw your email [13:11:29] Yeah that thing Megan uncovered is interesting. [13:11:47] I've been imagining how to collect all our gotchas... it's a long list. [13:11:58] i think she's only concerned about the new subscriptions [13:12:16] perfect [13:13:53] I'm very curious to hear what she's currently using to look at this data, though. Like I wrote, I don't remember having this discussion yet. [15:44:08] (CR) Awight: [C: 2] Fix undefined var warning [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/225560 (owner: Ejegg) [15:44:10] (Merged) jenkins-bot: Fix undefined var warning [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/225560 (owner: Ejegg) [15:44:59] ejegg|away: Holding off on the crm->DI rollback, cos the tests fail... That's still the plan though, right? [15:45:17] (PS3) Awight: amend update to fix Direct Mail Appeal custom field [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/225492 (owner: Cdentinger) [15:50:54] (CR) Awight: [C: 2] "Thanks for writing this, it makes the code way more readable! There are some small things to fix, see the inline comments, but totally me" (3 comments) [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/225492 (owner: Cdentinger) [15:51:05] (PS3) Awight: Rearrange order of CC logos on GC form for Japan [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/225359 (https://phabricator.wikimedia.org/T102496) (owner: XenoRyet) [15:51:46] (Merged) jenkins-bot: amend update to fix Direct Mail Appeal custom field [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/225492 (owner: Cdentinger) [15:51:50] (CR) Awight: [C: 2] Rearrange order of CC logos on GC form for Japan (1 comment) [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/225359 (https://phabricator.wikimedia.org/T102496) (owner: XenoRyet) [15:52:15] (Merged) jenkins-bot: Rearrange order of CC logos on GC form for Japan [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/225359 (https://phabricator.wikimedia.org/T102496) (owner: XenoRyet) [15:52:24] (CR) Awight: [C: -2] "I'm temporarily blocking this, cos it looks like it depends on the newer DI?" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/225261 (https://phabricator.wikimedia.org/T105848) (owner: Ejegg) [15:54:53] (CR) Awight: [C: 2] "Sorry, I think I reviewed this from an airport. I'm merging cos it's great, but there are things to fix inline, and the cross-repo CG iss" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/223971 (owner: Ejegg) [15:55:03] (PS2) Awight: Generate CurrencyRates.php instead of old tables [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/223971 (owner: Ejegg) [15:55:22] (PS4) Awight: Update currency rates [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/223827 (owner: Ejegg) [15:56:50] (CR) Awight: [C: 2] "I'm not sure what's going on here--we shouldn't depend on actual reference data for testing, there should be separate fixtures which set a" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/223827 (owner: Ejegg) [15:57:12] (PS2) Awight: Dynamically generate JS currency minimum amounts [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/223839 (owner: Ejegg) [15:57:14] (Merged) jenkins-bot: Update currency rates [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/223827 (owner: Ejegg) [15:58:45] (CR) Awight: [C: 2] "Clever! It would be much less complicated to just generate the .js file, though. What am I missing?" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/223839 (owner: Ejegg) [15:59:13] (Merged) jenkins-bot: Dynamically generate JS currency minimum amounts [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/223839 (owner: Ejegg) [16:00:51] (CR) Awight: [C: -1] "Can you fix it in your local setup instead? We shouldn't have to have an origin repo, and many of us are cloning from https, then using g" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/224872 (owner: Ejegg) [16:08:06] (PS4) Awight: enable passthru of utm_* fields during check import [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/217018 (owner: Cdentinger) [16:09:19] (CR) Awight: [C: 2] "Nvm, my comments weren't valid. We won't be importing recurring contrib in the near future, and having something in utm_medium is just fi" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/217018 (owner: Cdentinger) [16:10:13] (Merged) jenkins-bot: enable passthru of utm_* fields during check import [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/217018 (owner: Cdentinger) [16:16:54] Fundraising-Backlog: Put real-time email ecom data in Google Spreadsheet - https://phabricator.wikimedia.org/T105900#1463674 (awight) Source code for my prototype is here, http://git.wikimedia.org/tree/wikimedia%2Ffundraising%2Ftools/HEAD/live_analysis The roadmap, if we decide to pursue this, would be some... [16:18:21] (PS3) Awight: Generate CurrencyRates.php instead of old tables [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/223971 (owner: Ejegg) [16:19:08] (CR) Awight: [C: -1] Use static data as ultimate forex fallback [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/223973 (owner: Ejegg) [17:59:32] (CR) AndyRussG: "Thanks!!" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/225669 (owner: Ori.livneh) [18:01:10] (CR) Legoktm: "Since this is being exported as a JS variable, do we need to check if any banners are using it...?" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/225669 (owner: Ori.livneh) [18:01:32] (CR) Legoktm: "...I'm slow." [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/225669 (owner: Ori.livneh) [20:19:18] Fundraising-Backlog: Set up import for Major Gifts events payment/invitation tool - https://phabricator.wikimedia.org/T101191#1464047 (awight) Looks like this is on track. Here are our canonical templates, or the link @ejegg sent works fine, too. http://git.wikimedia.org/tree/wikimedia%2Ffundraising%2Fcrm/... [20:22:40] Fundraising Sprint Lou Reed, Fundraising Sprint Miles Davis, Fundraising Sprint N*E*R*D, Fundraising Sprint ODB, and 4 others: Create an Civi import function for hand keyed (including DAF) donations - https://phabricator.wikimedia.org/T88836#1464054 (awight) @LeanneS: Those are up to date, but the... [20:24:18] Fundraising Sprint Lou Reed, Fundraising Sprint Miles Davis, Fundraising Sprint N*E*R*D, Fundraising Sprint ODB, and 4 others: Create an Civi import function for hand keyed (including DAF) donations - https://phabricator.wikimedia.org/T88836#1464056 (awight) omg. One moment please, we're having t...