Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-06 Thread Sumit Bose
On Sat, May 04, 2013 at 08:24:52PM +0300, Alexander Bokovoy wrote: On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 06:02:27PM +0300, Alexander Bokovoy wrote: On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 08:13:17AM +0300, Alexander Bokovoy wrote: On Fri, 03 May

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-06 Thread Petr Vobornik
On 05/06/2013 10:47 AM, Sumit Bose wrote: On Sat, May 04, 2013 at 08:24:52PM +0300, Alexander Bokovoy wrote: On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 06:02:27PM +0300, Alexander Bokovoy wrote: On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 08:13:17AM

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-06 Thread Sumit Bose
On Mon, May 06, 2013 at 11:05:40AM +0200, Petr Vobornik wrote: On 05/06/2013 10:47 AM, Sumit Bose wrote: On Sat, May 04, 2013 at 08:24:52PM +0300, Alexander Bokovoy wrote: On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 06:02:27PM +0300, Alexander Bokovoy wrote: On Sat, 04 May

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-06 Thread Petr Vobornik
Hello, here is review of Web UI part. ACK for abbra-102, it's a fix of error (typo), but it won't probably have any impact. Because links associated with 'a' elements are different than the resulting ones. NACK for abbra-103: NACK for abbra-pvoborni... (is included in new abbra-103) Are

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-06 Thread Martin Kosek
On 05/06/2013 01:28 PM, Martin Kosek wrote: On 05/04/2013 07:13 AM, Alexander Bokovoy wrote: On Fri, 03 May 2013, Sumit Bose wrote: On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi! Attached are patches to allow resolving SIDs in Web UI in external membership panel for

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-06 Thread Sumit Bose
On Mon, May 06, 2013 at 05:55:35PM +0200, Martin Kosek wrote: On 05/06/2013 01:28 PM, Martin Kosek wrote: On 05/04/2013 07:13 AM, Alexander Bokovoy wrote: On Fri, 03 May 2013, Sumit Bose wrote: On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi! Attached are patches

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-05 Thread Jakub Hrozek
On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi! Attached are patches to allow resolving SIDs in Web UI in external membership panel for groups. Please see more detailed description in the main patch. I haven't rebased it yet on top of Petr's Web UI rework, hopefully

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-04 Thread Alexander Bokovoy
On Sat, 04 May 2013, Alexander Bokovoy wrote: On Fri, 03 May 2013, Sumit Bose wrote: On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi! Attached are patches to allow resolving SIDs in Web UI in external membership panel for groups. Please see more detailed description in

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-04 Thread Sumit Bose
On Sat, May 04, 2013 at 08:13:17AM +0300, Alexander Bokovoy wrote: On Fri, 03 May 2013, Sumit Bose wrote: On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi! Attached are patches to allow resolving SIDs in Web UI in external membership panel for groups. Please see more

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-04 Thread Alexander Bokovoy
On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 08:13:17AM +0300, Alexander Bokovoy wrote: On Fri, 03 May 2013, Sumit Bose wrote: On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi! Attached are patches to allow resolving SIDs in Web UI in external membership

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-04 Thread Sumit Bose
On Sat, May 04, 2013 at 06:02:27PM +0300, Alexander Bokovoy wrote: On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 08:13:17AM +0300, Alexander Bokovoy wrote: On Fri, 03 May 2013, Sumit Bose wrote: On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi!

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-04 Thread Alexander Bokovoy
On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 06:02:27PM +0300, Alexander Bokovoy wrote: On Sat, 04 May 2013, Sumit Bose wrote: On Sat, May 04, 2013 at 08:13:17AM +0300, Alexander Bokovoy wrote: On Fri, 03 May 2013, Sumit Bose wrote: On Fri, May 03, 2013 at 09:46:47PM +0300,

[Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-03 Thread Alexander Bokovoy
Hi! Attached are patches to allow resolving SIDs in Web UI in external membership panel for groups. Please see more detailed description in the main patch. I haven't rebased it yet on top of Petr's Web UI rework, hopefully it should be simple. https://fedorahosted.org/freeipa/ticket/3302

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-03 Thread Sumit Bose
On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi! Attached are patches to allow resolving SIDs in Web UI in external membership panel for groups. Please see more detailed description in the main patch. I haven't rebased it yet on top of Petr's Web UI rework, hopefully

Re: [Freeipa-devel] [PATCH] Resolve SIDs in Web UI

2013-05-03 Thread Alexander Bokovoy
On Fri, 03 May 2013, Sumit Bose wrote: On Fri, May 03, 2013 at 09:46:47PM +0300, Alexander Bokovoy wrote: Hi! Attached are patches to allow resolving SIDs in Web UI in external membership panel for groups. Please see more detailed description in the main patch. I haven't rebased it yet on top