Re: [Freeipa-devel] [PATCH 0046] Fix a typo in the otptoken doc string

2014-05-06 Thread Petr Vobornik
s. https://www.redhat.com/archives/freeipa-devel/2014-May/msg00039.html Nathaniel Pushed to master: 797974b09fdd078c8ad645c217a464b69ce72f66 -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH 0001] Fixed typo in ipa_backup.py

2014-05-06 Thread Petr Vobornik
On 2.5.2014 17:20, Nathaniel McCallum wrote: ACK Nathaniel Pushed to master: 3f3c8eee24f98807ff8a95dd0f6a022b2b3a5bf5 -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 16-17 Attribute box in permission UI is too small

2014-05-06 Thread Petr Vobornik
ch to this widget by the context parameter ($node). It will also allow you to use simpler selector. -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 18 webui otptoken test data added

2014-05-06 Thread Petr Vobornik
On 5.5.2014 16:39, Misnyovszki Adam wrote: On Wed, 30 Apr 2014 13:37:10 +0200 Petr Vobornik wrote: On 29.4.2014 16:30, Misnyovszki Adam wrote: On Fri, 25 Apr 2014 17:16:48 +0200 Misnyovszki Adam wrote: Hi, this patch adds some static test data for the webui otptoken part. Adam Attached

Re: [Freeipa-devel] [PATCH] 588 webui: fix switching between multiple_choice_section choices

2014-05-05 Thread Petr Vobornik
On 5.5.2014 14:40, Misnyovszki Adam wrote: On Fri, 25 Apr 2014 19:24:35 +0200 Petr Vobornik wrote: - required indicators are not present for all sections except the last - validation has wrong color for the same sections There was only one layout for all choices. Layout should not be reused

Re: [Freeipa-devel] [PATCH] 587 webui-ci: adjust id range tests to new validator

2014-05-05 Thread Petr Vobornik
On 5.5.2014 14:05, Misnyovszki Adam wrote: On Fri, 25 Apr 2014 15:02:27 +0200 Petr Vobornik wrote: SSIA LGFM, integration tests for range now runs smoothly. ACK Thanks Adam Pushed to master: 7eff8ad7dc385070b600ddb2b2ba1f39835718b6 -- Petr Vobornik

Re: [Freeipa-devel] [PATCH] 18 webui otptoken test data added

2014-04-30 Thread Petr Vobornik
ation of otptoken_get_records.json and otptoken_find_pkeys.json is enough). In general, it's OK to fake the data if there is some bug which causes errors and we know that it will be fixed. -- Petr Vobornik ___ Freeipa-devel mailing list Fre

[Freeipa-devel] [PATCH] 591 webui: add idnsSecInlineSigning option to DNS zone details facet

2014-04-30 Thread Petr Vobornik
Web UI part of pviktori-543 https://fedorahosted.org/freeipa/ticket/3801 -- Petr Vobornik From 2652953332ec5f5ee6f131a389168f5ee37099a5 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Wed, 30 Apr 2014 12:24:25 +0200 Subject: [PATCH] webui: add idnsSecInlineSigning option to DNS zone details

[Freeipa-devel] [PATCH] 589-590 webui-ci: save screenshot on test failure

2014-04-30 Thread Petr Vobornik
`screenshot_dir: /path/to/dir` to specify target directory otherwise screenshots are saved to current directory -- Petr Vobornik From 179c2dca196d211ef9454dae84e9cc7a0580af71 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Tue, 29 Apr 2014 09:16:24 +0200 Subject: [PATCH] webui-ci: decorate

Re: [Freeipa-devel] [PATCH] webui: regression - enable fields on idrange type change (add)

2014-04-29 Thread Petr Vobornik
On 29.4.2014 16:25, Misnyovszki Adam wrote: On Fri, 25 Apr 2014 15:01:36 +0200 Petr Vobornik wrote: ID range adder dialog was not properly addressed in field binding refactoring. The usage of reset caused some weird loops. https://fedorahosted.org/freeipa/ticket/4326 tests with and

Re: [Freeipa-devel] [PATCH] 14 webui: select all checkbox remains selected after operation

2014-04-29 Thread Petr Vobornik
On 24.4.2014 14:57, Misnyovszki Adam wrote: On Wed, 23 Apr 2014 16:57:35 +0200 Petr Vobornik wrote: On 18.4.2014 10:43, Misnyovszki Adam wrote: Hi, this patch fixes select_all checkbox issue, after any bulk modify or delete operation, the checkbox is deselected. https://fedorahosted.org

[Freeipa-devel] [PATCH] 588 webui: fix switching between multiple_choice_section choices

2014-04-25 Thread Petr Vobornik
hoice. https://fedorahosted.org/freeipa/ticket/4327 -- Petr Vobornik From e76604f82bacf23c51c4bc8821421aaa657ea0de Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Fri, 25 Apr 2014 19:15:52 +0200 Subject: [PATCH] webui: fix switching between multiple_choice_section choices - required indicator

[Freeipa-devel] [PATCH] 587 webui-ci: adjust id range tests to new validator

