[04:01] <bac> Hi I'm on-call now.  bring out your reviews...
[04:05] <StevenK> Except you set yourself as the branch being reviewed, not who is on-call
[04:15] <bac> StevenK: yeah, well...
[04:15] <bac> more happy StevenK?  :)
[04:29] <StevenK> Heh, slightly. :-)
[04:32] <wgrant> I feel timezone-confused.
[05:06] <bac> why is that wgrant?
[11:40] <bac> hi bigjools
[11:40] <bigjools> hey bac
[11:40] <bac> bigjools: did you verify your def-row db query change on the live db?
[11:41] <bac> it looks good to me...
[11:41] <bigjools> bac: it's already cowboyed, so yes :)
[11:41] <bac> :)
[11:41] <bac> alrighty
[11:41]  * bigjools is doing two CP MPs at the moment :/
[14:59] <jcsackett> anyone available to review a 93 line MP? https://code.edge.launchpad.net/~jcsackett/launchpad/mirrors-tell-where-they-live-652134/+merge/37925
[15:48] <allenap> jcsackett: I'll do it. Can you review one of mine?
[15:49] <jcsackett> allenap: i am not a reviewer yet. :-P
[15:49] <allenap> jcsackett: Drat! You get this one for free then :)
[15:50] <jcsackett> thanks allenap. i'll owe you one in november when i start the reviewing thing. :-)
[16:16] <sinzui> allenap, I can review
[16:17] <allenap> sinzui: Woohoo, thanks :)
[16:17] <allenap> sinzui: You have probably found it already, but if not: https://code.edge.launchpad.net/~allenap/launchpad/sub-search-ui-bug-656823/+merge/37976
[16:33] <sinzui> allenap, I see metal:use-macro="view/macro:page/main_side", but this template does not use the side. Is the intent to add navigation to see other listings such as indirect bug subscriptions or blueprints?
[16:34] <allenap> sinzui: I don't know if that's the intent, but it can be changed back later. I'll switch to main_only.
[16:34] <sinzui> okay
[16:35] <sinzui> r=me. This branch does make the code clearer.
[16:39] <allenap> sinzui: Thanks.