19:04 #startmeeting Ubuntu touch rssreader app meeting 19:04 Meeting started Thu Aug 1 19:04:32 2013 UTC. The chair is popey. Information about MeetBot at http://wiki.ubuntu.com/meetingology. 19:04 19:04 Available commands: #accept #accepted #action #agree #agreed #chair #commands #endmeeting #endvote #halp #help #idea #info #link #lurk #meetingname #meetingtopic #nick #progress #rejected #replay #restrictlogs #save #startmeeting #subtopic #topic #unchair #undo #unlurk #vote #voters #votesrequired 19:04 blueprint: https://blueprints.launchpad.net/ubuntu-phone-commons/+spec/initial-rssreader-development 19:04 bugs: https://bugs.launchpad.net/ubuntu-rssreader-app/+bugs?orderby=-id&start=0 19:04 reviews: https://code.launchpad.net/ubuntu-rssreader-app/+activereviews 19:05 burndown: http://status.ubuntu.com/coreapps-13.10/ 19:05 hows it going dragly ? 19:06 * balloons stalks quietly 19:06 Good. Just pushed a branch today that aims to do the organic grid layout. 19:07 Although I just realized that the design specs had changed :-P 19:07 oops 19:07 Are these the current specs? https://docs.google.com/file/d/0BzWzYe656QsATG9hUlhRczhMcmM/edit 19:08 that's the latest I have seen 19:08 mhall119: see ^^ is that right? 19:08 dragly: how does your grid differ from joey's? 19:09 popey: I'm not sure, I've never seen those before 19:09 ok 19:09 According to Joey, those are the current designs. 19:10 It's only a matter of tweaking the current design in any case, so it's no biggie. I'll see what Joey has to say :) 19:12 Other than that, there are not many updates from my end. I've barely been able to keep up with the current development on the mailing lists while on vacation, hehe. 19:14 ok 19:14 balloons: did you have something you wanted to mention? 19:15 popey, sure thing. There's a bug with adding feeds that would be nice to get sorted 19:15 we'd like to push the autopilot tests back into the branch and it's holding on that bug 19:16 https://code.launchpad.net/~carla-sella/ubuntu-rssreader-app/ubuntu-rssreader-test-add-view-feeds/+merge/173316 19:16 https://bugs.launchpad.net/ubuntu-rssreader-app/+bug/1203528 19:16 Yep, I noticed that today too. 19:17 If you search for a feed, by for instance typing "engadget", you'll find that it works. 19:17 I'll assign it to Roman, he implemented the feed management. If I figure out what's wrong, I'll try to fix it. 19:19 ok, thanks 19:20 balloons: Are you Sam or Nicholas by the way? 19:20 * popey edits blueprint 19:20 dragly, ahh ok.. I'll try that and see if it lets us manuever through 19:20 dragly, I'm nicholas :-) 19:21 ty for looking at the bug :-) 19:22 Hi all 19:22 sry I forgot :( 19:22 Hi Roman! No problem ;) 19:22 >> Although I just realized that the design specs had changed 19:22 yep, really sad :( 19:23 Hehe, no big problem. It will just need some tweaking. But I'll wait to see what Joey has come up with, in case he has implemented the exact same thing and his solution works better. 19:23 might be worth pinging joey a mail, just to catch up? 19:24 w8, did you find some bugs in "add feed" ? :) 19:25 mrqtros, indeed.. it's holding up the re-merging of the autopilot tests :-) 19:25 https://bugs.launchpad.net/ubuntu-rssreader-app/+bug/1203528 19:25 popey: Yes, but I think he might just reply to the merge request. I'll send an e-mail if not, though. 19:26 strange, let me test 19:26 ok 19:27 someone can provide me with test feed url? :) 19:27 http://popey.com/blog/feed/ 19:27 :D 19:27 perfect, thx 19:29 * dragly bookmarks popey's blog 19:29 seems that this bug isn't actual 19:30 someone can test same feed url in trunk version? :) 19:30 * popey does 19:31 I see different log messages :) 19:32 hmm, no articles in my feed 19:32 http://paste.ubuntu.com/5937524/ 19:33 * balloons will await results 19:33 Seems to work now. Strange. 19:33 I spent some time with this, but I haven't checked again since I left a comment on the bug report a few days ago 19:34 basically you couldn't get through selecting a topic for the feed, and therefore couldn't add one 19:34 Perhaps you fixed it in your newest branch that you merged today, Roman? 19:34 popey use "update" in context menu :) 19:34 Maybe :) 19:34 Look, you can see different log messages 19:34 http://popey.com/~alan/rss.png 19:34 I saw the bug earlier today, so I think that has to be it. 19:34 for example "Topic Selected" instead of "CLICKED" 19:35 ah okay 19:35 works now I have refreshed 19:35 sweet! 19:35 ohh lookey 19:35 I wonder if the other bugs are fixed too with topics 19:36 "update" - temporary solution :) 19:36 love this! 19:36 ahh I see delete with feed is still marked todo :-) 19:36 Today we talked with Lisette - now I know how it must look and work :) 19:37 Nope, you can delete as far as I know 19:37 blueprint needs updating? 19:38 I'll update my work items later :) 19:38 ok 19:38 mrqtros: On a different topic. I was thinking about doing some code-cleaning at some point and was wondering if you agree with the following: 19:39 1) Variable names should be camelCase not under_score 19:39 1) My variables (of course) in camel case :) 19:39 2) Using global id's to refer to objects should be avoided. Parent objects should listen to signals instead. 19:39 I have advices too :) 19:40 Agree again :) 19:40 Ok, good :) Just wanted to make sure I don't do any changes you disagree with. 19:41 Joey sometimes uses under_score style 19:41 Thing which really looks terrible is signal changed handler 19:41 Ah, I'll send him an e-mail and ask about his opinion then. 19:41 onRss_readChanged or something like that :) 19:42 yeah, onSomething_with_underscoreChanged :-p 19:42 I even forgot what such syntax means, with my QML experience :D 19:42 Ok, we've overrun a bit.. 19:43 My advice - we can use console.assert(cond, msg) to print some thing, which already tested well 19:43 sry, 1 minute :) 19:43 k 19:43 such as database operation (for ex. SELECTED 5 articles and so on) 19:43 Didn't know about that. Sounds reasonable. 19:44 Yep, console in QML really tuned! 19:44 console object I mean :) 19:44 Ok, that's all ? :) 19:45 I have little sin - I forgot to patch debian file :( my new catalog "listview" must be included in config files 19:46 so if you have 10 minutes, Svenn-Arne, can you approve this very little fix? :) 19:48 ok. lets wrap, thanks 19:48 #endmeeting