2014-04-25 Thread Petr Vobornik
SSIA -- Petr Vobornik From 66410a435d641a90da7bc0f525d5e73e3a5c549d Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Thu, 24 Apr 2014 17:24:59 +0200 Subject: [PATCH] webui-ci: adjust id range tests to new validator --- ipatests/test_webui/task_range.py | 33

[Freeipa-devel] [PATCH] webui: regression - enable fields on idrange type change (add)

2014-04-25 Thread Petr Vobornik
ID range adder dialog was not properly addressed in field binding refactoring. The usage of reset caused some weird loops. https://fedorahosted.org/freeipa/ticket/4326 -- Petr Vobornik From 9e9e72dc795156811662b5130ad58084a898974c Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Thu, 24 Apr

Re: [Freeipa-devel] [PATCH] 15 webui doc: typo fixes in guides

2014-04-23 Thread Petr Vobornik
On 18.4.2014 14:41, Misnyovszki Adam wrote: Hi, $SUBJ tells everything. Thanks Adam ACK Pushed to master: 260c5bd10927d3e01202ed1cccbdf4463c98e8a5 -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com

Re: [Freeipa-devel] [PATCH] 14 webui: select all checkbox remains selected after operation

2014-04-23 Thread Petr Vobornik
facet (group/external) (not tested). -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 569-583 New Login Screen

2014-04-15 Thread Petr Vobornik
On 15.4.2014 12:05, Misnyovszki Adam wrote: On Tue, 15 Apr 2014 09:39:54 +0200 Petr Vobornik wrote: On 11.4.2014 14:31, Misnyovszki Adam wrote: On Fri, 28 Mar 2014 14:04:13 +0100 Petr Vobornik wrote: Attached patches replace IPA.unauthorized dialog with new Login Screen. To make it

[Freeipa-devel] [PATCH] 585 webui: fix OTP Token add regression

2014-04-15 Thread Petr Vobornik
Petr Vobornik From d047720e6f00a2579546b7f60716b5846df034c6 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Tue, 15 Apr 2014 09:47:21 +0200 Subject: [PATCH] webui: fix OTP Token add regression OTP Token add failed because of invalid function call. qr_widget doesn't contain `on_value_chang

[Freeipa-devel] Changing RCUE to PatternFly

2014-04-03 Thread Petr Vobornik
fies upgrades). Our own LESS files don't use any new Less markup features so we are still able to compile it with current version of python-lesscpy. -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 565-568 webui: field and widget binding refactoring

2014-04-03 Thread Petr Vobornik
On 2.4.2014 17:57, Misnyovszki Adam wrote: On Thu, 27 Mar 2014 16:07:55 +0100 Petr Vobornik wrote: The last refactoring I did while implementing RCUE login or more precisely support for standalone facets which have forms but are not details facets. [PATCH] webui: field and widget binding

[Freeipa-devel] [PATCH] 584 migration: fix import of wsgiref.util

2014-03-31 Thread Petr Vobornik
https://fedorahosted.org/freeipa/ticket/4293 -- Petr Vobornik From dc4eaf9d622b4eac9fb3d942b23083bfb903be9a Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Tue, 1 Apr 2014 08:45:08 +0200 Subject: [PATCH] migration: fix import of wsgiref.util https://fedorahosted.org/freeipa/ticket/4293

Re: [Freeipa-devel] [PATCH]Extending user plugin with employeenumber field

2014-03-28 Thread Petr Vobornik
: 0f626a9cc870ba0bdcc1322d765687b0e597c26c -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 562-563 webui: move RPC code from IPA module to its own module

2014-03-27 Thread Petr Vobornik
On 27.3.2014 14:38, Misnyovszki Adam wrote: On Wed, 19 Mar 2014 16:02:29 +0100 Petr Vobornik wrote: depends on path #561(make navigation module independent on app module) [PATCH] 562 webui: move RPC code from IPA module to its own module - moves RPC code from ipa.js to it's own m

Re: [Freeipa-devel] [PATCH] 561 webui: make navigation module independent on app module

2014-03-27 Thread Petr Vobornik
On 26.3.2014 15:36, Misnyovszki Adam wrote: On Wed, 19 Mar 2014 16:02:19 +0100 Petr Vobornik wrote: When some module used 'freeipa/navigation' it pulled the entire Web UI because navigation depended on app. This patch splits the app into two modules: app and app_container. App spe

Re: [Freeipa-devel] [PATCH] 560 webui: rename domNode to dom_node

2014-03-26 Thread Petr Vobornik
On 25.3.2014 16:26, Misnyovszki Adam wrote: On Tue, 25 Mar 2014 12:49:24 +0100 Petr Vobornik wrote: On 20.3.2014 16:51, Misnyovszki Adam wrote: On Wed, 19 Mar 2014 16:02:12 +0100 Petr Vobornik wrote: - unites domNode and dom_node usage to dom_node Nack, install/ui/test/details_tests.js

Re: [Freeipa-devel] [PATCH]Extending user plugin with employeenumber field

