[00:01:30] hahaha [00:01:33] at least it's possible [00:01:42] i actually really like the 1 task : many projects thing [00:01:48] it's a little confusing but i think it's goign to work great for us [00:02:09] hell yeah [00:06:11] K4-713: at what point after the cc icons are disabled are we ok with re-enabling? [00:06:24] the user might need to change their choice etc [00:07:01] the card says "until the request returns with something" [00:07:31] so on success in the ajax I'm assuming? [00:10:15] it's kind of hard testing locally [00:17:42] (PS1) Ssmith: Fix double click issue [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179353 [00:17:44] (PS1) Ssmith: Prevent double clicks on gc cc forms [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179354 [00:18:04] (Abandoned) Ssmith: Fix double click issue [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179353 (owner: Ssmith) [00:20:10] ^screwed up the merge [00:24:42] blah everything out of order when I try to squash/interactive rebase [00:24:55] I am going to be a farmer [00:24:59] bye [00:25:02] hey ejegg K4-713 - for the new GC CSS, i know you were also looking at tying that in with the RTL issue [00:25:07] pizzzacat: things will grow with all this wayter! [00:25:09] water. [00:25:40] not sure what the new GC CSS is! [00:25:54] :p so much water [00:26:27] yeah ejegg and K4-713 were looking at it [00:26:51] oh I see now you didn't address that to me [00:26:57] atgo: they can't put a language on the css url, unfortunately [00:27:05] gotcha [00:27:18] so we can't do it in the iframe, but we can still do it for the rest of the form, right? [00:27:40] atgo: we can, but I didn't mix that in with the current patch [00:27:52] the current patch for which? [00:28:20] the current patch adding the css from the campaign to both gc gateway css and iframe css [00:28:36] I'll keep looking at that in the morning [00:28:47] cool [00:28:48] sounds good [00:29:07] Gotta head out for now. Have a good evening, y'all, and try to stay dry! [00:29:11] ciao! [00:29:20] i haven't even opened a door that leads outside today.. [00:30:50] aww haha I did…it wasn't a great idea [00:31:13] I'm glad it's not snow! [00:32:50] pizzzacat: Ah, sorry. IRL got me. [00:33:29] To answer your question: Yes, we want to re-enable the cc radio buttons after the iframe loads. [00:34:24] Hopefully not before the content actually loads, or... during. [00:36:16] atgo: I think ejegg answered everything you were asking. Anything left? [00:36:32] yeah i think i'm good - thanks [00:36:34] ok, I submitted a patch and hopefully I got that right [00:36:52] pizzzacat: You said you were having trouble testing? [00:36:55] if the RTL thing was going to be the same work again when we wanted it, then i'd say we should tie it to this. but it sounds like it's basically totally separate [00:36:58] just made it happen in the success function [00:37:01] and the error [00:37:11] K4-713 I think I got it sorted [00:37:39] I was getting errors from the form trying to connect to the actual services, or something [00:37:44] aaaha. [00:37:51] Are you coming in tomorrow? [00:38:05] It's slightly easier to use the sandbox if you don't have to VPN in. [00:38:15] ejegg|away: re: RTL, the only possibility I saw was that we somehow encode language into the merchantreference, which is available in the resourceloader URL if we ask for that. [00:38:41] ejegg|away: on the other hand, K4-713 is slowly twisting their thumb about the doc text that said we can switch css on language. [00:38:53] awight: Very slowly. [00:39:06] I basically stopped. [00:39:06] I noticed. I'm full out of popcorn. [00:39:38] Think of it as a dramatic pause. [00:39:57] Pinteresque [00:40:17] yeah it makes sense, it will be more effective once they think the coast is clear again. [00:40:26] pizzzacat: This is sort of what I got stuck on earlier. [00:40:49] The issue is, the gc form loading in the iframe takes... a silly amount of time sometimes. [00:41:06] ...aw. [00:41:22] * K4-713 twiddles [00:41:23] K4-713: awight i think it's ok to let the iframe being RTL go if it's not super simple [00:41:33] but we should at least be able to improve the top half relatively easily [00:41:35] right? [00:41:48] atgo: Yeah, I think they're still trying to work out if their system supports what their docs said they did. [00:41:53] cool [00:41:58] Until they do that, we're stuck with "no". [00:42:03] i mean, not cool, but whatevs [00:43:07] Yeah, I can't believe... [00:43:14] ...what kind of collect did you say? [00:43:16] :D [00:43:29] That joke never, ever, ever gets old. [00:43:40] ejegg|away: I'm getting no dice on the https from boron, either. [00:45:12] sorry, internet [00:45:20] yo [00:45:23] the wind is blowing it away [00:45:37] My bandwidth is sublimating! [00:45:39] Sad. [00:45:57] Anyway. I was going for: [00:45:59] ha [00:46:08] The iframe content takes silly amounts of time to load sometimes. [00:46:25] were you asking me if I was coming in tomorrow? because yes I plan to [00:46:31] I see [00:46:37] And, right now, the place you're re-enabling the cc radio buttons will do it right after the form starts trying to load. [00:47:13] I tried that, and it looks pretty funky. :/ [00:47:24] hmm so the success callback approach isn't sufficient here? [00:47:30] Sadly, no. [00:47:39] because there''s one more step [00:47:42] The data coming in with that ajax call is just the URL to try to load. [00:47:43] ok. [00:47:49] word [00:48:09] But. [00:48:21] The one on the error condition should totally stay put. :) [00:48:49] awight: https://wikimedia.mingle.thoughtworks.com/projects/online_fundraiser/cards/1847 ?? [00:49:07] ok. [00:50:19] atgo: what what [00:50:28] <_< [00:50:28] We really should have a way to fake this whole thing on dev installs. [00:50:50] awight: where'd that come from? [00:50:57] looks like there has to be a callback with generatePaymentForm() [00:50:59] atgo: my skunky wish list [00:51:05] * K4-713 copies my last statement, pastes in to every outstanding mingle card [00:51:21] womp womp [00:51:22] haha K4-713 :) [00:51:55] * awight tries to be cool and pastes "my skunky wish list" onto all mingle cards... [00:52:03] That's more like a tag. [00:52:04] btw your cat of the day, team: http://img.pandawhale.com/post-42136-kitten-looking-around-corner-g-vTFg.gif [00:52:10] awwww [00:52:46] That kitty looks deeply conflicted. [00:52:50] http://media.catmoji.com/post/vv4f/catzilla.jpg [00:53:27] awight: is there some urgency for that one/ i get that it's super important/helpful.. but there's a bunch of stuff that it would've been great to get in the sprint plan yesterday that we didn't accept [00:53:32] that / was supposed to be a ? [00:53:42] atgo: u can pile in anything you wish [00:53:46] I was gonna ask... [00:53:55] but thought sneaking off and just doing random things would be more helpful :p [00:53:59] hahaha [00:55:01] here's my deal... i'm concerned about next quarter and trying to make sure we're in a position to only promise what we can deliver [00:55:18] but in order to do that, we sort of need to scope those projects, which is mostly what i've got as high-priority right now [00:55:19] * K4-713 shudders at the word "promise" [00:55:32] yeah... me too K4-713 [00:55:40] that's probably too strong :) [00:55:41] atgo: sounds real. Can you point at the cards? [00:55:47] but you know what i'm trying to get at? [00:55:58] https://wikimedia.mingle.thoughtworks.com/projects/online_fundraiser/cards?favorite_id=12220&view=Sprint+planner [00:56:02] fwiw this is groundwork for the 4.4 upgrade. [00:56:08] here's what i was thinking for the sprint, it's mostly review docs and whatnot [00:56:10] not super fun, i know :P [00:56:20] yeah those look boooring :p [00:57:28] yeah, i gotcha [00:57:47] no... seriously, I'm happy to work off of that list. [00:58:11] i get the want for CI, too [00:58:31] CI is a must-have for Civi 4.4, if I mentioned... [00:58:45] to do the upgrade? [00:58:53] we can't upgrade without protection against regressions... [00:58:56] yah [00:58:59] yeah, that makes tons of sense [00:59:11] if it's ok to revisit that in a couple of weeks, that would be super helpful to me [00:59:36] because astropay + china + refactor [00:59:49] = PM worry [01:00:11] hah [01:00:53] I'm still recommending we don't attempt new integrations... [01:01:14] in general? [01:01:17] yeah [01:01:24] ? [01:01:27] we have serious technical debt issues. [01:02:00] and, as has been said: there are already new toys laying around that no one plays with ;) [01:02:06] yeah word [01:02:18] so the technical debt is something the whole foundation wants to address [01:02:30] (yay! also.. scary!) [01:02:43] and.. are you talking about adyen? [01:02:51] This is a silly time/place for this conversation, I fully understand that your job is to make sure things proposed by both tech and non-tech move forward ... [01:03:13] ha ok [01:03:34] i am super happy to talk about this more though! [01:03:40] adyen, worldpay, coinbase... [01:03:51] actually, all the new integrations we've done in the past 2 years [01:04:03] we're actively using 2 of those... and adyen i'd like to figure out [01:04:06] wat. [01:04:23] $700/week for coinbase and we can't solve the simplest issue: people hate us for the mandatory address fields. [01:04:28] yeah. that one is silly [01:04:38] and worldpay just... smells bad. [01:04:41] hahaha [01:04:45] so does Honey, but i love her anyway. [01:05:00] and her ability to literally sleep all day. that dog hasn't moved today. [01:05:20] atgo: can you rank the sprint planner cards, or choose a top 3 to pull into the now? [01:05:42] yeah they're more or less ranked.. let me grab a couple [01:05:52] that paypal link for nirzar thing is an easy win, too [01:06:01] ok, fraud, alipay, and CUP? [01:06:27] are those things interesting enough? :) [01:06:46] we could look for something a bit less boring, to throw into the mix, too [01:06:55] i know this is a ton of documents [01:07:20] atgo: still, I'm bothered, and non-IRL makes this part of the conversation difficult, but I not quite buy the argument that any unused capacity (me) should be applied to the next sprint, if these are things we were already planning to not do... [01:07:24] does that make sense? [01:07:34] yeah, i get that [01:07:51] i think that's also a note for better shrimp prawning though, too [01:07:55] meh [01:08:01] how about this, let's do fraud + alipay + CI? [01:08:14] prawning was just yesterday [01:08:39] I mean... I'm looking at a thing at 5pm on thursday with nobody here, which will require lots of kicking of the can between parties before it's usable. It still makes sense for me to squander the remaining hour on CI. [01:08:49] ok [01:08:53] sigh [01:08:57] that's fine [01:09:07] i'm just worried about the future :) [01:09:11] baahaha [01:09:12] that's my job [01:09:13] I noticed [01:09:19] me too. [01:10:28] yes [01:10:37] CI will save us from ourselves and everyone else. [01:10:37] it's good that we both worry about different sides of the same coin :) [01:10:52] yeah, i'm totally on board for the importance [01:10:53] Maybe save some people from us, too. [01:10:59] Rar. [01:11:03] ROOOOAR [01:11:11] I was being subtle. [01:11:19] i'm over sublety [01:11:40] mew. [01:12:22] honey just turned around and flopped in the exact same place. [01:12:24] to be a dog... [01:13:08] so in conclusion, CI yes, let's also look at Alipay if there's time? [01:13:20] sniff... [01:13:37] sniff? [01:13:42] * K4-713 restrains self from licking that cookie [01:13:46] baa [01:13:48] it's yours [01:13:54] I saw you touch it [01:13:55] CI? [01:14:09] no, K4-713 totally just volunteered to scope Alipay [01:14:14] Alipay. It's just that I told people so emphatically that it was *so easy* through GC. [01:14:14] * awight flags down the bus [01:14:22] So. Easy. [01:14:25] ha then let's just do it! [01:14:30] K4-713: your bus is here! [01:14:33] If it's not, I want to be the first to know. :p [01:15:23] there's also another China thing that might also be easy... called CUP [01:15:29] No. [01:15:33] hahahaha [01:15:39] well let's start with alipay [01:15:46] get that one out of the way [01:15:52] atgo: your job is not easy. [01:15:56] esp if it's as easy as you say K4-713 [01:16:00] awight: :P [01:16:01] * K4-713 dies laughing briefly [01:16:16] is it beer o'clock yet? [01:16:29] ...14 minutes. [01:16:57] shhh [01:17:13] hobbitses. [01:17:42] but for reals, in conclusion, sprint look OK? [01:18:52] I think so. [01:18:53] atgo: I think the 1-week thing is really challenging [01:18:58] ...but, yes. That. [01:19:05] yeah, i was thinking we should check in on that next week [01:19:10] now that it's been a few [01:19:17] (PS1) Awight: add a .gitreview [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179366 [01:19:33] it was something to try, but if it's not working for the team then we don't have to keep it [01:22:26] (PS1) Awight: Squashed merge from upstream/master to master [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179371 [01:23:11] thanks guys [01:23:42] and for a change in subj, awight i saw you moved the IE11 fix with the p3p headers to deployed - good enough that we should let ccogdill know? [01:23:59] AndyRussG: ^^ sorry, I moved your card unilaterally [01:24:08] I kind of agree with the 1-week sprint. don't hate it but I think 2-week ones made more sense [01:24:18] * pizzzacat has been lurking [01:24:23] hehe [01:24:30] ooh yay! [01:24:43] ccogdill: I can't say if it's fixed yet :) [01:24:48] ccogdill: lets confirm with AndyRussG first :) [01:24:51] oohh okay [01:25:04] ccogdill: yeah, AndyRussG was saying he got some hide cookies, but I'm not sure if it was *all* the hide cookies. [01:25:21] lemme undeploy that... [01:33:50] (PS1) Awight: Update repos [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179372 [01:33:52] (PS1) Awight: Enable new modules [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179373 [01:40:16] hide them all! [01:40:19] (PS2) Awight: add a .gitreview [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179366 [01:40:21] (PS1) Awight: manage vendor subdir as a submodule [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179374 [01:45:46] (PS1) Awight: populate with current modules [wikimedia/fundraising/civicrm-buildkit/vendor] - https://gerrit.wikimedia.org/r/179377 [01:46:16] (PS1) Awight: include vendor content [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179378 [01:50:33] (PS1) Awight: Squashed merge from upstream/master [wikimedia/fundraising/civicrm-buildkit/vendor/totten/amp] - https://gerrit.wikimedia.org/r/179379 [01:50:35] (PS1) Awight: add .gitreview [wikimedia/fundraising/civicrm-buildkit/vendor/totten/amp] - https://gerrit.wikimedia.org/r/179380 [01:53:03] (PS1) Awight: replace totten/amp with local fork [wikimedia/fundraising/civicrm-buildkit/vendor] - https://gerrit.wikimedia.org/r/179381 [02:01:26] awight: ccogdill: atgo: looks like it was caching [02:02:13] ejegg|away seemed to have understood the assembly... er, I mean, minified Javascript... oh, no, sorry, it was varnish code... that handles that [02:02:38] and so I think they're cached for a day (?) and should regenerate soon [02:04:56] hrm? [02:07:35] (PS1) Awight: WIP MySQLPrecreated db implementation which does not require admin rights [wikimedia/fundraising/civicrm-buildkit/vendor/totten/amp] - https://gerrit.wikimedia.org/r/179384 [02:21:34] atgo: I meant, regarding the IE hide cookie issue. It seems it'll continue to fix itself as the cache refreshes [02:21:48] And also I made a silly joke [02:28:40] haha ok :) [02:28:50] so we can check in on it tomorrow or monday? [18:36:04] K4-713: ... [18:36:09] it's possible.. ihad my laptop muted [18:36:15] i'm not sure [18:36:21] wah wah [19:00:33] ccogdill: meeting?! [19:00:41] I'm running up there now if so [19:00:42] eek yes, coming now! [19:00:47] ok sure [19:03:08] hey ejegg [19:03:14] you joining the MG meeting? [19:03:18] it was cancelled and uncancelled [19:04:13] oh, I must have the wrong link then! [19:04:23] finding new hangout [19:22:33] awight|fleeting: ejegg: FYI the IE donate hide cookie issue is now solved on production [19:22:41] atgo: ^ [19:23:02] rad! [19:27:50] AndyRussG: awesome! [19:28:05] ejegg: yea thanks for looking into the caching stuff [19:28:32] Didja see bblack's description of vcl? "the assembly language of HTTP" ... 8p [19:29:14] heh [19:34:37] oooh just reproduced the "banner flash and disappear" bug too.. on IE9, with this campaign: http://en.wikipedia.org/wiki/Main_Page?country=DE&uselang=de [19:35:10] also confirmed that current FR banners don't display on IE9, nor do the WMFDE's fancy slider ones [19:37:08] wow! [19:40:57] Yeah could be the same bug, or similar, some DOM injection thing... [19:43:56] (PS5) Ejegg: Update input and select styling for GlobalCollect [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/174617 [19:43:58] (PS3) Ejegg: Clean up GlobalCollect style sheet [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179052 [19:46:55] ^^ should get rid of the iframe scrollbars. It's also applied to the staging css on foundationwiki [19:47:07] ok cool, I'll try it out [19:48:23] Wow, GC facelift day. [19:49:34] (PS6) Awight: Update input and select styling for GlobalCollect [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/174617 (owner: Ejegg) [19:49:40] (CR) Awight: [C: 2] Update input and select styling for GlobalCollect [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/174617 (owner: Ejegg) [19:49:47] (PS4) Awight: Clean up GlobalCollect style sheet [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179052 (owner: Ejegg) [19:49:52] (CR) Awight: [C: 2] Clean up GlobalCollect style sheet [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179052 (owner: Ejegg) [19:50:26] thanks awight! [19:50:33] no, thank you :p [19:51:00] ejegg: the CC fields are centered in a funny way, but still somehow better than before. [19:51:17] What can we do with an iframe working against us... [19:51:32] awight: yeah, not sure why we make the iframe 318px [19:51:35] Hmm, I just noticed that GC uses labels, and we use placeholders. sigh [19:51:47] ejegg: probably, it's the only size GC supports :) [19:52:00] ehh, good enought for now! [19:52:10] (Merged) jenkins-bot: Update input and select styling for GlobalCollect [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/174617 (owner: Ejegg) [19:52:15] (Merged) jenkins-bot: Clean up GlobalCollect style sheet [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179052 (owner: Ejegg) [19:52:56] ejegg: hey, while atgo-brb isn't looking... feel free to chip away at my PCR from yesterday, it's a pain to maintain this mess all on review branches... [19:53:33] Ah, I don't think anybody is going to argue with PCR. [19:53:38] If they do, send 'em to me. [19:53:43] * K4-713 puts on mean face [19:53:53] Ah, civi-buildkit? Looks interesting. [19:53:56] I don't understand... [19:54:09] I reserve atgo-brb's right to argue... [19:54:10] Code Review = Good. [19:54:13] :p [19:54:20] Even if we can't do anything with the result for a while. [19:54:24] unless it's extremely helpful UX fixes to CentralNotice [19:55:00] * awight smokes a fake cigarrette, after considering odd french spelling for a bit [19:55:03] Unreviewed changes are... basically automatically going to rot. And that's bad. [19:55:26] * awight sighs very loudly [19:55:49] ... [19:56:12] I feel that there is a huge difference between unmerged, and pending deployment. [19:56:15] Massive. [19:59:50] awight: The weird thing about this conversation is that I came in to it heartily backing you up. Now it sounds like we're arguing. So confused. :p [20:00:07] Nah [20:00:21] I'm just considering whether this is a good time to punt any other CR over the fence. [20:00:27] My room is a mess. [20:00:35] That's all the rooms. [20:00:41] We're all over the place. [20:01:00] I feel like we're about to have a party with everybody we know in a really f'd up house. :D [20:01:18] on the bright side, I think we're actually good to go with this CI thing. [20:01:25] F yes. [20:01:30] Seriously? [20:01:41] I thought that was going to not be even next week. [20:01:57] I had a despondent moment last night, but now that I scrape together everything that was just laying around, it's... running. [20:02:24] Huh [20:02:45] That's pretty cool. [20:03:08] The big missing piece of glue was, I had to modify totten's provisioning thing to accept pre-created dbs, and we left that patch unresolved. but it works locally. [20:03:12] which is good enuf for me :p [20:03:15] heh [20:03:32] That definitely counts as progress, yes. [20:03:39] Usable progress, even. [20:03:57] hehe thanks awight :P [20:04:05] yipeS! [20:04:06] atgo-brb: Are you coming in soon? [20:04:12] hehehe you'd better [20:04:14] yeah about to leave my house [20:04:24] in like 5... just packing up [20:04:33] Okay. I have a thing I want to merge and deploy, but I want you to see it in action first. [20:04:41] atgo-brb: join our mutiny! [20:04:55] Aw. This was actually on the wall. [20:05:00] oh neat [20:05:05] merge and deploy on a friday, eh? [20:05:10] urgent enough you wan to screenshare or...? [20:05:19] FUNDRAISING, WOO. [20:05:20] (CR) Ejegg: [C: 2] Log missing transaction identities [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178920 (owner: Awight) [20:05:25] Ah, no. It can wait. [20:05:34] (Merged) jenkins-bot: Use currency codes from DonationInterface without copypasta [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178966 (owner: Awight) [20:05:51] ...Didn't I do that yesterday? [20:06:00] * K4-713 frowns [20:06:30] Oh, it had a dependency [20:06:33] Also, why is gerrit trying to run flash? [20:06:36] which I just +2ed [20:06:40] ?? flash?? [20:06:58] eee [20:07:49] ejegg: I think it's been that way forever. It's to help you copy text. [20:07:53] isn't that evil? [20:08:11] [20:08:13] Er... [20:08:16] because I was suddenly too lazy to double-click a field, I actually want that to break, and single-click to have a novel behavior. [20:08:32] (PS1) Awight: Fredge and Donations dbs are in Drupal schema for now [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179534 [20:08:36] What the Flash? [20:09:52] (PS1) Awight: update vendor submodule [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179535 [20:10:01] K4-713: it's to make single-clicking do something novel. [20:10:13] (CR) Ejegg: [C: -1] "missing assignment to $xmlstart" (1 comment) [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178972 (owner: Awight) [20:10:14] Yeah, I was just looking at that too. [20:10:29] ...foodnow. [20:10:56] (CR) Awight: Parse XML charset declaration (1 comment) [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178972 (owner: Awight) [20:11:49] (PS2) Awight: Parse XML charset declaration [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178972 [20:14:34] (CR) Ejegg: [C: 2] Parse XML charset declaration [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178972 (owner: Awight) [20:14:51] (PS2) Ejegg: Always log the message that would be sent to Civi [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178981 (owner: Awight) [20:24:25] (CR) Ejegg: [C: -1] "Looks like there is a progress indication string of '!', 'X', and '.' characters being output. Maybe call wmf_audit_echo with $verbose=tr" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178981 (owner: Awight) [20:48:48] For some reason in IE9 the fundraising div is getting set to display:none... [20:51:25] (CR) Awight: "That's a really tough call. I want the msg contents to show up in our ordinary job logging, actually. I don't see a convenient resolutio" [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178981 (owner: Awight) [20:51:31] ejegg: good point [20:51:40] ejegg: dragging K4|food into the conversation... [20:51:42] awight|lunch: ncurses! [20:51:52] heh [20:51:54] WHAT [20:51:56] hur? [20:52:09] with animations in your terminal [20:52:10] K4-713: hey uh...https://gerrit.wikimedia.org/r/178981 comment thread [20:52:30] aha ANSI foiled again [20:52:55] Hm. [20:53:08] I'm not sure what you mean by " Maybe call wmf_audit_echo with $verbose=true when not in test mode?" [20:53:49] Those char strings are... *incredibly* valuable to have all the time, though. [20:53:51] That's wrt the "message to send" logging, here, https://gerrit.wikimedia.org/r/#/c/178981/2/sites/all/modules/wmf_audit/wmf_audit.module,unified [20:54:17] K4-713: assuming you agree that we should be logging entire msgs... (?) [20:54:25] Sorry, my brain is on another planet right now. [20:54:28] The only compromise I can imagine is to have one of the streams be non-realtime [20:54:31] This is going to take a second. [20:54:33] K4-713: It just looked like you had a nice visual summary going on [20:55:10] so either the hashes are queued in a buffer (annoying when run from the terminal), or the full message dump goes into a buffer. [20:55:41] Generally, I'll observe that it seems sketchy for us to suppress all logging while we're emitting hash characters. [20:55:47] Well, first off, if you run this from the terminal and not jenkins, all the file perms go sideways. [20:55:55] So, don't do that. [20:55:59] nah it can be done. [20:56:04] Yes, it can. [20:56:09] "drush" on the prod server already has a sudo wrapper [20:56:15] which will make it act just like Jenkins [20:56:28] but I'm okay with "don't do that". [20:56:39] Yeah, it's nicer to be able to see all the runs in jenkins anyway. [20:56:44] exactly [20:56:58] ok, then I'm leaning towards buffering the hash characters. [20:57:10] And, jenkins doesn't give you dots or whatever with echochar. It's already entire lines. [20:57:16] Which made me sad, like, a year ago. [20:57:19] whaa [20:57:24] Watch it go. [20:57:26] ?? [20:57:41] weird [20:57:42] The globalcollect wr1 nightly audit script does the same thing. [20:58:23] K4-713: looks like it's emitting characters just fine [20:58:27] /job/Globalcollect WR1 Log Audit/242/consoleFull [20:58:41] If you watch it in realtime, it doesn't do partial lines. [20:58:47] ah [20:58:57] Just waits until you reach the charlength or whatever the var is called that limits to one line. [20:59:23] Go ahead and crack off a GC_wr1 run and see what I mean, if you want. [20:59:46] oh I see. it's buffered by line [20:59:49] aye [21:00:11] I sort of like how the errors are inline with the char hashes... [21:00:15] at the same time as hate it [21:00:19] Same here. [21:00:22] it's a thiiin line... [21:00:23] Anyway, it's not a new probelm. [21:00:28] Or, a problem. [21:00:55] ok thx I'll buffer the hash chars [21:01:22] I used to really like the realtime dots, though. [21:01:40] It's... the only thing that ever gave me the sense of scale on this stuff. [21:02:28] we could compromise, and buffer a line of hash chars at a time :) [21:02:29] It's pretty hairy to detect whether we're running under a console or not [21:02:43] No, that's not worth doing. [21:03:13] ejegg: AndyRussG: We think atgo's on the bus btw, so maybe late standup? [21:03:18] ok [21:03:19] Particularly since we're calling not running in jenkins bad behavior. :) [21:04:00] awight: ejegg: K4-713: pizzzacat: sure lemme know when you're set, then? [21:04:24] * awight hears sounds IRL [21:04:40] I was just talking to my computer. [21:04:44] ... [21:04:51] that happens sometimes. [21:05:09] * K4-713 fidgets [21:05:31] I must be pretty tired. [21:05:46] Hmm I often hear sounds IRL... that turn out not to be IRL! [21:06:02] AndyRussG: ejegg: pizzzacat1: surprise! atgo arrived [21:06:14] Sometimes I hear singing in other rooms that isn't actually happening. [21:06:25] awight: ejegg: K4-713: Ah OK, so standup yes? now? when? [21:06:34] sure! I think. [21:08:04] Soo... should I join the hangout? [21:08:25] yes please! [21:09:22] awight: K coming [21:18:23] awight: sorry, I missed the last thing u said :( [21:18:44] I was glad to see you and ejegg were not replaced by bots yet! [21:19:01] atgo-brb of course added, "or they are very convincing bots." [21:20:19] is ananova still doing the news broadcast? [21:20:40] awight: how do you know? [21:20:41] OK i thought this was funny, found it while searching for "gerrit flash" earlier: https://code.google.com/p/gerrit/issues/detail?id=1956 [21:20:51] shit, the droids know we're onto them. [21:21:14] I had to look up ananova. spooke! [21:24:45] (CR) Awight: [C: -2] "Agreed. So I think we have a solution, to buffer audit characters. Will do soon." [wikimedia/fundraising/crm] - https://gerrit.wikimedia.org/r/178981 (owner: Awight) [21:30:14] (CR) Katie Horn: [C: 2] Prevent double clicks on gc cc forms [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179354 (owner: Ssmith) [21:30:38] pizzzacat: Looks like this needs a rebase: https://gerrit.wikimedia.org/r/#/c/179354/ [21:31:08] ejegg: K4-713: whenever you're up for it, most of those civibuildkit and friends patches should be simple. Confirmed that the code is actually working "enough" [21:31:15] blarg and no rebase button [21:31:41] awight: ok, cool [21:31:50] K4-713 so we are good to go on it? [21:31:58] (I mean, after I rebase) [21:32:32] pizzzacat: Yeuuup. [21:32:51] coool [21:32:57] I may not get it today, though. [21:33:03] But, very soon. [21:33:59] (PS2) Awight: MySQLPrecreated db implementation which does not require admin rights [wikimedia/fundraising/civicrm-buildkit/vendor/totten/amp] - https://gerrit.wikimedia.org/r/179384 [21:40:49] (CR) Ejegg: [C: 2] Squashed merge from upstream/master [wikimedia/fundraising/civicrm-buildkit/vendor/totten/amp] - https://gerrit.wikimedia.org/r/179379 (owner: Awight) [21:40:57] (CR) Ejegg: [V: 2] Squashed merge from upstream/master [wikimedia/fundraising/civicrm-buildkit/vendor/totten/amp] - https://gerrit.wikimedia.org/r/179379 (owner: Awight) [21:41:11] (CR) Ejegg: [C: 2 V: 2] add .gitreview [wikimedia/fundraising/civicrm-buildkit/vendor/totten/amp] - https://gerrit.wikimedia.org/r/179380 (owner: Awight) [21:44:13] Yeah this is definitely some kind of IE9 css bug [21:44:50] For some reason in IE9 the centralNotice div gets set to display:none [21:48:29] (CR) Ejegg: [C: 2 V: 2] populate with current modules [wikimedia/fundraising/civicrm-buildkit/vendor] - https://gerrit.wikimedia.org/r/179377 (owner: Awight) [21:51:20] (CR) Ejegg: [C: 2 V: 2] replace totten/amp with local fork [wikimedia/fundraising/civicrm-buildkit/vendor] - https://gerrit.wikimedia.org/r/179381 (owner: Awight) [21:51:41] ejegg: thx! [21:52:04] awight: walking this tree from the leaves on down [21:54:41] (CR) Ejegg: [C: 2 V: 2] Squashed merge from upstream/master to master [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179371 (owner: Awight) [21:56:38] (CR) Ejegg: [C: 2 V: 2] "much cleaner!" [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179372 (owner: Awight) [22:01:25] hey the-wub... will this Lithuania thing require tech work or only on the front end? [22:02:37] atgo: national currency becoming EUR? There's a thing we should do on the backend as well. [22:02:43] ok cool [22:02:46] that's what i thought [22:03:10] ah thanks awight, I wasn't sure if you needed to do anything [22:03:18] ejegg when you get a chance, there are some patches in dash that are ready for review [22:03:36] pizzzacat: ok, cool, I'll take a look shortly! [22:03:49] np! [22:03:55] (PS1) Awight: Deploy once Lithuania is on the Euro. [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179557 [22:05:26] thanks awight :) [22:05:43] * awight does some more donuts on the rainy asphalt [22:06:13] want to get some cake? [22:07:29] (CR) Ssmith: [C: -1] "I know this says WIP, but just a reminder that we'd also like a country grouping. We will also want to be able to specify the choices unde" (1 comment) [wikimedia/fundraising/dash] - https://gerrit.wikimedia.org/r/179340 (owner: Ejegg) [22:11:25] (CR) Ejegg: "How about large_donation and wmf_eoy_receipt? Guessing we don't care about oauth!" [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179373 (owner: Awight) [22:21:44] Hrm, nice surprise! In IE9, if javascript dies... _only_ if you run it with debug=true! [22:21:57] I'm so happy I learned about this [22:22:32] it is a dark road you tread, AndyRussG [22:23:11] I've mostly only seen it from afar [22:23:22] That way lies madness! [22:23:42] I used to use GWT for Web programming, which nicely abstracted a way a lot of these... issuz [22:26:30] Yeah something in javascript is making the CN dev display:none [22:28:48] This is almost definitely the same "banner flashes" issue, I've seen the banner flash several times... [22:30:01] hey K4-713... [22:30:02] https://wikimedia.mingle.thoughtworks.com/projects/online_fundraiser/cards/2104? [22:30:04] sometime..? [22:31:06] atgo: Agh, yes. [22:32:01] (PS2) Awight: Enable new modules [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179373 [22:32:23] (CR) Awight: "Good catch. wmf_eoy_receipt is not functional yet." [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179373 (owner: Awight) [22:34:17] (CR) Ejegg: [C: 2] Enable new modules [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179373 (owner: Awight) [22:34:28] (CR) Ejegg: [V: 2] Enable new modules [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179373 (owner: Awight) [22:34:39] (PS3) Ejegg: add a .gitreview [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179366 (owner: Awight) [22:34:49] (CR) Ejegg: [C: 2 V: 2] add a .gitreview [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179366 (owner: Awight) [22:34:59] (PS2) Ssmith: Prevent double clicks on gc cc forms [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179354 [22:35:30] (PS2) Ejegg: manage vendor subdir as a submodule [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179374 (owner: Awight) [22:35:39] (CR) Ejegg: [C: 2 V: 2] manage vendor subdir as a submodule [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179374 (owner: Awight) [22:35:55] K4-713: i'll stop bugging you, but do you think it'd be possible to look at the gc iframe changes after the meeting you're in? [22:36:09] if not, i'll find a time for monday [22:36:23] atgo: This ends at 4pm... [22:36:34] yes... we leave at 430, right? [22:36:34] Maybe? [22:36:42] email said 4:15. [22:36:52] oh ok. i'll put something on the cal to review on monday [22:36:55] Hmmm... IE9 also doesn't work on our simple banners in http://en.wikipedia.beta.wmflabs.org, so it's nothing in the bannes themselves... [22:38:21] (PS2) Ejegg: include vendor content [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179378 (owner: Awight) [22:38:22] atgo: It shouldn't take me too long to set that up. [22:38:30] (CR) Ejegg: [C: 2 V: 2] include vendor content [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179378 (owner: Awight) [22:39:41] (PS2) Ejegg: Fredge and Donations dbs are in Drupal schema for now [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179534 (owner: Awight) [22:39:49] (CR) Ejegg: [C: 2 V: 2] Fredge and Donations dbs are in Drupal schema for now [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179534 (owner: Awight) [22:40:39] (PS2) Ejegg: update vendor submodule [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179535 (owner: Awight) [22:40:47] (CR) Ejegg: [C: 2 V: 2] update vendor submodule [wikimedia/fundraising/civicrm-buildkit] - https://gerrit.wikimedia.org/r/179535 (owner: Awight) [22:41:21] O_o ejegg you almost killed the whole litter [22:44:58] We should document CN caching and robots.txt and... stuff... [22:45:14] exciting to be this close to CI for civi! [22:45:19] I guess I should [22:45:22] ejegg: for real [22:45:31] AndyRussG: sorry to hear you volunteer :p [22:45:39] re CI: Ah sounds great guys way to go [22:46:01] You may do some to if you like :) [22:46:27] ...I was just recalling your "no backsies" statement, is all.. [22:48:32] maybe this IE bug is like something funny in a skin or some other JS code outside CN, doing something funny with the CN div that only happens on IE 9 [23:08:58] (PS2) Awight: QUnit: provide fixtures via a RL module [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/178974 (owner: AndyRussG) [23:13:41] so ccogdill... on your recurring thing. you're seeing the same issue with paypal and amazon as with credit card? [23:15:12] i see that now on the card. how weird. [23:15:28] yeah it’s everything [23:16:09] (CR) Awight: [C: 2] "Great! Two small things to fix up later..." (2 comments) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/178974 (owner: AndyRussG) [23:18:30] awight: thanks 4 the review and comments! :) [23:20:42] (PS4) Awight: Test that two GC API requests send different info [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178713 (owner: Ejegg) [23:20:48] (CR) Awight: Test that two GC API requests send different info (1 comment) [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178713 (owner: Ejegg) [23:20:56] (CR) Awight: [C: 2] Test that two GC API requests send different info [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178713 (owner: Ejegg) [23:21:09] AndyRussG: is there anything else PCR? [23:21:24] PRC? [23:21:41] * awight starts hearing the internationale... [23:21:52] you know... polymerase chain reaction... [23:21:53] PCR? [23:21:59] pending code review :) [23:22:18] awight: not yet I think ... [23:22:57] (PS2) Awight: Clean up a little logspam [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178661 (owner: Ejegg) [23:23:01] (CR) Awight: [C: 2] Clean up a little logspam [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178661 (owner: Ejegg) [23:23:24] awight: that exception thing was especially hacky. really wanted to simulate one request waiting for curl while the other ran [23:23:48] ejegg: I think you were right though, that from a testing perspective, it was the same. [23:24:05] Actually, you're right [23:24:12] for the bug we found, this will catch regressions [23:24:27] but for the bugs we didn't find :)... we do want two threads [23:24:40] yah [23:25:05] (PS3) Awight: Prevent double clicks on gc cc forms [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179354 (owner: Ssmith) [23:25:48] (Merged) jenkins-bot: QUnit: provide fixtures via a RL module [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/178974 (owner: AndyRussG) [23:27:59] ejegg: just one more patch :) https://gerrit.wikimedia.org/r/#/c/179384/ [23:28:33] So are you getting that to work with multiple dbs now? [23:28:35] ejegg: there's a known bug and some comments from totten at the Github pull request [23:28:37] (Merged) jenkins-bot: Test that two GC API requests send different info [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178713 (owner: Ejegg) [23:28:38] nah [23:28:39] (CR) jenkins-bot: [V: -1] Clean up a little logspam [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178661 (owner: Ejegg) [23:28:50] ejegg: but it's good enuf for the rough pass... [23:28:52] IMO [23:28:55] OK, cool [23:29:40] it won't catch certain bugs, caused by not switching databases. [23:30:31] still a great starting point [23:31:55] there's gonna be a lot of cleanup... [23:32:20] ejegg: hey, this looks like a typo. the docstrings are identical for testNoDupeOrderId() and testNoClaimRetryOnBoringCodes() [23:32:31] Oops, yep! [23:32:39] Will fix that with the rebase [23:32:51] (PS3) Awight: Clean up a little logspam [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178661 (owner: Ejegg) [23:32:59] (CR) Awight: "PS3: rebase" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/178661 (owner: Ejegg) [23:33:18] sorry--taking hands off the wheel now. [23:33:35] heh, thanks! [23:33:50] back in a bit... [23:37:30] haha awight is the title of this some sort of subliminal message? https://wikimedia.mingle.thoughtworks.com/projects/online_fundraiser/cards/2103 [23:37:46] atgo: I thought it was too awesome to correct :) [23:37:58] haha well i'm glad i saw it, too : [23:38:07] i've been giggling to myself over here [23:38:19] it would probably damage an android to read that sentence [23:38:31] what do i do!? do i diagnose it or ignore it?! [23:38:35] aaaack [23:38:39] * awight emits smoke [23:39:17] atgo: ok, I'm ready for hints about which card to mangle next [23:39:22] haha [23:39:30] mangle cards [23:40:11] phabricator doesn't sound enough like "mangle" for our taste [23:40:30] (PS4) Awight: Prevent double clicks on gc cc forms [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179354 (owner: Ssmith) [23:40:51] it's already a synonym for liar... [23:41:19] haha [23:42:06] all the things i want for [not christmas] are doc reviews [23:42:14] atgo: I noticed #2287 isn't in the sprint? Not urgent then? [23:42:16] man i'm getting boring [23:42:18] atgo: hah ok [23:42:31] they all smell like K4-713 peed on them tho [23:42:34] haha [23:42:37] <_< [23:42:38] >_> [23:42:39] well if you want 2287 go for it [23:42:43] i have a thought.. more process stuff [23:42:46] Ehrm. [23:42:49] O_O [23:42:54] we should talk about at retro/prawning on wednesday [23:43:23] also israel is back on for january, so jessica just asked me about 1680 [23:43:28] either of those would be super [23:44:13] ok fwiw 1680 is gonna be a real "visit GC with a club" situation [23:44:52] oh is that just the iframe part? [23:45:02] yep, the other thing is already deployed AFAICT [23:45:03] because isn't there a part for our form? [23:45:07] oh yeah? that's awesome [23:45:10] there is a bit, but it's done. yeah [23:45:26] yeah it is! [23:45:31] awesome! [23:45:50] (CR) Awight: "Why is this unmergeable?" [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179354 (owner: Ssmith) [23:45:56] (CR) Awight: [C: 2] Prevent double clicks on gc cc forms [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179354 (owner: Ssmith) [23:47:34] (Merged) jenkins-bot: Prevent double clicks on gc cc forms [extensions/DonationInterface] - https://gerrit.wikimedia.org/r/179354 (owner: Ssmith) [23:51:49] (PS1) AndyRussG: QUnit: small RLmodulefixturefixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/179574 [23:55:07] (CR) AndyRussG: "Thanks much!!!" (1 comment) [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/178974 (owner: AndyRussG) [23:57:07] (CR) Awight: [C: 2] QUnit: small RLmodulefixturefixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/179574 (owner: AndyRussG) [23:57:17] * AndyRussG is sequestered by snow elves [23:57:47] (Merged) jenkins-bot: QUnit: small RLmodulefixturefixes [extensions/CentralNotice] - https://gerrit.wikimedia.org/r/179574 (owner: AndyRussG)