[00:02] kewl single inspector stuff is done with a config factory [12:41] hey luca. when you have a moment, I'd love to have a hangout with you on the scrolling behavior. it sounds like you have something in mind that I will like a lot, but I just don't understand it yet. [12:43] gary_poster: sure, I'll hopefully be free in a bit, trying to get the service blocks done [12:43] cool thanks luca [12:59] arosales, we still need to actually release the GUI so the contest's export instructions work. Please let me know ASAP when you all have verified that it works as you need, so we can start the qa for making a release [13:43] gary_poster: it looks like the updated staging isn't updating the css. I'm not getting my changes for the related charms stuff on there atm but it shows up in trunk and works locally. [13:44] rick_h, hm, ok, I'll dig for a sec [13:48] rick_h, poking at things and not sure what to look at to see if I did anything. :-) what do I check? [13:50] gary_poster: so a manual force would be to rm build-shared/juju-ui/templates.js && make build-shared/juju-ui/templates.js [13:50] gary_poster: did you want to discuss scroll bars? [13:50] rick_h, but what do I need to look on uistage at verify that the update happened? [13:51] luca, on call for the next 39 min. Will you be available then? [13:51] gary_poster: sure [13:51] thanks [13:52] gary_poster: so the trouble. is that the juju-gui.css is out of date. I guess we can verify it's timestamp vs the lib/views/browser/*.less [13:54] gary_poster: there should be a badge_approved_17.png in that combined css file coming from the lib/views/browser/charm-token.less file [13:55] for a quick grep-able bit to look for [13:55] rick_h, ran out of time just forced. fixed on uistage? [13:55] morning [13:56] gary_poster: yes, thanks [13:56] cool welcome [13:56] morning hatch [14:10] gary_poster: re your dec email - none are showing up for me in CA - should I do it or is it broken? :) [14:11] hatch, we have to add them ourselves in CA [14:11] ohh ok, I misunderstood [14:15] annnnd done [14:16] you can review a 4K lines diff in about 30 seconds! great opportunity, today only, don't miss it! https://codereview.appspot.com/10746043 [14:17] teknico: can we really trust you? :) [14:17] maybe you put h4x0r code in there [14:17] *shifty eyes* [14:17] hatch: thaaat's the question, isn't it? ;-) [14:18] hatch: you have the tools to catch that, but it will turn the review into a 90 sec one ;-) [14:19] haha [14:24] hatch: thanks [14:40] gary_poster, I have tested a gui only / gui-wordpress-mysql / gui-wordpress-mysql-mongo-ceph [14:40] on aws [14:41] the yamls look correct, but I haven't yet been able to re-import with deployer just yet [14:41] gary_poster, I'll check on HP cloud this morning, and check the status of the deployer with hazmat and then sync up with your on our 1x1. [14:42] orangesquad, I want to have another 15 minute chat in 20 minutes. We can reuse our standup hangout to discuss the big picture for July and maybe August [14:42] arosales, ok cool. ideally I'd wait till we have that confirmation before we release, but I may start optimistically before then so we have enough qa time before eod. [14:43] luca, guichat? [14:43] hopefully fast :-) [14:43] gary_poster, agreed, from my testing it is looking good [14:43] great thanks arosales [14:43] gary_poster: sure, let me grab a room [14:43] thanks for delivering that in a release no less :-) [14:43] cool thanks [14:44] arosales glad to help! thanks for promoting gui [14:44] literally its our pleasure [14:44] :-) [14:48] gary_poster: could you do me a favour and check out my branch to see if you can actually get the error you brought up? [14:48] I thought it wouldn't work either [14:48] but I can't make it break [14:48] hatch on call will try [14:48] later [14:49] thanks - I just want to be sure I'm not missing something obvious [14:50] jujugui call in 10, kanban now [14:51] hazmat, on that note is there a branch I can test the deployer import on charm store charms? [14:52] arosales, ~hazmat/juju-deployer/python-env/ [14:52] hazmat, thanks. [14:53] jujugui creating relations is broken in trunk [14:54] hatch, works for me.. [14:54] really? [14:54] Yes. [14:54] hmm [14:54] * hatch clears caches [14:55] poo [14:55] looks like it's my branch and bad cache [14:57] darn and my branch doesn't appear to have any reason that would break [14:59] jujugui call now [14:59] ish [15:02] rick_h, jcsackett, adeuring, https://plus.google.com/hangouts/_/cf491220dfb7736cee2876baf29110eeb52f5997 [15:23] So. [15:23] Good thing I can hear everyone. [15:23] closer to the new style, gary_poster, just minus some styling and checkboxes. i.e. interaction. [15:45] orangesquad: heads up that they're working on the power around here after the storms last night. Things are flickering and lost it twice now. I might come/go a bit [15:46] rick_h: ack. [15:52] rick_h: above ground power? [15:53] hatch: well when I got back from the vet all the lights were out with trucks around the poles and power's gone out and hit the UPS twice so far. [15:53] hatch: so yea...I guess it's above ground [15:53] or it's the block's below ground, no idea. [15:54] ahh - around here the power is under ground in residential areas except where there is the transformers [15:54] usually it's those things that go in storms here [15:54] hah [15:54] ka------boooooooom [15:54] hatch: yea, I mean they're clearly working on local power. We kept it through the storm, but a block away didn't. So I'm not sure what's up. [15:56] that video looked like it was a good storm too :) [15:57] yea, I was surprised/happy we kept power tbh [16:01] rick_h: got a sec to hat on hangout? [16:01] s/hat/chat/ [16:02] jcsackett: sure thing [16:07] gary_poster: any chance we can get the templates rebuilt/recompiled on staging? they have not, and so despite the flag being gone the share button will not render. [16:07] which causes other fun problems for the sharing widget, as you might imagine. :-P [16:07] jcsackett, tell me what command to run and I will do so. big extra points for figuring out what needs to be fixed in Makefile. I'd look but other priorities atm [16:07] jcsackett: I'd ping bac but I don't think he's around today. I wonder if someone else has access to the staging setup? [16:08] gary_poster: that same command you ran for me needs running. What command is the staging server running on a new version update? [16:08] gary_poster: rm build-shared/juju-ui/templates.js && make build-shared/juju-ui/templates.js [16:09] rick_h, jcsackett done [16:09] rick_h, make build-prod [16:10] oop another make error....time to switch to grunt /troll [16:10] hatch: well I'm guessing some command wasn't moved to the new location when staging was split up. [16:10] hatch: as it was working well on the old url for a long while [16:10] nope this is clearly a make error [16:10] hatch: shush [16:10] haha [16:11] rick_h: huh, it's not fixed. [16:11] before you get assigned to fix it [16:11] jcsackett: yea, looking [16:11] so my insurance company lets us renew our insurance online.....but we have to go into the branch to set up an account :/ [16:11] convenience gain.... 0 [16:12] gary_poster: any chance the command was on the wrong checkout? [16:12] jcsackett: can you check if the css file has all the rules in your branch that was deployed out? http://uistage.jujucharms.com:8086/juju-ui/assets/juju-gui.css [16:12] jcsackett: I'm guessing something is missing since the share widget has the li dots on it [16:12] rick_h: i can, but there aren't rules to make the share button not render. [16:12] rick_h, http://pastebin.ubuntu.com/5808236/ [16:12] jcsackett: right, but just checking if the file's up to date or not. [16:13] gary_poster: thanks, will do a test here with that and see if there's a build step missing. [16:13] gary_poster: I'm going to switch to the settings page - so just let me know whwnever you have a moment to do that QA, no rush [16:13] thanks rick_h [16:13] hatch if someone else verifies for I am AOK with it [16:13] rick_h: looks like it. [16:14] jcsackett: looks like it's up to date? or out of date? [16:14] that is to say, looks like all the rules are there. [16:14] e.g. so the dots should be there? [16:14] * rick_h is confused [16:15] they get cut off by sizing, not decoration. [16:15] rick_h: and again, no css rules were changed in this branch, so even an out of date css file could have all the rules. [16:16] Makyo: bcsaller either of you guys have a moment for a quick QA? [16:16] https://codereview.appspot.com/10724043/ === frankban_ is now known as frankban [16:21] luca, try again in guichat? :-) [16:21] frankban, i privmsg'd you, in case it didn't ping [16:25] rick_h: what did you do? it's working. [16:26] jcsackett: nothing atm. I don't have access to the staging server. I do see some issues though. Subapp tempaltes aren't part of the makefile's template list. [16:26] * jcsackett wonders if there's a delay between templates rebuilding us getting them, even with cache clearing... [16:26] jcsackett: and not sure if there's a cache on the server side? [16:26] rick_h: there's almost certainly a server side cache, come to think of it. [16:27] rick_h: probably worth filing a bug about the subapp templates in makefile's template list. [16:27] jcsackett: trying to see if the css files are keyed in the makefile as well since ours are in a subdir [16:27] jcsackett: yea, trying to finish tracking down bits and maybe I can get my find-fu on to fix it. /me is spoiled by zsh **/* [16:28] I never use find any more [16:29] jcsackett: so there's en ETAG on the js file, but that sohuld have changed/updated when the source was updated. [16:29] why use find when you can F9-c-f? ;-) [16:29] teknico: huh? [16:30] rick_h: no midnight commander, eh? :-) [16:30] teknico: heh, no. ls app/**/*.(handlebars|partial) [16:30] I always forget about midnight commander until I typo `mv` [16:30] teknico: and I don't think out Makefile will be able to run mc [16:31] /out/our [16:31] oh, ok, missed the context [16:32] and man I can't wait until we get to scss and can ditch this treating the css files as a template build step. Just call scss on the files in the makefile themselves. [16:32] and I'm also going to miss you all for ten days: wish me a happy Europython! :-) [16:32] teknico: jealous! [16:33] jcsackett: well now that it's showing make sure to let jcastro see the sexy he's been wanting :) [16:33] yeah! [16:34] I just realized with the oscon reveal that that means I shouldn't take videos of deploying stuff with the gui? [16:34] jcastro: I'd say not :/ [16:34] jcastro: http://uistage.jujucharms.com:8086/precise/ceph-11/ for your enjoyment [16:35] rick_h: which part is different today? [16:35] jcastro: share button [16:36] I don't see it [16:36] jcastro: next to add? [16:36] jcastro: clear cache maybe? [16:36] ah [16:36] not in FFx [16:36] weird [16:37] jcastro: hmm, in FF here. Bet it's a cache issue [16:37] k [16:38] jcastro: ok, so the template list is incomplete and there's no actual make target on the css files so it doesn't rebuild when that's all that changes. [16:38] oops, jcsackett ^ [16:43] gary_poster: oh hey, that Discourse charm gui bug, I'd like to demo it at oscon [16:44] not a hard requirement, but dang it would be awesome [16:45] jcastro, I figured, but we haven't gotten to it. probably no time before we have to make a release for you. will see if I can squeeze it in. [16:45] man, this drag and drop [16:45] I am just sitting here dragging shit into the ui [16:45] for no reason other than just doing it [16:46] lol [16:46] out of curiosity what's the discource gui bug? [16:46] it doesn't deploy from ~marcoceppi/blah blah [16:46] ohh the local charm thing [16:47] no I mean from cs:~marcoceppi blah blah [16:47] it does work fine in the uistage though. ;) [16:47] cheatin' uistage! [16:47] haha yup [16:48] you could demo on sandbox ;) [16:48] "look how fast servers spin up with juju" [16:48] haha [16:48] hah yeah === matsubara is now known as matsubara-lunch [17:06] rick_h, jcsackett, do both of you have time to join my in the stand-up hangout? [17:06] sinzui: rgr [17:08] jcastro, oh, you want discourse bug fixed for *OSCON*. Yes, definitely plan on it. I read it as fixed for Monday. [17:09] oh ok, thanks! [17:14] gary_poster: [I was lunching/relocating.] Sure, I'll get those entered before my EOD. [17:14] benji, cool thanks. we need to countersign/countercountersign before then too. [17:14] sinzui: sure. [17:15] gary_poster: oh, in that case I'll do it right now [17:15] cool [17:17] gary_poster: is the target date 2014-04-01? [17:18] benji yes [17:18] k [17:20] bcsaller: do we know what charm data is provided by default by the delta? [17:22] hatch: I *think* its just the reference id by default in the service data, I'd have to check [17:23] hatch: I think we use that Id to then go to one of the charm store apis, where that is currently the old one [17:24] alright so when we get a delta to create a new service we should go pull the charm data, then show the service in the UI [17:24] then we can get rid of the charm.loaded flag [17:25] gary_poster: https://www.youtube.com/watch?feature=player_detailpage&v=ReJ3RltihME#t=309s [17:25] I'm ready for simultanious key turn. [17:26] benji, lol cool [17:27] I'm both proud and ashamed that it only took me 30 seconds from thinking of that clip to finding it. [17:27] heh [17:27] benji, I don't have it. I think allhands is having issues. I'll send an email to P&C about it [17:27] :/ [17:28] thank you for entering [17:28] yeah [17:28] four people have issues so far [17:28] "Objective Sheet Submitted" [17:45] * hatch holds out hand for donations to buy faster machine to run lbox_check tests [17:46] rick_h: I hear you just got a new machine that you aren't using :P [17:49] hatch: hah, I use it to run my lbox_check :P [17:49] lol [17:49] maybe I can push my changes to your box [17:49] lol [17:50] the rick_h lbox service...wonder if I could turn that into a startup [17:50] lol [17:51] I have to admit I've heard worse startup ideas [17:57] bcsaller: mind taking a peek at the new code [17:57] the charm.loaded needs to stay - at least for a follow-up to fix the charm system later [18:04] jujugui can I get a couple of sanity checks on the small makefile change please. QA instructions included. https://codereview.appspot.com/10756043 [18:04] * benji looks. [18:21] gary_poster: if you get a moment mind taking a look at https://codereview.appspot.com/10724043/ for a followup LGTM [18:22] LGTM hatch === matsubara-lunch is now known as matsubara [18:23] thanks! [18:29] * rick_h fears his 3 lines broken benji's computer [18:29] broke, bah. come on EOD [18:29] heh [18:29] I got distracted by trying to get my tests to pass. [18:29] benji: ah ok. I feared things were going bad. [18:30] rick_h: nope, but the review is done with QA; I had a suggestion on the regex. [18:31] benji: awesome. thanks for the suggestion. [18:31] my pleasure [18:33] gary_poster: landing the makefile update. Hopefully this fixes the issues we've seen today. Let me know if anyone runs across any others like it. /cc jcsackett [18:33] yay thanks rick_h [18:40] bcsaller, or hatch, I'm having a hard time binding an event in a viewlet. Either of you have time in ~5min for a quick chat? [18:41] sure [18:41] Makyo: in chat now [18:50] Makyo: in guichat [18:50] come on by [18:51] hatch, just chatted with bcsaller, sorry. Thanks though. [18:51] oh heh alright np [18:52] Was defining the event on 'events', needed to be 'viewletEvents' [19:09] ahh [19:09] * hatch remembers now why he doesn't do CSS [19:20] sinzui, hi. one more topic. MS asked that uistage.ubuntu.com become comingsoon.jujucharms.com when OSCON starts. May I ask you to add that to your IS coordination list? [19:20] sinzui, ideally uistage would redirect, but if they simply point to same machine that is OK [19:20] I suppose we could handle the redirect ourselves if they point to the same machine [19:21] I think I understand [19:22] gary_poster, anyone going to uistage.ubuntu.com -> comingsoon.jujucharms.com...and the reveal one or both domains become jujucharms.com? [19:23] do we have a card/ticket for removing all of the old unused css after the switchover? [19:35] sinzui, anyone going to jujucharms.com goes to stable production GUI. anyone going to comingsoon.jujucharms.com sees equivalent of current uistage.jujucharms.com:8086. (this one is different from what I said before) finally, anyone going to uistage.jujucharms.com is redirected to jujucharms.com (the stable version) [19:35] sinzui, we could have comingsoon turned on now [19:36] we could also control the uistage redirect if desired but probably better if dns points to production gui...we can implement a redirect later so that uistage actually changes the url [19:38] hatch no. I'll make one. [19:39] gary_poster, so as I am asking for the current stable charm and the stable code to be deployed to what will be jujucharmscom. If webops and I see no issues, we should do the DNS change now? [19:40] sinzui, +1 on comingsoon DNS change now. -1 on the uistage DNS change now, but we can do that at or after OSCON [19:40] okay. Thank you for clarifying. [19:40] thank you! [19:42] hey benji, I wanna make a release. I want your branch in it. what's the timeline? [19:42] gary_poster: thanks [19:42] thank you for mentioning it hatch [19:42] gary_poster: I just submitted it for review; I'm looking over it before soliciting reviews [19:42] s/just submitted/am submitting/ [19:42] ok [19:43] cool [19:43] let's try to speed it through if we see no probs [19:43] sounds good [19:44] Makyo, did frankban hand off to you for his constraints work? [19:44] gary_poster, yes. Still working on getting modifyUnits plugged in, though. I have events properly firing now, at least. I have his code already branched, though. [19:44] orangesquad, gary_poster. The sun was just swallowed by a storm and I cannot hear the music over the thunder. I might loose power/net if this new house problems. [19:45] sinzui, ack. [19:45] cool Makyo thanks [19:49] jujugui: I need two quick reviews so this branch can go into the pending release: https://codereview.appspot.com/10763043 [19:49] benji, on it. [19:49] thanks [19:49] benji: i'll take one [19:49] thanks hatch [19:49] needs QA? [19:52] gary_poster: http://www.sublimetext.com/blog/articles/sublime-text-3-public-beta [19:53] hatch QA yes [19:53] hatch sublime, already installed but haven't done much with it yet [19:55] benji: can I request that you add this functionality to the new deploy method in trunk? [19:55] hatch: you can request anything your heart desires [19:55] well...I am asking becuase if I ask for that I won't lgtm it [19:55] :) [19:56] so depends if you want the lgtm or not haha [19:56] or follow-up [19:56] I think gary_poster will want it in a follow-up. [19:56] * gary_poster vites for follow-up :-) [19:56] votes [19:56] alrighty then :) [19:56] benji, LGTM. [19:57] Closed Reitveld, will LGTM through there in a sec. [20:00] Oh btw Makyo did you look at those svgs from luca? [20:00] gary_poster, Oh! Yeah! They should be fine, I think. We'll have to re-jigger some of the margins and such, but that should be relatively easy, I think. It helps that they don't have drop-shadows. [20:00] Sorry, will email. [20:01] cool Makyo. May I make a card and assign you? [20:01] lol re-jigger [20:01] Yep! [20:01] cool Makyo thanks [20:01] what show is that from [20:01] just a word for me [20:01] It's kind of a south/south-central US idiom. Friend from CA laughed at it last time I used it. [20:03] etymology http://www.etymonline.com/index.php?term=jigger [20:03] http://www.etymonline.com/index.php?term=jigger that is [20:03] languages! [20:04] I'm gona speak in clicks from now on [20:04] _The Gods Must Be Crazy_ [20:04] don't remember the movie but remember liking it [20:05] (he spoke in clicks) [20:05] oh haha [20:06] I was wondering where that was going [20:06] :-) [20:06] bushman discovers coca cola bottle; hilarity ensues [20:06] the story of my life [20:06] lol [20:09] benji: fyi there is a conflict with trunk [20:09] in the less file [20:09] * benji looks [20:10] hatch: fixed (pushing merged branch now) [20:10] yeah it's a super small thing [20:14] benji: lgtm'd I included the issues caused by the missing deploy method in the reply [20:14] benji, I need to be out of here in 20 minutes, fwiw. I can do the release later this evening though. hatch, are those qa issues? [20:14] * benji reads. [20:14] gary_poster: they are - but can only be fixed by the follow-up branch [20:15] :-( [20:15] unless we delay this branch to add those features [20:15] that feature* [20:15] ack [20:17] hatch: I haven't QAed the branch after merging with trunk, are you saying that there is a new deploy method that I should be calling instead? [20:17] benji: under the flag yes [20:17] ah [20:18] guichat about it? [20:18] hatch, this breaks the ghost inspector, but is fine outside of the feature flag? or, it doesn't break anything under the feature flag but doesn't work either? [20:18] those qa issues were under the feature flag [20:18] sorry I wasn't clear there [20:19] gotcha [20:19] and it doesn't break anything pre-existing [20:19] but we need those changes in order to work with the inspector, right? [20:19] so fine to make a release with this [20:19] work with the inspector from deploying by dragging [20:19] yes a release is fine [20:19] right [20:19] cool [20:19] thanks [20:19] deploy by click still works with the flag [20:20] cool [20:20] benji, you good on this? You can make a card and ask someone else to do it if you like :-) [20:20] I'm cool doing a follow-up branch. [20:21] cool thanks benji. [20:21] * gary_poster brb [20:21] I'll also address the review suggstions there, since we are in a rush. [20:21] if that ^^^ is ok with hatch [20:21] benji: yep for sure - my comments were super trivial [20:22] cool [20:22] * benji lands. [20:22] before the release is rolled someone should probably do a qa on it :) [20:23] hatch, was going to do that, but will be packing for trip tonight, so if you Makyo or bcsaller were willing to do the pre-release qa that would be much appreciated [20:23] if not I will do it [20:24] if you did the release that would be cool too! :-) the docs are pretty good I think [20:24] ok don't worry about it, one of us will be able to [20:24] I've never done it :) [20:24] not sure I want to go alone first time on a friday night haa [20:24] ok thank you. shoot me a mail telling me how far it got [20:24] yeah friday is not the best :-/ [20:25] ugh the resize is broken on the internal pages again :-/ [20:25] fixed at least once before [20:25] if not twice [20:25] forgot what the deal was [20:28] darn. I was going to talk to jeff and rick about reusing charm widget [20:28] hatch, guichat for 5? [20:28] if busy no problem [20:29] yup [20:29] thx [20:35] * benji waits for lbox sumbit to finish [20:40] gary_poster: whats up? [20:40] rick, hey. hatch will share evil plan. must run. :-) thanks [20:40] k [20:40] * gary_poster runs [20:40] rick_h: guichat plz [20:41] hatch: can you invite me? on the tablet [20:41] sure one sec [20:44] * benji suspects lbox has frozen. [21:11] branch landed, follow-up branch began, bug filed: time for EOW [21:12] Have a good weekend all. [21:36] benji: you broke the build lol [21:42] [21:42] hatch: I'm looking at the test failure [21:42] benji: oh ok I just started on it too [21:42] I wonder how that happened. [21:42] heh thought you went already [21:42] it only fails in FF [21:42] (repeated locally) [21:43] hatch: maybe I should leave you to it. Do you prefer one over the other? [21:44] I don't really know what the purpose of these asserts are [21:44] why does it 'need' to be 0 [21:53] benji: I think I have fixed it [21:53] cool! [21:54] I am more and more of the opinion that an assert without an explanitory comment is a bug. [21:54] yeah sometimes it's just not very clear as to why the assert is necessary [21:56] ok yup fixed [21:56] proposing [21:57] benji: if you wouldn't mind sticking around for a bit to LGTM this for me so I can at least have a single reviewer :D [21:58] hatch: sure [21:58] thanks I'll ping when lbox completes [22:04] jujugui could I get some reviews plz https://codereview.appspot.com/10769043 [22:04] hatch: looking [22:04] thanks [22:04] hatch: oh, that's much better than what I had [22:05] hatch: review done [22:05] haha well it's the same idea, just removed the cruft :) [22:05] thanks, landing [22:06] lol nice review [22:07] ok once that lands I'll QA and try and not screw up the release ;) [22:09] the release docs are quite good [22:09] yeah they look like it [22:09] ok, I really have to go now; bye! [22:09] cya :D [22:23] hopefully its just me but are other folks seeing the uistage load [22:23] http://uistage.jujucharms.com:8080/ [22:23] canvas and charm store nav is not coming up for me. [22:24] checking [22:24] no store, canvas though [22:24] but the store isn't at that url [22:24] did you want this one? http://uistage.jujucharms.com:8086/ [22:25] ah yes, that works better [22:25] * arosales was just looking for a working version. [22:25] :) the 8086 is the sekrit version ;) [22:37] hatch, thanks [22:37] pwd [22:37] no problem [22:59] jujugui - is the DD supposed to work on rapi? I don't get the (+) on the cursor and it doesn't drop [22:59] DD? [23:00] drag and drop [23:00] Oh,. [23:00] I'm trying to do the release but I can't release it if that's supposed to work heh [23:01] Thats very odd as thats just a UI specific way to call deploy, no? [23:01] that's what I thought [23:01] I'm investigating now [23:02] makedragstart handler is never called [23:05] hmm [23:05] odd [23:05] dragstart is fired but it's callback isn't called [23:05] that don't make no sense mang [23:05] oh [23:06] nope that's not it [23:06] heh [23:12] Where does createServiceInspector live? [23:12] I can't grep quickly because I have stuff in vim copy buffer. [23:12] yup 'drop' never fires [23:13] Makyo: environment.js [23:13] views/ [23:13] Wow, that sucks. [23:13] Most of today's work is moot. [23:13] uh oh, why? [23:14] Just surrounding event stuff, which got weird when I merged with trunk. [23:14] hey [23:14] I'l ljust GUESS [23:14] oh hey gary [23:14] glad you're still here :) [23:15] so you said that dnd doesn't work with improv? [23:15] noope, the drop event doesn't fire [23:15] that's as far as I've gotten so far [23:15] wow that's pretty weird [23:15] yeah....I'm pretty blown away hah [23:16] if I change sandbox to true, I get a drop target, if it's false, I dont' [23:16] trying to dupe [23:16] I can dupe. [23:16] ok [23:16] it's a D3 issue....it's makyo's fault [23:16] that sucks :-/ [23:16] lol [23:16] * hatch runs away [23:17] haha [23:17] I doubt that [23:17] Nah, not cool. [23:17] yeah actually it's in the YUI section [23:17] lol [23:17] OHHH [23:17] I KNOW!!!! [23:17] I KNOW!!!!!!! [23:18] Do tell. [23:18] yeah I was kinda expecting a follow-up [23:18] :-) [23:18] and duped btw [23:18] lol [23:18] it drags very nicely fwiw [23:18] well my guess is that we are missing a yui module [23:19] oh wait [23:19] not so much wit hthe drop though [23:19] nm [23:19] nope the drop whitelist is there [23:19] damnnnnn [23:19] I was so excited [23:19] and now so let down [23:19] :'( [23:20] well darn [23:21] it looks so nice in the sandbox... [23:22] my chromium and chrome are broken for some reason :-( [23:22] they start flashing and wiggling [23:22] when I load a page with stuff going on [23:23] maybe it's checked out for the week [23:23] :) [23:23] heh, been that way for the whole week [23:25] benji is going to be very.....irritated [23:29] ahh getting closer [23:30] ok I lied [23:30] 'dragover' fires [23:30] so I suppose that's a little closer [23:30] I have a suspicion. trying to verify [23:31] in show_environment [23:31] in app.js [23:31] useDragDropImport: true, #this.get('sandbox'), [23:31] that is, I bet that something is configured in the environment that ben did [23:31] which benji needs [23:31] and assumed he already had [23:33] that would stop it [23:35] yes, hatch, bcsaller, that is the trick [23:35] haha damn [23:35] can that be on all the time? [23:35] so just never tested without [23:35] yup [23:35] it has to be now [23:35] well [23:35] thats the old version of a feature flag [23:35] :) [23:35] right on [23:35] but now its a real feature, all grown up [23:36] heh [23:36] good work gary :)\ [23:36] thanks :-) [23:36] so.....who's gona fix? ;) [23:37] I can do it in a bit then do the release pending CI [23:37] well and any other obscure issues haha [23:37] I have to take off in a few minutes for something or I'd offer. [23:38] alright I have nothing to do tonight so I'll git-r-dun! [23:38] Man, this reminds me of my last job in some pretty awful ways :P Can I suggest no more Friday afternoon releases? [23:38] ha, have a good weekend [23:38] well, bcsaller, hatch, thank you. mm [23:38] Makyo: bring it up on Friday [23:38] lol [23:38] ouch [23:38] Makyo, yeah, this was a extra special request for arosales [23:39] gary_poster, Ah, alright! I'm all for getting things out the door, just maybe not Fridays as standard practice :) [23:39] Makyo, huge +1 [23:39] would not have done this otherwise [23:39] Old job was Fridays at 11PM, on call until Monday at 5AM. [23:39] heh [23:39] yeah [23:39] They had the QA people sleeping on cots in the bathrooms testing in 6hr shifts. [23:39] It was nuts. [23:39] wow [23:40] ok great so I'll fix that, CI, QA, release 0.6.2 [23:40] yeah, I was already kind of grimacing about how this might go [23:40] if this is the worst we've got... [23:40] we're lucky [23:40] hatch 0.7.0 pls [23:40] we actually have two real features that people will care about! [23:40] :-) [23:41] dnd and deployer format export [23:41] ahhh right ok cool [23:42] hatch other thoughts (bcsaller if you are around feel free to disagree): [23:42] (1) consider making card to rename importexport.js to dragdrop.js? [23:43] (2) consider passing the flag in to the import export module and disable the file part? [23:43] trying to see if it actually is separate... [23:46] 1 - agree, will make a card [23:46] 2 - not really sure why/when we would want to disable any of it [23:48] hatch, well tbh my first concern is that I still don't understand why the import export module needs to be there. it only has two branches to handle a drop [23:48] and neither of them should be used for the charm drop [23:49] hmm [23:49] hatch, and then my second point is that neither of those branches are actually interesting if you are not using a sandbox [23:50] wow, yeah [23:50] you can comment out all of _handleFileDrop [23:50] throwing this out there - it might be the preventeDefault/stopPropogation [23:50] in _ignore on the dragover/enter [23:51] (sorry trying to feed the dogs right now, will test soon) [23:51] ok trying [23:52] hatch, yes that appears to be it [23:53] hmm that's odd [23:53] hatch, so all we need to get this to work is the [23:53] dragenter: '_ignore', [23:53] dragover: '_ignore', [23:53] registration [23:54] I'd prefer to move that out of the importexprt [23:54] into some generic module [23:54] yeah - I would like to know why that's required [23:54] agreed [23:54] hatch, you really cool with doing this? I'll take the bullet if you need me to [23:55] nope that's fine - I don't have anything better to do :) [23:55] heh [23:55] ok hatch [23:55] thank you [23:56] I'll try to check in with you later [23:56] but thank you! [23:56] no problem :) have a good one