2014-03-25 Thread Petr Vobornik
On 21.3.2014 11:00, Misnyovszki Adam wrote: On Fri, 21 Mar 2014 10:13:55 +0100 Misnyovszki Adam wrote: On Fri, 21 Feb 2014 16:06:27 +0100 Petr Vobornik wrote: On 21.2.2014 15:45, Adam Misnyovszki wrote: Hi, According to http://tools.ietf.org/html/rfc2798 ipa client and web ui extended

Re: [Freeipa-devel] [PATCH] 560 webui: rename domNode to dom_node

2014-03-25 Thread Petr Vobornik
On 20.3.2014 16:51, Misnyovszki Adam wrote: On Wed, 19 Mar 2014 16:02:12 +0100 Petr Vobornik wrote: - unites domNode and dom_node usage to dom_node Nack, install/ui/test/details_tests.js:236 install/ui/test/details_tests.js:242 only finds element, because context(ie domNode) is undefined

Re: [Freeipa-devel] [PATCH 0038] Fix generation of invalid OTP URIs

2014-03-24 Thread Petr Vobornik
. Since this patch rotted a bit, attaching rebased version. IMO we should not push this patch. Wasn't it superseded by: https://git.fedorahosted.org/cgit/freeipa.git/commit/?id=21ff4f920e4ff7c1e2870024f007f067fc3cf6c8 ? -- Petr Vobornik

[Freeipa-devel] [PATCH] 564 webui-ci: fix test_rebuild_membership_hosts on server without DNS

2014-03-21 Thread Petr Vobornik
Host adder dialog differs on installations with and without DNS. Previous test used values for adding hosts which were suitable only for IPA servers installed with DNS. -- Petr Vobornik From 9ca559347b4f233772b2179da725eb28bddbe4e2 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Fri, 21 Mar

Re: [Freeipa-devel] [PATCH] [DOC] login as another user in the WebUI

2014-03-20 Thread Petr Vobornik
On 20.3.2014 00:03, Gabe Alford wrote: Thanks for the review. Here's an updated patch. Gabe ACK Pushed to master: befe9632c4636a151f5e4ed11a58e7e48ba8c877 On Wed, Mar 19, 2014 at 3:35 AM, Petr Vobornik wrote: On 19.3.2014 02:29, Gabe Alford wrote: All, Just added a couple of

Re: [Freeipa-devel] [PATCH] 559 webui: update license information of used third party code

2014-03-20 Thread Petr Vobornik
On 19.3.2014 16:19, Simo Sorce wrote: On Wed, 2014-03-19 at 16:02 +0100, Petr Vobornik wrote: I have some cleanup and refactoring patches which were created along with RCUE login effort. They are not that much connected so I'm sending them now (in 4 mails) to make the subsequent review sm

Re: [Freeipa-devel] [PATCH] 552-557 Permissions v2 Web UI

2014-03-20 Thread Petr Vobornik
On 19.3.2014 17:03, Misnyovszki Adam wrote: On Wed, 19 Mar 2014 10:52:10 +0100 Petr Viktorin wrote: On 03/18/2014 04:56 PM, Petr Vobornik wrote: On 18.3.2014 15:07, Petr Viktorin wrote: On 03/18/2014 01:09 PM, Petr Vobornik wrote: New revision for patch patch #557 attached. On 17.3.2014

Re: [Freeipa-devel] [PATCH] 550 webui-css: improve radio, checkbox keyboard support and color

2014-03-20 Thread Petr Vobornik
On 19.3.2014 16:32, Misnyovszki Adam wrote: On Fri, 14 Mar 2014 18:39:14 +0100 Petr Vobornik wrote: On 13.3.2014 16:55, Petr Vobornik wrote: On 7.3.2014 15:34, Petr Vobornik wrote: checkboxes and radio buttons: - do not change color on hover when disabled - are focusable and checkable by

[Freeipa-devel] [PATCH] 559 webui: update license information of used third party code

2014-03-19 Thread Petr Vobornik
ng of RCUE styles along with Bootstrap 2.0. - removal of jQuery BBQ plugin - removal of rhino enviroment for selenium which was used in old UI tests -- Petr Vobornik From 0782e0e71641af4df1f59f78a71853f9b09a8606 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Mon, 18 Nov 2013 14:49:37 +0100 Su

[Freeipa-devel] [PATCH] 560 webui: rename domNode to dom_node

2014-03-19 Thread Petr Vobornik
- unites domNode and dom_node usage to dom_node -- Petr Vobornik From da7c558e3e8a3086b57b844062421b71b657adf0 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Fri, 22 Nov 2013 10:22:15 +0100 Subject: [PATCH] webui: rename domNode to dom_node - unites domNode and dom_node usage to dom_node

[Freeipa-devel] [PATCH] 561 webui: make navigation module independent on app module

2014-03-19 Thread Petr Vobornik
ation boot classes. Navigation now depends on app_container. -- Petr Vobornik From 1c7459e8d5d0a3acd416db32b022ba73ad6b7d79 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Wed, 19 Mar 2014 15:29:58 +0100 Subject: [PATCH] webui: make navigation module independent on app module When some module use

Re: [Freeipa-devel] [PATCH] [DOC] login as another user in the WebUI

2014-03-19 Thread Petr Vobornik
t 3) kinit anotheruser 4) open web ui -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 558 webui: remove hover effect from disabled action button

