[08:40:43] (CR) Hashar: "recheck" [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/46700 (owner: Hashar) [12:06:06] Fundraising-Backlog, MediaWiki-extensions-CentralNotice, Easy: Clone button for CN campaigns - https://phabricator.wikimedia.org/T91078#1721142 (Base) [13:26:57] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Easy: Change title of search box on Find Contributions page to only read "Contributor Name" - https://phabricator.wikimedia.org/T104885#1721379 (Devirk) Is this using any extension? [13:55:03] Fundraising-Backlog: [BUG] Donation form stuck with 0 as donation amount; donors can't modify it - https://phabricator.wikimedia.org/T113530#1721436 (MBeat33) This last donor sent additional feedback: (Using Chrome) I clicked on https://donate.wikimedia.org/ as requested. I was able to select my amount unde... [15:45:53] Fundraising-Backlog: Test Amazon (USD) as a test in non-US country - https://phabricator.wikimedia.org/T115342#1721910 (atgo) NEW [15:46:39] Fundraising Sprint UB40, Fundraising Tech Backlog, Fundraising-Backlog, Unplanned-Sprint-Work: EXTERNAL: Worldpay HCG search by email address - https://phabricator.wikimedia.org/T114183#1721923 (atgo) [15:48:46] Fundraising-Backlog: Compare amounts and currencies between payments logs and Civi, fix discrepancies - https://phabricator.wikimedia.org/T115173#1721944 (atgo) Is this for a 1-off audit/fix or something that will run constantly in the background? [17:01:12] dstrine: I'll be a couple minutes late to the meet, other meeting just ended 8p [17:04:27] AndyRussG: ok thanks [17:07:53] Fundraising Sprint UB40, Fundraising-Backlog, MediaWiki-extensions-DonationInterface, Unplanned-Sprint-Work, and 2 others: Record actual amount charged regardless of what's in session - https://phabricator.wikimedia.org/T115171#1722273 (Ejegg) [17:07:59] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, WorkType-NewFunctionality: Create an import method for matching gifts and payroll deductions - https://phabricator.wikimedia.org/T115044#1722275 (DStrine) [17:08:35] Fundraising Sprint UB40, Fundraising-Backlog, MediaWiki-extensions-DonationInterface, Unplanned-Sprint-Work, and 2 others: Record actual amount charged regardless of what's in session - https://phabricator.wikimedia.org/T115171#1716920 (Ejegg) Fixes deployed for AstroPay and GlobalCollect, work st... [17:14:31] Fundraising-Backlog: [BUG] Donation form stuck with 0 as donation amount; donors can't modify it - https://phabricator.wikimedia.org/T113530#1722325 (DStrine) [17:15:54] Fundraising Sprint UB40, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: CI tests are failing for Civi 4.6 branch - https://phabricator.wikimedia.org/T113422#1722331 (DStrine) [17:16:11] Fundraising Sprint UB40, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: CI tests are failing for Civi 4.6 branch - https://phabricator.wikimedia.org/T113422#1664584 (awight) [17:22:55] Fundraising Sprint Snoop (Dogg|Lion), Fundraising-Backlog, MediaWiki-extensions-CentralNotice, MediaWiki-extensions-Translate: Central Notice message groups are slow to index - https://phabricator.wikimedia.org/T111189#1722373 (DStrine) p:Triage>Normal a:AndyRussG>Nikerabbit [17:24:10] Fundraising Sprint Snoop (Dogg|Lion), Fundraising-Backlog, MediaWiki-extensions-CentralNotice, MediaWiki-extensions-Translate: Central Notice message groups are slow to index - https://phabricator.wikimedia.org/T111189#1722385 (awight) Hi @Nikerabbit, thanks for noticing this! We're happy to sup... [17:26:47] Fundraising-Backlog: Compare amounts and currencies between payments logs and Civi, fix discrepancies - https://phabricator.wikimedia.org/T115173#1722402 (DStrine) [17:28:45] Fundraising Tech Backlog, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, fundraising-tech-ops: Create new civicrm repo - https://phabricator.wikimedia.org/T114831#1722409 (DStrine) [17:29:32] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Easy: Deprecate wmf_unsubscribe module - https://phabricator.wikimedia.org/T115066#1722415 (DStrine) [17:31:27] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: CiviCRM: Change default Activity status from "Scheduled" to "Complete" - https://phabricator.wikimedia.org/T115036#1722421 (atgo) p:Triage>Low [17:32:16] Fundraising Tech Backlog, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, MediaWiki-extensions-DonationInterface: Formalize message type for all donation queue wire formats - https://phabricator.wikimedia.org/T114955#1722429 (atgo) [17:36:26] Fundraising-Backlog, fundraising-tech-ops, Performance: Serve cache control headers for static resources on paymentswiki - https://phabricator.wikimedia.org/T114404#1722449 (DStrine) [17:38:05] Fundraising-Backlog: EXTERNAL: CVV error code on WorldPay form - https://phabricator.wikimedia.org/T114677#1722463 (atgo) [17:38:09] Fundraising-Backlog, MediaWiki-extensions-CentralNotice: Banner history: Record count of pageviews when the user didn't get a campaign - https://phabricator.wikimedia.org/T113815#1722466 (AndyRussG) [17:39:26] Fundraising-Backlog: Compare amounts and currencies between payments logs and Civi, fix discrepancies - https://phabricator.wikimedia.org/T115173#1722469 (awight) Random code I started and abandoned: https://gerrit.wikimedia.org/r/#/c/234417/ Definitely a big win to have a reusable log parsing component tha... [17:50:23] Fundraising-Backlog, fundraising-tech-ops: Use Redis 3 when it's available - https://phabricator.wikimedia.org/T113399#1722531 (awight) Ubuntu "Wily" will include Redis 3, release date is Oct 2015. So we won't get Redis 3 until some time in 2016. Keep eyes peeled for a backport, or switch to Debian? [17:58:23] Fundraising-Backlog: Spike: Track email clickthroughs on donate wiki - https://phabricator.wikimedia.org/T114010#1722557 (CCogdill_WMF) @awight sounds good to me! A test email will be really easy, so just let me know when you're ready and I'll confirm link URLs with you at that point. [18:06:39] I'd like to deploy SmashPig and CRM to get the Amazon audit and refund stuff up. Any objection to doing that now K4-713 cwdent AndyRussG awight XenoRyet? [18:07:09] I've got nothing, but I'm probably not paying attention to relevant environmental factors at the moment. [18:07:16] Trying to figure out how to pay maxmind, actually. [18:07:23] ejegg: np! I was hoping to get the civi BH thing done but I've had issues w/ my vagant civi setup, so it's not done yet... So yeah np, thanks 4 asking! [18:07:26] YAY MANAGER BS. [18:07:44] oh man, gonna have to just pay yourself and get reimbursed? [18:08:46] (CR) Awight: [C: 2] "cool!" [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244593 (owner: Eileen) [18:09:00] ejegg: Nah. There's no way I'm floating that number. [18:09:04] Too many zeroes. [18:09:09] ahh, bignums... [18:09:58] (CR) Awight: [C: 2] Revert "wmf-sunset-patch add refund form" [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244603 (owner: Eileen) [18:10:57] (CR) Awight: "How did the code change without a release?" [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244631 (https://phabricator.wikimedia.org/T99838) (owner: Eileen) [18:11:22] (CR) Awight: [C: 2] "Great, with the above caveats :)" [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244631 (https://phabricator.wikimedia.org/T99838) (owner: Eileen) [18:11:28] ejegg: no problem here [18:11:36] w/ sp deploy [18:11:38] (CR) Awight: [C: 2] CRM-17000 bypass separatevalues for getfields for performance [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244854 (owner: Eileen) [18:12:37] cool, going ahead with it [18:13:49] (Merged) jenkins-bot: CRM-17144 add developer tab to lybunt [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244593 (owner: Eileen) [18:13:51] (Merged) jenkins-bot: Revert "wmf-sunset-patch add refund form" [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244603 (owner: Eileen) [18:13:53] (Merged) jenkins-bot: CIvi46 Update to latest 4.6.9 version [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244631 (https://phabricator.wikimedia.org/T99838) (owner: Eileen) [18:13:55] (Merged) jenkins-bot: CRM-17000 bypass separatevalues for getfields for performance [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244854 (owner: Eileen) [18:15:22] Fundraising-Backlog: Compare amounts and currencies between payments logs and Civi, fix discrepancies - https://phabricator.wikimedia.org/T115173#1722627 (awight) Small correction, although I do like @ejegg's callback idea, I was suggesting something slightly different. We need the ability to use regex sear... [18:28:29] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, WorkType-NewFunctionality: Create an import method for matching gifts and payroll deductions - https://phabricator.wikimedia.org/T115044#1722684 (atgo) @leannes we're getting into crunch time for December and I want to check in on priority for this o... [18:28:57] atgo: fyi, the motivator here ^^ is to get the 3rd party out of Civi [18:29:25] yeah that's what i was thinking, awight... but i'm just looking at some of the other things that we've got going on and trying to make some hard decisions [18:29:27] :/ [18:30:15] We just started the discussion with them, to get them onto spreadsheets. Let's talk about what exactly we're postponing here. [18:31:11] We can keep them out of Civi and improve data quality with spreadsheets the way they are now. But it will create a lot of extra work for LSchreibstein, not really acceptable. [18:31:51] The other alternative is to leave them in Civi, then potentially retraining them once we've upgraded. [18:32:23] Now that I've spoken with them and gotten a sense of what's going on there, I'm pretty anxious to get them out of Civi... [18:32:29] !updated SmashPig from 513ec01123e6dbb97b00888a3610a7c5ec24a63b to b5ff2a7d5f17aaaa33a169ca101cbea639769c90 [18:35:54] Fundraising-Backlog, Epic: [epic] PayPal upgrade - https://phabricator.wikimedia.org/T87621#1722710 (awight) I think T107384 is unrelated, not a blocker [18:43:40] (CR) Awight: [C: 2] "Don't be ashamed :p" [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/244703 (owner: Ejegg) [18:46:13] atgo: fwiw, some more rambling about the Engage thing above. [18:49:04] Fundraising-Backlog: Spike: Track email clickthroughs on donate wiki - https://phabricator.wikimedia.org/T114010#1722719 (awight) Any time! [18:49:27] ccogdill: So you aren't participating in the rolling blackouts? [18:49:40] hm? [18:49:48] which blackouts are these awight? [18:50:17] I read that SoCal or at least L.A. was totally hammered by the A/C usage [18:50:30] oh, nope, I don’t think that’s been a problem in SD [18:50:49] awight: thanks for the thoughts. i totally get what you're saying, i'm just looking at what we've got and weight the different pieces [18:50:51] got the nuke plants nearby, I guess :p [18:51:24] haha I think the big plants were shut down a year or two ago... [18:51:30] ccogdill: any time you want to talk about the URLs, just holler! [18:51:35] I’m not up to date on my SD environmental politics though [18:51:38] hehe [18:51:43] now is good! [18:51:52] so are we ready for a test? [18:51:59] sure--just to us, right? [18:52:24] yeah I can do it to just you and me - maybe the_wub too ? [18:52:28] We haven't done any backend work yet, I just want to see what the URLs will look like without the Silverpop redirect [18:52:31] sounds perfect. [18:53:31] okay so I’m not actually modifying the URL at all to work for donatewiki, just turning off click tracking [18:53:33] is that right? [18:53:48] I think so [18:54:01] k give me 10 minutes, I should have something out to yo [18:54:03] *you [18:54:33] maybe put me on the mailing list twice, actually--my second address should be "awight+2" or something [18:54:58] that way I can tell whether we'll get unique mailing destination IDs w/o clicktracking enabled. [18:55:21] cool [18:56:16] (Merged) jenkins-bot: Shameless self-promotion [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/244703 (owner: Ejegg) [18:59:27] (CR) Awight: [C: 2] Set subscr_id on Amazon recurring if missing [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/244708 (https://phabricator.wikimedia.org/T111431) (owner: Ejegg) [18:59:45] (PS2) Awight: Minor cleanup [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/244705 (owner: Ejegg) [19:00:27] (CR) Awight: [C: 2] Minor cleanup (1 comment) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/244705 (owner: Ejegg) [19:01:37] okay awight I just sent you a test mailing - this is what Silverpop calls a test which means it’s not technically a live email to a real contact list [19:01:57] but it’s still assigned a mailing ID. let me know if you need an email outside of this staging environment [19:02:26] will just be a few more minutes of work to make a new contact list for you and me [19:03:12] (Merged) jenkins-bot: Set subscr_id on Amazon recurring if missing [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/244708 (https://phabricator.wikimedia.org/T111431) (owner: Ejegg) [19:04:51] ccogdill: cool, received it [19:05:55] cool [19:06:45] so I’m we would add the contact ID or respondent ID to the URL if tracking through donatewiki - we can customize and add whatever fields you need [19:07:17] That's a great idea, to simplify how we link back to our contact list [19:08:24] ccogdill: Well, the bad news is that turning off clicktracking... turned it off. [19:08:31] There are no unique IDs in there. [19:08:43] right [19:08:46] But your proposal to add the contact ID will fill that hole [19:08:52] so that’s why we have to add the id param in the url [19:08:53] yep [19:09:00] contact ID & utm_source will make a unique pair [19:10:03] okay awight - you’re in our silverpop DB (as we all are - the fundraising team is an excellent group of donors :p) [19:10:19] want me to send you a live email using your civi contact record? and add the contact_id [19:10:24] Totally [19:10:27] at your leisure [19:10:32] on it [19:10:39] (Merged) jenkins-bot: Minor cleanup [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/244705 (owner: Ejegg) [19:10:44] If that works, I'd say you're pretty much done on the SP side of things! [19:10:49] wee! [19:10:50] sweeeet [19:12:07] looks like country + language both get correctly populated using those links. [19:12:23] the donatewiki link is the same as what SP redirected to, right? [19:13:22] yep, all I did was turn off silverpop’s click tracking portion [19:13:32] Great. tried 'n' true [19:13:40] we don’t pass the country code, fwiw. we let it geolocate [19:14:43] In the future, it might be nice to include a hint like altcountry=MX or something, so that people can quickly switch to the correct payment methods and currency if they're traveling, etc. [19:14:52] I'll TODO that if you agree [19:16:35] hmm, sure, I guess I don’t see why not [19:16:50] (PS4) Awight: Back-end logic for Amazon recurring donations [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/243360 (https://phabricator.wikimedia.org/T111430) (owner: Ejegg) [19:18:38] awight email is out! [19:20:13] (CR) Awight: [C: 2] Back-end logic for Amazon recurring donations (2 comments) [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/243360 (https://phabricator.wikimedia.org/T111430) (owner: Ejegg) [19:20:59] ccogdill: perfect! Check out the one I just forwarded u, though--looks like we have lots of issues when the donor isn [19:21:02] (Merged) jenkins-bot: Back-end logic for Amazon recurring donations [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/243360 (https://phabricator.wikimedia.org/T111430) (owner: Ejegg) [19:21:03] isn't actually a contact [19:21:28] so there’s also silverpop’s respondent ID field [19:22:16] awight is the issue you’re seeing the "NULL"? [19:24:26] so you can see your contact id was passed in the URL - your contact record just has a blank first name. but this is good to know. I was under the impression blank first names showed up as blank in our silverpop DB, not null... [19:26:36] aha [19:27:12] but you bring up a good point - there will be infrequent cases where we will email people without civi records [19:27:21] so perhaps we need to use silverpop’s unique ID instead? [19:27:53] Maybe both? Even better if we can switch parameters using some kind of conditional... [19:28:48] it’s technically possible to switch, but makes the email templates tricky to use if all of our links are based on dynamic content... [19:29:31] I can definitely include both; I think the URL will just default to say &contact_id=%%CONTACTID%% if there’s no corresponding field data [19:30:17] Well, I'm not totally convinced that we need contact_id for anything on donatewiki. We can't trust it without adding hashing anyway, so it's not like we can pop up the donor's name or preferred payment method. [19:30:28] Maybe SP respondent id is plenty... [19:30:31] okay [19:30:53] I have a dream that one day all this data will integrate with the civi record [19:30:54] Do you have anything you'd like to do by linking the clicktracking record to the donor record? [19:30:58] but that can wait [19:31:11] which data? clicktracking or the fact that we sent an email... or both [19:31:16] I think it would be helpful to mbeat, and to me for querying reasons [19:31:16] both [19:31:19] k [19:31:28] but like I said, it can wait. that’s a nice to have, not a need [19:31:31] I'm not opposed to leaving contact_id in [19:31:44] It's... honest, at least [19:31:55] well… is that useful at all? if not, we might as well leave it out until we’re doing something with it [19:32:30] It might be useful for deduping, in early 2016 [19:32:35] if it somehow helps with dedupe [19:32:36] hah [19:32:41] okay, we can leave it in [19:32:51] do you want one more email with the respondent id included? [19:32:59] sure! [19:33:36] awight: i totally get it, i do, but i'm looking at the options and we aren't going to do all the tings in the next couple of sprints that we think we are. caitlin doesn't want to rock the boat at this point either [19:34:04] atgo: rock which boat, the Engage one? [19:34:10] yes [19:34:33] i promise you i am talking to all of our stakeholders and balancing what's happening [19:34:36] That's a lot of what I'm anxious about, my conversation with them was a bit scary. [19:34:44] ok [19:34:51] well we can talk about it more, but we have to make some hard decisions. [19:34:55] I don't want to go through this twice, but more importantly, I really don't want them in Civi now. [19:35:26] gotta bite something... [19:35:54] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, WorkType-NewFunctionality: Create an import method for matching gifts and payroll deductions - https://phabricator.wikimedia.org/T115044#1722893 (atgo) [19:35:55] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: [milestone] Engage exclusively uses spreadsheet imports - https://phabricator.wikimedia.org/T114683#1722892 (atgo) [19:46:07] (CR) Ejegg: Back-end logic for Amazon recurring donations (1 comment) [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/243360 (https://phabricator.wikimedia.org/T111430) (owner: Ejegg) [19:46:22] (PS1) Cdentinger: Send IsCVNMEM with PT request [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/245980 [19:53:48] (PS1) Ejegg: Update SmashPig and dependencies. [wikimedia/fundraising/crm/vendor] - https://gerrit.wikimedia.org/r/245985 (https://phabricator.wikimedia.org/T111425) [19:54:18] (CR) Ejegg: [C: 2 V: 2] Update SmashPig and dependencies. [wikimedia/fundraising/crm/vendor] - https://gerrit.wikimedia.org/r/245985 (https://phabricator.wikimedia.org/T111425) (owner: Ejegg) [19:57:38] (PS1) Ejegg: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/245997 [19:58:21] (CR) Ejegg: [C: 2 V: 2] Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/245997 (owner: Ejegg) [19:58:23] (Merged) jenkins-bot: Merge branch 'master' into deployment [wikimedia/fundraising/crm] (deployment) - https://gerrit.wikimedia.org/r/245997 (owner: Ejegg) [19:59:22] !log updated civicrm f25fbe856b92373104985185db77311ea3a4d841 to c7af7634e75eb8702f5e16081a86ab86ce69c7c2 [19:59:26] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log, Master [20:01:33] Sorry! This site is experiencing technical difficulties. <-- mediawiki.org [20:01:56] yoiks [20:01:59] can't log in [20:02:00] hmm, worked for me [20:02:06] trying login [20:02:27] ooh, yep, problem for me too [20:02:37] Me three [20:02:41] bet ops is on it - let's see what's in their chan [20:05:15] dstrine-meeting: ejegg: awight|fud: cwdent: XenoRyet: I've got a bit of an IRL non-work sudden crisis, if it's OK, I'd like to miss standup :( Hope it's OK! [20:06:01] good luck andyrussg [20:20:10] (CR) Eileen: "I think you mean "Why aren't we on 4.6.10 now?" The reason is I started us on core code after it had forked for 4.6.9 rather than the rele" [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244631 (https://phabricator.wikimedia.org/T99838) (owner: Eileen) [20:32:02] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Easy: Have Civi export name data in UTF-8 format - https://phabricator.wikimedia.org/T90834#1723175 (Eileenmcnaughton) I just tested this on staging on 4.6 & it seemed fixed to me [20:33:14] hey cwdent dstrine-lunch what do you think about sending an email to fr-online about WorldPay update? We've still got the 2 blocking issues (audit + merch ref 2)... it would be good to have a conversation about timelines and such [20:33:27] i can talk to them in the standup tomorrow but i like the idea of having things written down [20:33:35] sounds good to me [20:33:38] want to draft something? [20:33:44] or should i? [20:34:16] atgo: also i have this patch https://gerrit.wikimedia.org/r/#/c/245980/ [20:34:38] dude told me to do that, i'm not sure why, maybe to get cvv check working better [20:34:49] but we haven't tried it live yet so there's every possibility it could explode [20:35:24] (CR) Awight: "Aha, okay thanks. Do you think that going forward we should try to stay on release branches, or does it not matter?" [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244631 (https://phabricator.wikimedia.org/T99838) (owner: Eileen) [20:35:52] Fundraising-Backlog: Civi MINIMAL Gateway report showing wrong amount for Square $$ - https://phabricator.wikimedia.org/T115393#1723193 (Ppena) NEW [20:38:48] (CR) Eileen: "Did you look at https://phabricator.wikimedia.org/T114831" [wikimedia/fundraising/crm/civicrm] (civi-4.6.9) - https://gerrit.wikimedia.org/r/244631 (https://phabricator.wikimedia.org/T99838) (owner: Eileen) [20:45:30] eileen: I like your proposal, except that it hides our revision history. I suppose that works, as long as we are really disciplined about including bug numbers inline for all patches... [20:46:56] (PS2) Awight: Send IsCVNMEM with PT request [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/245980 (owner: Cdentinger) [20:47:54] (CR) Awight: [C: 2] "Nothing is as reassuring as a random undocumented tweak requested by WP." [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/245980 (owner: Cdentinger) [20:48:23] awight: you mean because we rebase? [20:48:28] yeah [20:48:45] So the history will look like, nothing -> 4.6.10 -> wmf patch [20:49:10] ah, nvm I guess you mean we rebase our entire series of patches... [20:49:27] awight: yeah - hopefully some will drop out [20:49:57] I guess if any patches don't go on cleanly we would need to amend the changeid? [20:49:58] wheee! [20:50:26] Good point--the amend workflow would really wreck history in that case [20:51:19] yeah - but it might be good - ie. if we have to resolve a conflict we could see the patch has changed? [20:51:31] or would it cause chaos even with the unchanged ones? [20:51:52] Good point, it will cause chaos. We'll have to create new change-ids [20:52:14] or, work outside of the Gerrit CR process [20:52:21] for every patch hmm [20:53:40] This is a sort of minor issue, as you said we should be upstreaming everything in the long run. [20:53:53] If you look at https://github.com/fuzionnz/civicrm-core/commits/4.6.9-wmf you can see all the wmf -specific are at the top & then there are a bunch of patches that Fuzion has backported & then you see 'normal' 4.6 - probably on the second page [20:54:08] (Merged) jenkins-bot: Send IsCVNMEM with PT request [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/245980 (owner: Cdentinger) [20:54:19] Maybe we can do code review and history tracking on patchfiles? [20:54:31] the issue is the timing of upstreaming. My attitude is everything in our 4.6 must be in core by 4.8 [20:54:41] cool. [20:55:21] & if you look at that commit history I would get rid of the reverted patch altogether next rebase [20:56:00] Re code review - I guess normally you would just review the merge commit [20:56:23] which would contain merge resolution changes -but they wouldn't really leap out would they? [20:57:03] Ideally, the original patch and the fix-up would be separate commits. [20:57:34] Merge resolution changes do jump out in Gerrit, but not so much in regular git. [20:58:02] Fundraising Tech Backlog, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Easy: Point amp cache to $WORKSPACE - https://phabricator.wikimedia.org/T91911#1723306 (Eileenmcnaughton) This is the PR to allow $AMPHOME variable to override the config files path https://github.com/totten/amp/pull/29... [20:59:31] Reviewing patchfiles as a diff sucks, too, unfortunately. [20:59:45] awight: so if we did put the upstream-based branch into gerrit the first issue would be that there would be masses of commits to review [21:00:15] cwdent: ha.. ok [21:00:30] eileen: That part is OK. We can choose to not do CR on a branch, allow direct push, etc. [21:00:32] cwdent: i'm happy to send something [21:01:52] awight: so we would wind up with something like this each point release showing the changes in the new tarball https://gerrit.wikimedia.org/r/#/c/244631/ [21:02:22] but what we would lose in the code-review is seeing any specific amendments made to a patch to cope with upstream changes [21:03:47] Considering that this is a withering, vestigal thing, lemme retract my concerns, I'm fine with just refreshing the patches in some ad-hoc way until they go away forever. [21:05:32] brutal: https://twitter.com/bartt/status/653946266938818561/photo/1 [21:06:14] :-) [21:07:20] awight: I just spotted a theming issue on 4.6 (I'm just looking at some of the civi tickets & trying to replicate them on 4.6 as a way to test it a bit) & if I go to edit a phone number it goes off the side & is a bit awkward [21:07:31] cwdent: nasty watching the vultures circle in [21:07:59] (because Civi is not great with fixed-width themes) [21:08:55] eileen: Let's change the theme! [21:09:21] I don't like the default, and we might as well start fresh since the CSS tweaks didn't port [21:09:33] that's what I was thinking - I was just enabling civicrm_theme [21:11:44] What? "ReleaseTaggerBot added a project: WMF-deploy-2015-10-20_(1.27.0-wmf.4)." [21:11:47] eileen: oh, cool! [21:12:06] Yeah that would be great to make it part of the staging environment, so we can test that at the same time as the other logic. [21:12:47] awight: the module just allows you to define a different theme for CiviCRM [21:13:13] oh. Well I'm fine with the new theme applying to Drupal as well, nobody has any expectations there. [21:13:16] Some people use fubik as a theme for civi & I quite like that [21:13:21] It's probably best to be consistent, really [21:13:22] ok! [21:13:32] garland is the one we normally use - ugly but works [21:13:45] I thought we were currently on garland? [21:14:13] hehe, recently-cauterized-edge technology, "318 sites currently report using this theme" [21:14:52] ejegg: you got the SP email? [21:15:10] Missing required key 'data-store/refund/class' [21:15:19] not yet... [21:15:28] ok, lemme see what's up [21:15:35] I just stuck it on garland [21:15:42] ejegg: I... don't know how to stop that job [21:15:59] oh dang, right, I forgot the refund queue wasn't defined in settings [21:16:04] fixing... [21:16:12] amazon should try again in an hr [21:16:49] eileen: Ah, I see--we're defaulting to Bartik, for no real reason. [21:18:40] running upstairs for a few minutes. [21:23:26] hmm - not pretty - but a bit more width now [21:24:14] (CR) Ejegg: "Looking good! Refunds and currency code need a little tweaking." (5 comments) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/244486 (owner: Cdentinger) [21:29:31] Wikimedia-Fundraising-CiviCRM: Add "mobile" to list of phone type options on a contact record - https://phabricator.wikimedia.org/T89017#1723455 (Eileenmcnaughton) {F2717667} I'm just looking through tickets & trying them on 4.6 as a way to test them out. In this case CiviCRM sees mobile as a phone type ra... [21:35:19] Fundraising Sprint Lou Reed, Fundraising Sprint Miles Davis, Fundraising Sprint N*E*R*D, Fundraising Sprint ODB, and 6 others: Create an Civi import function for hand keyed (including DAF) donations - https://phabricator.wikimedia.org/T88836#1723501 (LeanneS) Updated templates to have no restrict... [21:43:13] hey all. CaitVirtue just sent an email claiming civi is really slow for her. would anyone know what's up? [21:44:07] I don't have access to prod - but presumably some run away query? [21:44:45] I just ran a search at lightning speed, dtirne [21:45:01] I’m not at the office - maybe connectivity issues there? [21:45:27] hmm [21:46:16] I like this nick better, dstrine :p [21:46:35] yeah I just saw an abnormally fast query happen, from the office [21:46:49] maybe they are all sucking away CaitVirtue's life force? [21:47:10] heh perhaps… no giant queries over here, promise [21:48:06] awight: https://media1.giphy.com/media/6aQCRnThnAi1a/200_s.gif [21:48:17] umm ok [21:50:39] resopnded [21:50:40] thanks [21:51:16] dstrine: whoa. thanks! [21:52:04] (CR) Cdentinger: WIP WPG RECON (SMASHPIG) (4 comments) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/244486 (owner: Cdentinger) [21:52:29] dstrine, awight, ccogdill: Is it something like this that's going on? https://www.youtube.com/watch?v=Tt6tYdo8Olg [21:52:36] * K4-713 is helping [21:53:44] K4-713 you know the software better than I do - good to finally know what’s going on in the backend during these multi-minute queries [21:54:03] Yep. There's a staring cat in there. [21:54:08] I will be more patient now that I know it’s a cat [21:54:14] * K4-713 pokes cat [21:54:39] * ccogdill needs to learn how to effectively poke cat [21:55:39] ccogdill: btw, what does audit do that you need for emails? [21:55:40] Fundraising Tech Backlog, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Performance: CiviCRM queries become slower over time, require manual "flushing" - https://phabricator.wikimedia.org/T87032#1723598 (Eileenmcnaughton) I just took a look at the CiviCRM job to clear caches (in 4.6 - not su... [21:56:02] ccogdill: In this case, it should only be filling in donor data for c. 5% of donations. [21:56:03] am I going nuts, or does php have a bug where you can't use a constant after instanceof? [21:56:07] awight maybe I have it wrong - I’m assuming that means we won’t have the new donation records in the civi/silverpop file [21:56:26] const MESSAGE_CLASS = 'blah'; [21:56:28] ccogdill: yeah, just means about 5% will be missing from both civi and silverpop [21:56:31] oh awight - I thougth from your email that possibly *all* WP transactions wouldn’t be in civi [21:56:33] ah [21:56:35] ejegg: gah. I thought I saw that happen. [21:56:41] my response should have been less dramatic, apologies [21:56:54] then in execute(): if ( !( $msg instanceof self::MESSAGE_CLASS ) ) { [21:57:25] syntax error, unexpected 'MESSAGE_CLASS' (T_STRING), expecting variable (T_VARIABLE) or '$' [21:58:12] but if I assign $c = self::MESSAGE_CLASS and test for instanceof $c, it works fine [21:59:28] funky! [21:59:33] world-class parser [21:59:34] ejegg: http://stackoverflow.com/questions/3002594/php-instanceof-failing-with-class-constant [21:59:40] just terrible [22:00:02] $this-> was legit, PHP is fine with ignoring staticness :( [22:00:35] yeah, just reading it... parser is so lame! [22:00:35] or static var... ugh [22:01:25] awight: $this-> was giving me PHP Fatal error: Class name must be a valid object or a string [22:01:27] Fundraising Tech Backlog, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, Performance: CiviCRM queries become slower over time, require manual "flushing" - https://phabricator.wikimedia.org/T87032#1723611 (awight) That actually sounds ideal, cos I wouldn't want to wipe out hard-won recent quer... [22:02:02] going back to get_class [22:02:21] offends me less than changing the const to anything else [22:03:24] (PS2) Cdentinger: WIP WPG RECON (SMASHPIG) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/244486 [22:06:39] That task is shouting at me. [22:07:39] Also it looks like you're whipping recon pigs. [22:07:44] Or something. [22:09:03] heh [22:09:07] From the 501st tactical swine devision? [22:09:25] (PS1) Ejegg: Fixes for Amazon IPN listener [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/246104 [22:10:01] K4-713: i don't show you as a review on there, wonder why it's bugging you [22:10:06] *reviewer [22:10:19] i added myself to a few more things https://www.mediawiki.org/wiki/Git/Reviewers [22:10:35] cwdent: I was just reading the backscroll. [22:10:39] No email. [22:10:46] atgo: would be good to get your feedback on the reports on staging. (note to open without the query running the report urls should have criteria=context on them - there are a few that don't - they seem to have been duplicated by 'something') [22:10:56] oh, whew [22:11:15] XenoRyet: Do we know anybody that makes patches? Because I'd totally want to advertize that. [22:11:32] cwdent: ...why, should I *not* be a reviewer on that? :p [22:11:43] (PS2) Ejegg: Fixes for Amazon IPN listener [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/246104 [22:11:47] (CR) Awight: [C: 2] Fixes for Amazon IPN listener (2 comments) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/246104 (owner: Ejegg) [22:11:49] We had some made when we were doing the Ref Squadron thing... [22:11:55] muahaha [22:12:32] does nobody make cloth unified diff patches yet?? [22:12:42] * K4-713 explodes [22:12:53] that would be dope [22:12:55] iron-on for the lazebots? [22:13:13] punx0rz [22:13:23] Turns out, it would be impossible to search for, so there goes the kickstarter campaign. [22:13:39] +"iron on" [22:14:11] I want a +2 patch. [22:14:31] And, yes. These things could exist right now and we would never ever find them. [22:14:40] It's beautiful. [22:15:25] awight: aargh, I need to fix one more thing [22:15:35] hehehe [22:19:09] K4-713: it would be really easy to get a patch made with the FB hand pointing thumbs down ... that's like a -1 [22:19:33] Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM, WorkType-NewFunctionality: Create an import method for matching gifts and payroll deductions - https://phabricator.wikimedia.org/T115044#1723678 (awight) We do have the option of, kicking Engage out of Civi and then not doing anything technical on ou... [22:20:09] http://www.cryptic-forest.com/gfx/shop/products/thumbsdown-patch.jpg [22:20:31] http://i.dailymail.co.uk/i/pix/2011/01/29/article-1351782-0CF6E157000005DC-831_468x313.jpg [22:20:34] screw this [22:28:28] damn [22:28:33] (PS3) Ejegg: Fixes for Amazon IPN listener [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/246104 [22:30:48] Phew! Crisis dialed down for about a 1/2 hour at least, though far from resolved [22:30:50] atgo: thanks! [22:31:54] awight: cwdent: XenoRyet: ejegg|vps: I'm still trying to fix my civi installation on vagrant, anyone have a sec to help me figure it out? [22:32:13] What I'm not getting right now is how to make drush work [22:32:23] Maybe I shouldn't be caring about it so much, dunno... [22:32:36] AndyRussG: yes I can help [22:32:43] awight: wooho thx! [22:32:47] Glad the non-work stuff did not eat you! [22:32:56] awight: it is still trying.... [22:33:11] gah [22:33:19] i can lend a drush ear too [22:33:33] the biggest gotcha is that you have to be in the crm/drupal directory [22:33:44] then, do something simple like "drush pml" [22:33:45] Rrrrg absolutely unbelievable. A kid in my younger daughter's grade making death threats... [22:33:50] Wat [22:33:53] Grade 3! [22:33:56] Yeah [22:34:01] and in canada no less [22:34:06] That happens. hopefully Canada doesn't lock them away for life? [22:34:21] Irony of ironies I tell ya [22:34:30] Yeah I had to rush off to talk to the principal before he left [22:35:37] The crazy thing is that Mariana, with her psychologist's brain, believes there is something truly wrong with the kid who made the threat, and that there's an actual risk of violence [22:36:41] Rrrg at least our daughter doesn't seem upset, but then, she does tend to keep her feelings to herself sometimes [22:36:42] His parents probably have something wrong as well... [22:36:53] Her [22:37:03] oof, sorry to assume [22:37:18] 3rd-grade girls can be just as murderous as anyone [22:38:17] way to assign gender roles [22:38:32] http://media.comicbook.com/uploads1/2015/02/chucky-124140.jpg [22:38:32] it's OK. I guess they do. Cecilia and her were friends last year, so we actually know them a bit... Though since last year saw some issues with the friendship, and this year's it's way crazy [22:38:51] Speaking as an ex-3rd grade girl: This surprises me not at all. [22:38:58] http://images5.fanpop.com/image/photos/29200000/Bride-of-Chucky-bride-of-chucky-29211673-1024-576.png [22:39:25] I found this new doll... it speaks to me [22:39:29] K4-713: I don't think you ever uttered death threats to other 3rd graders that were completely not in jest, or appeared so [22:39:45] Yeah, but I *knew* other 3rd grade girls. [22:40:14] Who would? [22:40:24] Mariana is like, Cecilia is _not_ going back to school [22:40:34] I'm also hesitant [22:41:03] Or rather, I just don't know what I think yet [22:41:03] doesn't sound like fun! [22:41:26] eileen: thanks 4 the sympathy!!! Yeah it's awful, I dunno what I'll do [22:42:13] Tomorrow supposedly the prinicipal will talk to our daughter to hear about it directly tomorow. Other friends of my daughter also heard the threats [22:42:54] I hate the fact you can't fix these thins for them [22:43:21] indeed... [22:44:09] Though one does have to do everything one can! Rrrrg if I only knew what... [22:46:05] I guess the most important thing is whether you feel happy with how the school is handling it [22:47:42] (CR) Ejegg: Fixes for Amazon IPN listener (2 comments) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/246104 (owner: Ejegg) [22:47:51] eileen: yeah! Well... we'll see tomorrow I guess! [22:48:02] rrgkjlhdfshjdfhgdfkjhgf [22:48:19] oy... best of luck, AndyRussG [22:49:02] ejegg: thanks! [22:49:15] (CR) Awight: [C: 2] "/me marvels at how you snatched this from the merging jaws" (1 comment) [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/246104 (owner: Ejegg) [22:50:01] thanks awight [22:50:14] little fix for crm coming up too [22:50:27] awight: is there anything I can do to help get the messages for testing - I feel like I can't do that much on the testing side [22:50:41] awight: mm to take you up on ur offer to peek at my civicrm vagrant install... So, I see I have these two directories with an install, one is /srv/org.wikimedia.civicrm, and the other is the same but with /vagrant tacked on at the beginning [22:51:18] (Merged) jenkins-bot: Fixes for Amazon IPN listener [wikimedia/fundraising/SmashPig] - https://gerrit.wikimedia.org/r/246104 (owner: Ejegg) [22:52:04] AndyRussG: that is nasty. XenoRyet: do you know what finally happened there? [22:52:14] the /srv directory seems like an accident. [22:52:41] awight: yeah, I followed his advice and changed the server config to point to the one under /vagrant. That's the one that's shared in any case [22:52:53] (PS1) Ejegg: Make smashpig settings file optional, fix comment [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/246121 [22:52:57] However, drush only recognizes the one directly under /srv [22:53:01] I never did figure out why it happened, but pointing my virtual host at the /vagrant got everything working for me. [22:53:03] gah [22:53:19] It defaults to use the one under /var, though I can't figuer out where that's configured [22:53:27] You all shouldn't have to touch config inside of vagrant, if you do it's really not OK [22:53:30] When I force drush to look at the one onder /vagrant, it gives me these errors [22:53:54] If you test stuff and determine that it should be the other directory, then we need to change the puppet files that create the config [22:54:10] We need to have working boxes after destroying and recreating the vagrant instance... [22:54:17] True [22:54:31] I'm happy to do that any time, just make a task that explains what should be changed... [22:55:26] Well since drush is working on the /srv one and not the /vagrant one, I'm a bit inclined to change the server config back, and rather make a symlink to expose the one under /srv, and use that one [22:56:45] eileen: Hmm, there is a huge subtask that you could tackle if you feel like it... which is to do some maintenance and expansion of the phpunit tests for processing queue messages. [22:57:24] If that sounds... fun... I could outline what's missing there. [22:58:11] AndyRussG: This is scary. Starting a new vagrant instance to see what's going on... From what I remember, the /srv stuff isn't shared with the host OS, which makes it a pain to develop. [22:58:44] awight: what's the link [22:58:54] awight: right... I mean, I was gonna make a symlink in the directory that is shared [22:58:58] eileen: mmm. there is none [22:59:26] eileen: This is where I was going to start adding meat: https://phabricator.wikimedia.org/T114466 [23:00:57] !log updated SmashPig from b5ff2a7d5f17aaaa33a169ca101cbea639769c90 to c431c8d77521270236c72532a50806b2e852cf7b [23:01:00] Logged the message at https://wikitech.wikimedia.org/wiki/Server_Admin_Log, Master [23:03:08] so awight I'm not sure how to grab sample messages [23:04:45] Rrrrg my 1/2 hour is up, now it's help-with-homework time, then supper...! Back "here" in 90 min or so.. Thanks 4 sympathies and help all!!! [23:04:54] :-) [23:04:59] awight: if you find any juicy vagrant info, I'll get backscroll in any case [23:05:01] eileen: This might not be the best thing to hand off... [23:05:07] AndyRussG: ok, sorry to miss the bell there [23:05:38] awight: np please don't worry :) [23:05:41] & thx [23:06:21] awight: am I right in thinking that the most critical things for the upgrade is the queue keeps working & the UI stays up most of the time [23:06:42] relocating, back in a bit... [23:10:36] eileen: Yeah, and I think I rambled about this already, but the scariest thing is that we really don't want to roll back if anything goes wrong. [23:10:57] Wikimedia-Fundraising-CiviCRM: Make "Primary" address default address to export - https://phabricator.wikimedia.org/T88446#1723926 (Eileenmcnaughton) I just tested this on the staging 4.6 instance & couldn't replicate it - so it may be fixed! Might be a good thing for others to try as part of testing the up... [23:12:05] awight: yeah - so I guess the question is what would be so wrong we couldn't deal with it [23:12:20] obviously if performance is so bad we can't keep the interface up [23:12:29] Good way to put it. We can turn off all the queue jobs if necessary. [23:12:29] or the import completely chokes [23:12:44] They can stay off for maybe 24 hours before we really start making people's lives unpleasant. [23:13:04] ok - scenarios [23:13:07] or the mass of contraptions we've built upon it can't do its job [23:13:24] The upgrade itself is a major issue too--like finding out that we've corrupted something about the database after a week. [23:13:48] yeah - some sort of dataloss in the script you are thinking? [23:14:17] yep [23:14:49] Also, I see you did a ton of work on https://phabricator.wikimedia.org/T114831 , I'll try to reply. That would be a major piece that needs to be resolved before we can upgrade. [23:15:11] I have doubts we can protect ourselves from that more than having a backup & writing a data-recovery - in that the upgrade has been out in the wild for a while so the changes of use finding something that hasn't been found by ad hoc guess-testing [23:15:14] is low [23:16:07] agh my computer is back to not charging. i may disappear at some point. hopefully it's just my adapter but... [23:18:13] eileen: Yeah, with any scenario like that, my biggest concern is for hand-entered data. [23:18:30] We have no way to audit what happened, so all the Civi admins will have to redo their work. [23:19:24] awight: right - although my guess is we would be going the other way - figuring out how to re-create 4.2 data into 4.6 [23:20:09] oh, in case of data loss. yes. It depends on the type of failure [23:20:52] e.g we had a scenario recently where a customer deleted a location_type & it turned out the rather cryptic message meant 'you have contacts with addresses with this location type and we are going to delete them if you click yes' - which they duly did [23:21:28] so we wound up writing nasty sql to re-introduce the lost data to their live DB [23:21:58] I'd like to not go there. Consider that each query could take hours. [23:23:24] I agree [23:23:38] (CR) Awight: [C: 2] "Hrm, I thought the override file was already called config.php." [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/246121 (owner: Ejegg) [23:28:52] (Merged) jenkins-bot: Make smashpig settings file optional, fix comment [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/246121 (owner: Ejegg) [23:35:29] civi looks snazzy eileen :) [23:41:15] ooh, modern [23:44:54] Fundraising Sprint UB40, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: All Civi hook code paths must be covered by PHPUnit tests - https://phabricator.wikimedia.org/T115426#1723980 (awight) NEW [23:46:34] (CR) Ejegg: "That's just a convention hardcoded into the HTTP request handler. I figured it was nicer to let you configure SmashPig right next to wher" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/246121 (owner: Ejegg) [23:49:05] (CR) Awight: "Let's not make the path a setting... Per T113301, I'd like for all components to look in /etc/fundraising for their config file. It shoul" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/246121 (owner: Ejegg) [23:50:13] Fundraising Sprint UB40, Fundraising-Backlog, Wikimedia-Fundraising-CiviCRM: Spot-check that all core and custom data is preserved during the migration - https://phabricator.wikimedia.org/T115427#1723997 (awight) NEW [23:50:26] agh configuring IRC is the worst. [23:52:48] (CR) Awight: "Oh, I see what you were saying about Civi and Drupal settings. Yeah, we already have a sketchy require_once which pulls in an /etc/fundra" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/246121 (owner: Ejegg) [23:53:33] considering the WPG audit files may not be showing up for 48 hours i think it's safest to assume we won't have them before the thursday test [23:54:04] Can we at least assume that the ones you downloaded will be the same? [23:54:08] we could download from the portal and run manually, hopefully it's safe to assume it will be the same data we'd receive via sftp [23:54:14] jinx ;) [23:54:19] aah heh yeah [23:54:38] atgo_: cool! [23:55:03] i'm wondering if there's a way to test more thoroughly locally, or if we should just deploy and try it [23:55:29] cwdent: We can leave it off until we've received a file and tested that. [23:55:29] eileen: did it work? [23:55:36] i think i'd have to download all the transactions from the db as well as the log files which sounds ngarly [23:55:40] cwdent: I usually leave the download job running [23:55:57] atgomez: yep - I think I'm supposed to get you to confirm by email that it was really you :-) [23:56:13] cwdent: and for logs, yeah I usually find a log from one of the days, and use that [23:56:56] ok yeah, so maybe trimming the report down and isolating something that should change is the best approach [23:56:56] eileen: atgomez: or gchat ;) just any separate communication channel [23:57:30] I'm a little confused when you say "amend" and "change"--that's a feature we haven't written yet. [23:57:48] cwdent: ^ currently, all that happens is the audit processor will fill in any contributions which are missing, but it won't amend anything. [23:58:32] awight: ah ok, so shouldn't it have imported _everything_ when i ran it locally? i don't think it did [23:59:11] cwdent: Does it need to incorporate pending queue messages or loglines? [23:59:17] That would be my first guess... [23:59:53] hmm that's beyond me at this point... does it try to cross reference from multiple sources before it will import?