Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-30 Thread Petr Vobornik
All pushed. master: * 15b6ed67056ce918e11f7ea5c2d193534b5ce6b5 webui: improve breadcrumb navigation * 26bd309c96446b9eda26a08e6924d6e1b4c621fc webui: treat value as pkey in link widget * 27196b92c60917d8488dad8721d2087e9fee716c webui: do not show internal facet name to user * 8b0e2ed991e9a1a49

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-28 Thread Endi Sukma Dewata
On 9/26/2014 10:25 AM, Petr Vobornik wrote: On 25.9.2014 19:07, Petr Vobornik wrote: All issues will be done separately as already stated in other sub-thread. I've removed issues which are discussed in the other sub-thread. 2. The tab titles in the ID view details page are quite long, and the

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-26 Thread Petr Vobornik
On 25.9.2014 19:07, Petr Vobornik wrote: All issues will be done separately as already stated in other sub-thread. I've removed issues which are discussed in the other sub-thread. 2. The tab titles in the ID view details page are quite long, and the "User ID overrides" and "Group ID overrides"

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-25 Thread Petr Vobornik
All issues will be done separately as already stated in other sub-thread. I've removed issues which are discussed in the other sub-thread. On 25.9.2014 09:25, Alexander Bokovoy wrote: On Wed, 24 Sep 2014, Endi Sukma Dewata wrote: OK, some comments/questions: 1. For consistency, the "ID view"

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-25 Thread Endi Sukma Dewata
On 9/25/2014 10:23 AM, Petr Vobornik wrote: On 25.9.2014 16:40, Endi Sukma Dewata wrote: 4. If I understand correctly the description field for the User ID Overrides and Group ID Overrides should be optional too because it's also used to optionally override the description attribute of the origi

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-25 Thread Petr Vobornik
Note: I'll send response also to previous mail. On 25.9.2014 16:40, Endi Sukma Dewata wrote: On 9/25/2014 2:25 AM, Alexander Bokovoy wrote: On Wed, 24 Sep 2014, Endi Sukma Dewata wrote: 4. If I understand correctly the description field for the User ID Overrides and Group ID Overrides should b

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-25 Thread Endi Sukma Dewata
On 9/25/2014 2:25 AM, Alexander Bokovoy wrote: On Wed, 24 Sep 2014, Endi Sukma Dewata wrote: 4. If I understand correctly the description field for the User ID Overrides and Group ID Overrides should be optional too because it's also used to optionally override the description attribute of the o

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-25 Thread Alexander Bokovoy
On Wed, 24 Sep 2014, Endi Sukma Dewata wrote: On 9/24/2014 9:43 AM, Petr Vobornik wrote: On 24.9.2014 16:30, Endi Sukma Dewata wrote: On 9/19/2014 7:29 AM, Petr Vobornik wrote: Hello, attached patches implements Web UI part of ID Views. Backend is currently on review as well - thread "[PATCHE

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-24 Thread Endi Sukma Dewata
On 9/24/2014 9:43 AM, Petr Vobornik wrote: On 24.9.2014 16:30, Endi Sukma Dewata wrote: On 9/19/2014 7:29 AM, Petr Vobornik wrote: Hello, attached patches implements Web UI part of ID Views. Backend is currently on review as well - thread "[PATCHES 247-259] ID views - management part". https:

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-24 Thread Petr Vobornik
On 24.9.2014 16:30, Endi Sukma Dewata wrote: On 9/19/2014 7:29 AM, Petr Vobornik wrote: Hello, attached patches implements Web UI part of ID Views. Backend is currently on review as well - thread "[PATCHES 247-259] ID views - management part". https://fedorahosted.org/freeipa/ticket/4535 I ex

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-24 Thread Endi Sukma Dewata
On 9/19/2014 7:29 AM, Petr Vobornik wrote: Hello, attached patches implements Web UI part of ID Views. Backend is currently on review as well - thread "[PATCHES 247-259] ID views - management part". https://fedorahosted.org/freeipa/ticket/4535 I expect that backed can change and that the UI mi

Re: [Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-22 Thread Petr Vobornik
On 19.9.2014 14:29, Petr Vobornik wrote: Hello, attached patches implements Web UI part of ID Views. Backend is currently on review as well - thread "[PATCHES 247-259] ID views - management part". https://fedorahosted.org/freeipa/ticket/4535 I expect that backed can change and that the UI migh

[Freeipa-devel] [PATCH] 749-754 webui: new ID views section

2014-09-19 Thread Petr Vobornik
Hello, attached patches implements Web UI part of ID Views. Backend is currently on review as well - thread "[PATCHES 247-259] ID views - management part". https://fedorahosted.org/freeipa/ticket/4535 I expect that backed can change and that the UI might influence it as well. Therefore no U