2014-03-17 Thread Petr Vobornik
On 17.3.2014 15:16, Petr Vobornik wrote: https://fedorahosted.org/freeipa/ticket/4256 Petr3 found that the effect is still there when the button is focused. It's caused by another bug - https://fedorahosted.org/freeipa/ticket/4258 Disabled action buttons are focusable New revision o

[Freeipa-devel] [PATCH] 558 webui: remove hover effect from disabled action button

2014-03-17 Thread Petr Vobornik
https://fedorahosted.org/freeipa/ticket/4256 -- Petr Vobornik From 3378be53351fb3521cb4b73b8b76c02703583463 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Mon, 17 Mar 2014 15:14:44 +0100 Subject: [PATCH] webui: remove hover effect from disabled action button https://fedorahosted.org/freeipa

Re: [Freeipa-devel] [PATCH] 550 webui-css: improve radio, checkbox keyboard support and color

2014-03-14 Thread Petr Vobornik
On 13.3.2014 16:55, Petr Vobornik wrote: On 7.3.2014 15:34, Petr Vobornik wrote: checkboxes and radio buttons: - do not change color on hover when disabled - are focusable and checkable by keyboard again. This uses a little trick where the real checkbox is hidden under the artificial

Re: [Freeipa-devel] [PATCH] 548 webui: change ipatokennotbefore and ipatokennotafter types to datetime

2014-03-14 Thread Petr Vobornik
On 25.2.2014 18:10, Petr Vobornik wrote: Depends on tbabej's patches # 137, 140 and pvoborni's 546 and 531-541. https://fedorahosted.org/freeipa/ticket/3369 Attaching rebased patch. -- Petr Vobornik From 5e3b639f5f7edd6d4539951e448e13666dc19795 Mon Sep 17 00:00:00 2001 From: Pet

Re: [Freeipa-devel] [PATCH] BuildRequires: rhino in .spec file

2014-03-14 Thread Petr Vobornik
On 14.3.2014 14:50, Petr Vobornik wrote: On 14.3.2014 14:20, Nalin Dahyabhai wrote: When I try to scratch build master using 'make srpm' and koji, the build log includes multiple errors like this: ../../util/make-ui.sh Error: Could not find or load

Re: [Freeipa-devel] [PATCH] BuildRequires: rhino in .spec file

2014-03-14 Thread Petr Vobornik
7.0-opnjdk no longer pulls rhino. Caused by: http://pkgs.fedoraproject.org/cgit/java-1.7.0-openjdk.git/commit/?id=63ecc0b8d60970f6d1d80f822cebf78441beb56a -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mail

Re: [Freeipa-devel] [PATCH] 550 webui-css: improve radio, checkbox keyboard support and color

2014-03-13 Thread Petr Vobornik
On 7.3.2014 15:34, Petr Vobornik wrote: checkboxes and radio buttons: - do not change color on hover when disabled - are focusable and checkable by keyboard again. This uses a little trick where the real checkbox is hidden under the artificial checkbox. That way it has the same position

[Freeipa-devel] [PATCH] 550 webui-css: improve radio, checkbox keyboard support and color

2014-03-07 Thread Petr Vobornik
with overflow set. https://fedorahosted.org/freeipa/ticket/4217 -- Petr Vobornik From a554df223b33de5c7ede4838cef6e558170761ab Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Thu, 27 Feb 2014 18:21:05 +0100 Subject: [PATCH] webui-css: improve radio,checkbox keyboard support and color checkboxes

Re: [Freeipa-devel] [PATCH] 546 webui: Datetime parsing and formatting

2014-03-06 Thread Petr Vobornik
On 6.3.2014 13:01, Misnyovszki Adam wrote: On Tue, 25 Feb 2014 18:05:28 +0100 Petr Vobornik wrote: prerequisite for patch 547, 548 depends on tbabej's datetime patch this patch implements: - output_formatter in field. It should be used in par with formatter. Formatter serves for datas

Re: [Freeipa-devel] Client-side command in the IPA framework

2014-02-28 Thread Petr Vobornik
third party products. Parsing on client would be understandable if there was some middle step which would require some action from user, i.e, pick only some tokens to import. -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-27 Thread Petr Vobornik
On 27.2.2014 16:51, Nathaniel McCallum wrote: On Thu, 2014-02-27 at 13:35 +0100, Petr Vobornik wrote: On 21.2.2014 15:24, Petr Vobornik wrote: On 10.2.2014 14:12, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI. Last 5 patches is

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-27 Thread Petr Vobornik
On 21.2.2014 15:24, Petr Vobornik wrote: On 10.2.2014 14:12, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI. Last 5 patches is the OTP UI. First 6 patches is a little refactoring/bug fixes needed for them. General password dialog is

Re: [Freeipa-devel] [PATCH] 6 webui: Too big font in input fields

2014-02-27 Thread Petr Vobornik
On 26.2.2014 18:24, Adam Misnyovszki wrote: - Original Message - From: "Petr Vobornik" To: "Adam Misnyovszki" , freeipa-devel@redhat.com, "Martin Kosek" Sent: Wednesday, February 26, 2014 2:32:52 PM Subject: Re: [Freeipa-devel] [PATCH] Too big font in

