=== jtv-afk is now known as jtv === jtv changed the topic of #launchpad-reviews to: On call: jtv || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jtv changed the topic of #launchpad-reviews to: On call: jtv || reviewing: Ursinha || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === StevenK changed the topic of #launchpad-reviews to: On call: jtv || reviewing: Ursinha || queue: [StevenK] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jtv changed the topic of #launchpad-reviews to: On call: jtv || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews [12:41] On call: jtv || reviewing: foxxtrot || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === jtv changed the topic of #launchpad-reviews to: On call: jtv || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === benji___ is now known as benji === leonardr is now known as leonardr-afk === salgado is now known as salgado-lunch === matsubara is now known as matsubara-lunch === gary_poster is now known as gary-lunch === jtv changed the topic of #launchpad-reviews to: On call: - || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === EdwinGrubbs changed the topic of #launchpad-reviews to: On call: Edwin || reviewing: - || queue: [] || RC branches jump the queue || This channel is logged: http://irclogs.ubuntu.com/ || https://code.launchpad.net/launchpad/+activereviews === matsubara-lunch is now known as matsubara === salgado-lunch is now known as salgado [18:13] jtv: are you not reviewing today? [18:13] EdwinGrubbs: yes, but done. [18:13] Well, done accepting new reviews. [18:13] ok, nm. [18:15] bac: according to the kanban board, you're waiting for a review. Do you want me to take it? I have one if your bored? === gary-lunch is now known as gary_poster === EdwinGrubbs is now known as Edwin-lunch [19:52] hi Edwin-lunch [19:53] the kanban is not exactly right. the branch has been reviewed but i'm making the suggested changes. === Edwin-lunch is now known as EdwinGrubbs [21:49] Edwin, can you review https://code.edge.launchpad.net/~leonardr/launchpad/grant-permissions-oauth/+merge/29425 ? [21:49] gary: i'd also like your opinion on the naming of the new access level and the wording of its description [21:49] leonardr: ack [21:50] f'rinstance, manage_permissions might be better than grant_permissions? [21:52] hey, I'm not around [21:52] * leonardr ponders that [21:52] but if I were, I'd be asking whether this is relevant to the gpg/ssh key discussion on the mailing list [21:52] leonardr: sure [21:53] as it is, I'm not. [21:53] jml, let me take a stab at it without knowing the exact details of the current discussion, based on past discussions [21:54] there is a solution to this problem that involves gpg keys and digital signatures. however, that solution is more confusing to the unskilled end-user than the 'open a web browser' solution [21:56] hey look, lifeless and wgrant are talking about it :) [21:56] all right, i'll go over there [21:56] leonardr, I'm going to wish you the best of luck while I fetch myself a digestiv, open a novel and retire for the evening. [22:08] leonardr: From an end user perspective, Grant Permissions is a much better title because it reflects the common action (and the danger); and let's keep the code name matching it, even though "manage" is more complete and accurate. [22:08] User description wordsmithing: WDYT of changing "Not only will the application will be able to access Launchpad on your behalf, it will be able to grant access to your Launchpad account to any other application." to "The application will be able to grant access to your Launchpad account to any other application " [22:08] IOW, get rid of "Not only will the application will be able to access Launchpad on your behalf," [22:09] makes sense [22:13] cool. Otherwise, looks good. Thank you. === salgado is now known as salgado-afk [22:42] leonardr: review sent [22:42] thanks === matsubara is now known as matsubara-afk