[12:58] morning all. [13:00] morning [13:09] rick_h_: want to chat about local inspectors? [13:09] jcsackett: sure thing [13:11] rick_h_: https://plus.google.com/hangouts/_/76cpij86oaqmt8r5e9lg691r88?hl=en [15:01] jujugui: call now, i guess? all, what, three of us? [15:01] jcsackett: ah sorry [15:01] jcsackett: kadams can't make it [15:01] jcsackett: so going to pull the plug [15:01] rick_h_: works for me. :) [16:27] Hey all, back in the saddle here. [16:27] woot [16:28] jcsackett, rick_h_ Give me a moment to spin things up and I'll be ready for some pairing [16:28] kadams54: awesome, heading to the desktop [16:35] kadams54: setup a hangout when you get a sec and a pull request of WIP to pull down and poke at [16:39] kk [16:42] https://plus.google.com/hangouts/_/76cpjv0pjdnrfmdg99cmtm9k5c [16:44] hmm, chrome is unhappy sec [16:44] this party is over [16:45] :-( [16:45] kadams54: https://plus.google.com/hangouts/_/72cpicrq2pbtqk329ri2577nug [19:36] kadams54: yay, pull request passed ci. Is that ready for review or are you cleaning? [20:01] kadams jcsackett out for a bit. Will be around tonight. Safe travels and catch up in vegas [20:07] rick_h_: Cleaning. Well, cleaned. Now I'm fixing the tests that cleaning broke. [20:09] rick_h_: cool beans. see you in vegas! [20:48] rick_h_: Hmm, I was running through QA and I'm seeing problems with il flag [20:48] Charm API error of type: parameter_not_supported [20:53] kadams54: k, what was the request made? [20:53] kadams54: the ajax url that was built? [20:53] Checking… [20:53] kadams54: and can you reproduce it on comingsoon? [20:54] https://manage.jujucharms.com/api/3/search?text%5Btext%5D=apache [20:55] When requesting http://192.168.33.10:8888/:flags:/il/?text=apache [20:55] hmm, yea that is messed up [20:55] the text=text stuff is all wrong [20:55] jcsackett: if you get a sec to peek at ^ [20:55] Not reproducible on comingsoon, but that's not that surprising [20:56] All this work originally started (IIRC) with the card about search info in the URL not being populated into the search box [20:56] kadams54: ok, so it is something from your branch then. I'll see if I can replicate [20:56] Which is what I was testing when I ran into this… [20:57] kadams54: off to pick up the boy from day care. Push up the latest and I can try to reproduce and debug tonight [20:57] Yeah, latest is pushed [20:57] ok, cool thanks [20:58] will peek after his bed time. Wife is away tonight [20:58] Alright [20:58] I need to make up my morning hours, so I'll also be up late. [20:58] kadams54: url for this branch? i have some notion what might be going on, but need to see your code. [20:58] kadams54: k, there should be something in jcsackett's branch involving state and your work that's mixed up generating the poor api url [20:59] https://github.com/kadams54/juju-gui/tree/search-widget [20:59] jcsackett: ^ [21:01] kadams54: so, https://github.com/kadams54/juju-gui/commit/2bab254d773a270de6ec43e6ac39b066c26696d1 [21:02] line 651 in the diff [21:02] Yeah [21:02] Yup [21:03] where you have viewCfg.filters.text = metadata.search, that doesn't account for the updates from my branch--switch that to either metadata.search.text, or see if you can just use state.get('filter) [21:03] sorry, i was syncing with your branch, and never saw that break. [21:03] rick_h_, what's the trick to use the console to update the environment name / type? [21:03] but that should fix your issue. [21:04] kadams54: if you can use state.get('filter') you should be doing view.Cfg.filters = state.get('filter') [21:04] (y) [21:05] kadams54: or rather viewCfg.filters = state.get('filter').getFilterData() -- double check the actual name of that method. [21:05] "(y)" ? that's an IRC shorthand i haven't seen before. :p [21:05] Yeah, that looks right [21:05] I think it's actually Facebook/iOS/Skype shorthand :-) [21:05] oh, like (ninja) [21:06] (y) == thumbs up [21:06] Yeah [21:06] cool. good to know. :) [21:06] (y) [21:06] ;-) [21:19] guihelp: https://github.com/juju/juju-gui/pull/262 ready for review and QA [21:56] arosales: second, I'll have to look it up. [21:56] rick_h_, ok thanks [22:04] arosales: pm'd to you [22:07] rick_h_, thanks