Re: [Freeipa-devel] Is there RPC documentation?

2014-02-26 Thread Petr Vobornik
ct requests/responses in network tab. -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] Too big font in input fields

2014-02-26 Thread Petr Vobornik
of jammed radios and labels. Martin, can we steal this CSS ticket and fix it with? input[type=radio], input[type=checkbox], .ui-widget input[type=radio], .ui-widget input[type=checkbox]{ margin-right: 5px; position: relative; top: 2px; } -- Petr Vobornik

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-26 Thread Petr Vobornik
On 26.2.2014 01:55, Dmitri Pal wrote: On 02/24/2014 10:21 AM, Nathaniel McCallum wrote: On Mon, 2014-02-24 at 15:48 +0100, Petr Vobornik wrote: On 24.2.2014 15:31, Nathaniel McCallum wrote: On Mon, 2014-02-24 at 11:04 +0100, Petr Vobornik wrote: On 21.2.2014 20:00, Nathaniel McCallum wrote

[Freeipa-devel] [PATCH] 549 webui: use unique ids for checkboxes

2014-02-25 Thread Petr Vobornik
icking' as default check method. https://fedorahosted.org/freeipa/ticket/3904 -- Petr Vobornik From e64cbab0e4a3ef427e4f74e6f1183a9231dfb6c5 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Thu, 16 Jan 2014 13:54:03 +0100 Subject: [PATCH] webui: use unique ids for checkboxes Checkboxes have

[Freeipa-devel] [PATCH] 548 webui: change ipatokennotbefore and ipatokennotafter types to datetime

2014-02-25 Thread Petr Vobornik
Depends on tbabej's patches # 137, 140 and pvoborni's 546 and 531-541. https://fedorahosted.org/freeipa/ticket/3369 -- Petr Vobornik From 7a9d5636978b77a6e1119bc7e5724a811fcbf669 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Wed, 15 Jan 2014 14:19:41 +0100 Subject: [PATCH] web

[Freeipa-devel] [PATCH] 546 webui: expose krbprincipalexpiration

2014-02-25 Thread Petr Vobornik
Depends on tbabej's patches # 137, 138 and my 546. https://fedorahosted.org/freeipa/ticket/3306 -- Petr Vobornik From 7b4a35b811a8918adefa281e00ee06fad5f54064 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Wed, 15 Jan 2014 14:15:24 +0100 Subject: [PATCH] webui: expose krbprincipalexpir

[Freeipa-devel] [PATCH] 545 webui: Don't act on keyboard events which originated in, different dialog

2014-02-25 Thread Petr Vobornik
es the key 7. second dialog reacts to keyup event - which is by default confirmation mixin's confirm event 8. UNDESIRED behavior occurs Now confirmation mixin remembers which keys were pressed and released and reacts only to those which originated there. https://fedorahosted.org/freeipa/tic

[Freeipa-devel] [PATCH] 544 webui: Focus expand/collapse link in batch_error dialog

2014-02-25 Thread Petr Vobornik
Dialog loses focus when the links are clicked making the dialog uncontrollable by keyboard. This patch focuses the link again after expanding/collapsing the error list. Thus keeping the focus in a dialog https://fedorahosted.org/freeipa/ticket/4097 -- Petr Vobornik From

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-24 Thread Petr Vobornik
On 24.2.2014 15:31, Nathaniel McCallum wrote: On Mon, 2014-02-24 at 11:04 +0100, Petr Vobornik wrote: On 21.2.2014 20:00, Nathaniel McCallum wrote: Is it possible to do something more intelligent for the key and date fields in the add-token UI? Date fields are currently just a text box. Is

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-24 Thread Petr Vobornik
would be better to fill the key and convert it to base32 string on a server. Nathaniel On Fri, 2014-02-21 at 15:24 +0100, Petr Vobornik wrote: On 10.2.2014 14:12, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI. Last 5 patche

Re: [Freeipa-devel] [PATCH]Extending user plugin with employeenumber field

2014-02-21 Thread Petr Vobornik
ployee within an organization' EQUALITY caseIgnoreMatch SUBSTR caseIgnoreSubstringsMatch SYNTAX 1.3.6.1.4.1.1466.115.121.1.15 SINGLE-VALUE ) -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-21 Thread Petr Vobornik
On 10.2.2014 14:12, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI. Last 5 patches is the OTP UI. First 6 patches is a little refactoring/bug fixes needed for them. General password dialog is introduced to avoid another

Re: [Freeipa-devel] [PATCHES] 0460-0463 - Fixes in project files

2014-02-12 Thread Petr Vobornik
dhat. kbaker is a different person. 0461: Contributors.txt Jenny changed her last name. 0462: README & BUILD.txt Update README with current info from http://www.freeipa.org/page/Leaflet, fix broken links, refresh build instructions. 0463: Remove TODO This file was just ancient cruft. We

Re: [Freeipa-devel] [PATCH 0037] Fix OTP token names/labels

2014-02-11 Thread Petr Vobornik
s the main purpose of object_name/s? I can see that it's sometimes used in error messages, IMO not very correct usage - label might be better. -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-10 Thread Petr Vobornik
On 10.2.2014 17:49, Nathaniel McCallum wrote: On Mon, 2014-02-10 at 17:16 +0100, Petr Vobornik wrote: On 10.2.2014 17:09, Nathaniel McCallum wrote: On Mon, 2014-02-10 at 14:12 +0100, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-10 Thread Petr Vobornik
On 10.2.2014 17:03, Nathaniel McCallum wrote: On Mon, 2014-02-10 at 14:12 +0100, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI. Last 5 patches is the OTP UI. First 6 patches is a little refactoring/bug fixes needed for them

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-10 Thread Petr Vobornik
On 10.2.2014 17:09, Nathaniel McCallum wrote: On Mon, 2014-02-10 at 14:12 +0100, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI. Last 5 patches is the OTP UI. First 6 patches is a little refactoring/bug fixes needed for them

Re: [Freeipa-devel] Web services in freeIPA

2014-02-08 Thread Petr Vobornik
ot;-vv" which will show you how we call the API: $ ipa -vv user-show admin Martin [1] http://adam.younglogic.com/2010/07/talking-to-freeipa-json-web-api-via-curl/ On 02/06/2014 04:04 PM, Alexandre Santos wrote: Is there any examples that can guide me. Thanks Alexandre Santos On 06 Feb

Re: [Freeipa-devel] Web services in freeIPA

2014-02-06 Thread Petr Vobornik
ssion/json JSON RPC API with session support https://hostname/ipa/ui - Web UI https://hostname/ipa/config/unauthorized.html - some config and error pages We don't have docs for the APIs yet. -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-06 Thread Petr Vobornik
On 5.2.2014 18:54, Alexander Bokovoy wrote: On Wed, 05 Feb 2014, Nathaniel McCallum wrote: On Tue, 2014-01-21 at 17:45 +0100, Petr Vobornik wrote: from ipaserver.dcerpc import DomainValidator Patch 541 is NACK because ipaserver.dcerpc only exists in freeipa-server-trust-ad. I agree. Instead

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-06 Thread Petr Vobornik
The sent patch was exactly the same as 537-1. Correct patch attached. On 6.2.2014 13:36, Petr Vobornik wrote: On 5.2.2014 20:59, Nathaniel McCallum wrote: The Add Token dialog asks the user to specify a unique id. This unique id should be generated instead. See how I did this in the CLI. It

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-06 Thread Petr Vobornik
Web UI, because it might affect details pages in unwanted manner. Because of that, otptoken dialog's `ipatokenuniqueid` field now use `required=false` override. On Tue, 2014-01-21 at 17:45 +0100, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implement

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-05 Thread Petr Vobornik
On 4.2.2014 19:00, Nathaniel McCallum wrote: On Tue, 2014-02-04 at 18:44 +0100, Petr Vobornik wrote: On 4.2.2014 17:53, Nathaniel McCallum wrote: On Tue, 2014-01-21 at 17:45 +0100, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI

Re: [Freeipa-devel] [PATCH] 531-541 OTP UI

2014-02-04 Thread Petr Vobornik
On 4.2.2014 17:53, Nathaniel McCallum wrote: On Tue, 2014-01-21 at 17:45 +0100, Petr Vobornik wrote: On 13.1.2014 17:09, Petr Vobornik wrote: Hi, these patches implements the OTP Web UI. Last 5 patches is the OTP UI. First 6 patches is a little refactoring/bug fixes needed for them. General

Re: [Freeipa-devel] OTP Sync Client

2014-01-23 Thread Petr Vobornik
different auth method (i.e., password auth is enabled), he can: - open OTP tokens page - open some token details - select 'sync' action - enter OTP1, OTP2, confirm This would require standard IPA command; something like `otptoken_sync ipatokenuni

Re: [Freeipa-devel] [PATCH] 543 Trust domains Web UI

2014-01-21 Thread Petr Vobornik
On 21.1.2014 10:43, Alexander Bokovoy wrote: On Tue, 21 Jan 2014, Petr Vobornik wrote: On 20.1.2014 18:01, Alexander Bokovoy wrote: On Fri, 17 Jan 2014, Petr Vobornik wrote: Note: this version of the patch is especially prepared for ipa-3-3 branch. Add Web UI counterpart of following CLI

Re: [Freeipa-devel] [PATCH] 543 Trust domains Web UI

2014-01-21 Thread Petr Vobornik
On 20.1.2014 18:01, Alexander Bokovoy wrote: On Fri, 17 Jan 2014, Petr Vobornik wrote: Note: this version of the patch is especially prepared for ipa-3-3 branch. Add Web UI counterpart of following CLI commands: * trust-fetch-domains Refresh list of the domains associated with the trust

[Freeipa-devel] [PATCH] 543 Trust domains Web UI

2014-01-17 Thread Petr Vobornik
. * trustdomain-disable Disable use of IPA resources by the domain of the trust * trustdomain-enable Allow use of IPA resources by the domain of the trust * trustdomain-find Search domains of the trust https://fedorahosted.org/freeipa/ticket/4119 -- Petr Vobornik From 152aaf52f7daaf50ae44a4761849603756c186d1

Re: [Freeipa-devel] [PATCH] 0135 resolve SIDs to names in group-show for external members

2014-01-17 Thread Petr Vobornik
Web UI. -- Petr Vobornik From f55645dc41728e1ac453633babbcfe4b39568551 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Fri, 17 Jan 2014 12:43:11 +0100 Subject: [PATCH] Remove SID resolve call from Web UI - it's called in group-show https://bugzilla.redhat.com/show_bug.cgi?id=1054391

Re: [Freeipa-devel] [PATCH] 463-530 First part of RCUE adoption

2014-01-16 Thread Petr Vobornik
On 3.1.2014 12:38, Petr Vobornik wrote: On 29.11.2013 17:02, Ana Krivokapic wrote: On 11/29/2013 02:45 PM, Ana Krivokapic wrote: >On 11/25/2013 04:48 PM, Ana Krivokapic wrote: >>On 11/15/2013 02:26 PM, Petr Vobornik wrote: >>>Hello list, >>> >>>this is

Re: [Freeipa-devel] [PATCH 0137] ipalib: Add DateTime parameter

2014-01-13 Thread Petr Vobornik
n str(value) Return datetime object here please, or at least unicode in generalized time format (or whatever the preferred format is, see above). +elif isinstance(val, datetime.datetime): +return val.strftime("%Y%m%dT%H:%M:%S") Again, please use the preferred format here please. Honza -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] FreeIPA OTP End-to-End

2014-01-03 Thread Petr Vobornik
DS plugin patch. I will merge this last patch in when submitted to the list. OTP CLI All of the patches are merged except npmccallum-0024, which is undergoing active review. https://www.redhat.com/archives/freeipa-devel/2013-December/msg00102.html OTP UI Thanks to Petr Vobornik for his set of patch

Re: [Freeipa-devel] [PATCH] 531 Increase stack size for Web UI builder

2013-12-13 Thread Petr Vobornik
On 12/13/2013 10:28 AM, Alexander Bokovoy wrote: On Fri, 13 Dec 2013, Petr Vobornik wrote: Web UI build fails on some architectures or configuration due to StackOverflow. This patch increases the stack size to solve it. 512k is usually enough but we encountered fail on ppc64 even with 2m

[Freeipa-devel] [PATCH] 531 Increase stack size for Web UI builder

2013-12-13 Thread Petr Vobornik
Petr Vobornik From 9eeb52abaa6a53bb3304f96dd17d0b1f74083b50 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Wed, 31 Jul 2013 15:12:19 +0200 Subject: [PATCH] Increase stack size for Web UI builder Web UI build fails on some architectures or configuration due to StackOverflow. This patch increases the

Re: [Freeipa-devel] [PATCH] 0333 test_webui: Allow False values in configuration for no_ca, no_dns, has_trusts

2013-12-10 Thread Petr Vobornik
x27;t have the WebUI test environment set up locally. Petr, could you check if it works? Tested, ACK -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

[Freeipa-devel] [PATCH] 531 Fix license in some Web UI files

2013-12-06 Thread Petr Vobornik
Modified web ui files had incorrect GPLv2 headers instead of GPLv3 ones. All of the affected code is of FreeIPA origin. -- Petr Vobornik >From 83300169da1e7a3a107a8a3c8c4f453380b5bcb1 Mon Sep 17 00:00:00 2001 From: Petr Vobornik Date: Fri, 6 Dec 2013 14:08:07 +0100 Subject: [PATCH] Fix lice

Re: [Freeipa-devel] [PATCH] Fix python setup tools license tags

2013-12-05 Thread Petr Vobornik
On 5.12.2013 16:02, Simo Sorce wrote: On Thu, 2013-12-05 at 15:38 +0100, Petr Vobornik wrote: On 5.12.2013 15:34, Simo Sorce wrote: On Thu, 2013-12-05 at 15:29 +0100, Petr Vobornik wrote: On 5.12.2013 14:09, Petr Viktorin wrote: On 12/03/2013 03:26 PM, Simo Sorce wrote: Some tags escaped

Re: [Freeipa-devel] [PATCH] Fix python setup tools license tags

2013-12-05 Thread Petr Vobornik
On 5.12.2013 15:34, Simo Sorce wrote: On Thu, 2013-12-05 at 15:29 +0100, Petr Vobornik wrote: On 5.12.2013 14:09, Petr Viktorin wrote: On 12/03/2013 03:26 PM, Simo Sorce wrote: Some tags escaped the relicensing we did a long time ago. Simo. Looks good, ACK, pushed to: master

Re: [Freeipa-devel] [PATCH] Fix python setup tools license tags

2013-12-05 Thread Petr Vobornik
est/widget_tests.js: * published by the Free Software Foundation; version 2 only It's most likely a mistake and should be changed. -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel

Re: [Freeipa-devel] [PATCH] 0086 Make Expression field required when adding automember condition

2013-11-27 Thread Petr Vobornik
On 11/26/2013 03:58 PM, Ana Krivokapic wrote: Hello, This patch addresses ticket https://fedorahosted.org/freeipa/ticket/4053. ACK, pushed to master. -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com

Re: [Freeipa-devel] [PATCH] 463-530 First part of RCUE adoption

2013-11-19 Thread Petr Vobornik
On 11/18/2013 08:47 PM, Petr Viktorin wrote: On 11/18/2013 06:17 PM, Petr Vobornik wrote: On 11/15/2013 05:43 PM, Petr Viktorin wrote: On 11/15/2013 03:28 PM, Petr Vobornik wrote: On 11/15/2013 02:40 PM, Petr Viktorin wrote: On 11/15/2013 02:26 PM, Petr Vobornik wrote: [...] It's qu

Re: [Freeipa-devel] [PATCH] 463-530 First part of RCUE adoption

2013-11-18 Thread Petr Vobornik
On 11/15/2013 05:43 PM, Petr Viktorin wrote: On 11/15/2013 03:28 PM, Petr Vobornik wrote: On 11/15/2013 02:40 PM, Petr Viktorin wrote: On 11/15/2013 02:26 PM, Petr Vobornik wrote: Hello list, this is a first part of RCUE adoption effort. Main themes of this patch set are: - use RCUE

Re: [Freeipa-devel] [PATCH] 463-530 First part of RCUE adoption

2013-11-18 Thread Petr Vobornik
On 11/16/2013 04:17 AM, Simo Sorce wrote: On Fri, 2013-11-15 at 14:26 +0100, Petr Vobornik wrote: It's quite a lot of patches so I did not attach them here. You can see the code in my private repo: branch 'rcue'. I can't see it listed here: http://fedorapeople.org/

Re: [Freeipa-devel] [PATCH] 463-530 First part of RCUE adoption

2013-11-15 Thread Petr Vobornik
On 11/15/2013 02:40 PM, Petr Viktorin wrote: On 11/15/2013 02:26 PM, Petr Vobornik wrote: Hello list, this is a first part of RCUE adoption effort. Main themes of this patch set are: - use RCUE navigation <https://fedorahosted.org/freeipa/ticket/3902> - new styles for textboxes, tex

[Freeipa-devel] [PATCH] 463-530 First part of RCUE adoption

2013-11-15 Thread Petr Vobornik
freeipa/ticket/3903> * new styles for standalone pages * necessary responsive enhancement (the ultimate future goal is responsive layout) It's quite a lot of patches so I did not attach them here. You can see the code in my private repo: branch 'rcue'. -- Petr Vobornik _

Re: [Freeipa-devel] [PATCHES] 0072-0074 Add automember rebuild membership to the web UI

2013-11-15 Thread Petr Vobornik
On 11/14/2013 10:55 AM, Petr Vobornik wrote: On 10/29/2013 12:33 PM, Ana Krivokapic wrote: On 10/24/2013 03:48 PM, Petr Vobornik wrote: On 10/16/2013 11:37 PM, Ana Krivokapic wrote: On 09/27/2013 04:38 PM, Petr Vobornik wrote: On 09/25/2013 11:51 AM, Ana Krivokapic wrote: Hello, This patch

Re: [Freeipa-devel] [PATCHES] 0019-0020 Broken Firefox configuration files in freeipa-client package

2013-11-15 Thread Petr Vobornik
On 11/13/2013 04:34 PM, Petr Vobornik wrote: On 11/12/2013 01:48 PM, Martin Basti wrote: On Fri, 2013-11-08 at 16:28 +0100, Petr Vobornik wrote: On 11/05/2013 10:34 AM, Martin Basti wrote: ipa-client-install now allows to configure firefox using --configure-firefox option and optionally with

Re: [Freeipa-devel] [PATCHES] 0072-0074 Add automember rebuild membership to the web UI

2013-11-14 Thread Petr Vobornik
On 10/29/2013 12:33 PM, Ana Krivokapic wrote: On 10/24/2013 03:48 PM, Petr Vobornik wrote: On 10/16/2013 11:37 PM, Ana Krivokapic wrote: On 09/27/2013 04:38 PM, Petr Vobornik wrote: On 09/25/2013 11:51 AM, Ana Krivokapic wrote: Hello, This patch set addresses ticket https://fedorahosted.org

Re: [Freeipa-devel] [PATCHES] 0080-0081 Add userClass attributes for users and hosts

2013-11-14 Thread Petr Vobornik
ng both patches again (the CLI patch has not changed since the last iteration). ACK for Web UI patch (81-03) 80-03 seems good to me as well, but I don't feel strong about my ability to judge correctness of ldap/ipalib part so I let somebody else to look at it. -- Petr Vobornik __

Re: [Freeipa-devel] [PATCHES] 0019-0020 Broken Firefox configuration files in freeipa-client package

2013-11-13 Thread Petr Vobornik
On 11/12/2013 01:48 PM, Martin Basti wrote: On Fri, 2013-11-08 at 16:28 +0100, Petr Vobornik wrote: On 11/05/2013 10:34 AM, Martin Basti wrote: ipa-client-install now allows to configure firefox using --configure-firefox option and optionally with --firefox-dir=INSTALL_DIR option. Old

<    7   8   9   10   11   12   13   14   15   16   >